rsync - Apr 2008 - DO NOT REPLY [Bug 5380] New: %M in log format converts following spaces into '-'

https://bugzilla.samba.org/show_bug.cgi?id=5380

           Summary: %M in log format converts following spaces into '-'
           Product: rsync
           Version: 3.0.2
          Platform: x86
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P3
         Component: core
        AssignedTo: wayned@samba.org
        ReportedBy: wolfgang.mayer@unisa.edu.au
         QAContact: rsync-qa@samba.org


Consider 

rsync -an --out-format="%M %f" /home /tmp

In the output, all spaces following %M are converted to '-'. (When using
tabs
to delimit the format specifiers, the problem does not seem to occur.)
The same happens when using custom log formats.

This is on Linux [Fedora 7, 2.6.21-1.3228.fc7, x86_64, glibc 2.6].


-- 
Configure bugmail: https://bugzilla.samba.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

https://bugzilla.samba.org/show_bug.cgi?id=5380





------- Comment #1 from wolfgang.mayer@unisa.edu.au  2008-04-08 19:14 CST
-------
Created an attachment (id=3252)
 --> (https://bugzilla.samba.org/attachment.cgi?id=3252&action=view)
proposed fix (the argument to strchr in log.c seems to be incorrect)


-- 
Configure bugmail: https://bugzilla.samba.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

https://bugzilla.samba.org/show_bug.cgi?id=5380


wolfgang.mayer@unisa.edu.au changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
Attachment #3252 is|0                           |1
           obsolete|                            |




------- Comment #2 from wolfgang.mayer@unisa.edu.au  2008-04-08 19:31 CST
-------
Created an attachment (id=3253)
 --> (https://bugzilla.samba.org/attachment.cgi?id=3253&action=view)
make the previous patch more efficient (don't rescan the already processed
string)


-- 
Configure bugmail: https://bugzilla.samba.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

https://bugzilla.samba.org/show_bug.cgi?id=5380


wayned@samba.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




------- Comment #3 from wayned@samba.org  2008-04-15 10:35 CST -------
Thanks for both the report and the patch!

I have applied this fix to the git repository.


-- 
Configure bugmail: https://bugzilla.samba.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.