smb.conf example: Clarify comment for max log size

Bug #182566 reported by Richard Laager
2
Affects Status Importance Assigned to Milestone
samba
Invalid
Low
samba (Debian)
Fix Released
Unknown
samba (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

This is a bit pedantic, but when I was studying the smb.conf file, this
seemed odd, so I went to verify it. It seems like an easy fix to clarify
this to save others the trouble:

The comment for max log size says "Kb". It should say "KiB" or at least
"KB", as it's in bytes, not bits. I'm attaching a patch which shows the
change. It would probably make sense to change the value from 1000 to
1024, which would make it an even 1 MiB. It seems odd to have a power of
ten value for a field that's in a power of two units.

Also, I found the "Put a capping" wording awkward, so this patch changes
that as well.

Related branches

Revision history for this message
Richard Laager (rlaager) wrote :
Changed in samba:
status: Unknown → Confirmed
Changed in samba:
status: Unknown → Confirmed
Revision history for this message
Chuck Short (zulcss) wrote :

Thanks for the bug report.

Changed in samba:
importance: Undecided → Wishlist
status: New → Confirmed
Revision history for this message
Nicolas Valcarcel (nvalcarcel) wrote :

Debdiff uploaded on Bug #182567

Changed in samba:
status: Confirmed → New
Changed in samba:
status: Confirmed → Invalid
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package samba - 3.0.28-1ubuntu3

---------------
samba (3.0.28-1ubuntu3) hardy; urgency=low

  * Added additional files to winbind.files. (LP: #156468)
  * Deleted winbind enum comment inconsistency. Thanks to Richard Laager for
    the patches. (LP: #182569)
  * Deleted configuration directive inconsistences. Thanks to Richard Laager
    for the patches. (LP: #182567)
  * Fixed typo on net(8) man pages using fix-documentation.patch. (LP: #182570)
  * Clarrified comment for max log size. Thanks to Richard Laager for the
    patches. (LP: #182566)

 -- Chuck Short <email address hidden> Tue, 22 Jan 2008 09:53:52 -0500

Changed in samba:
status: Confirmed → Fix Released
Changed in samba:
status: New → Fix Released
Changed in samba:
importance: Unknown → Low
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.