severity 538759 wishlist
thanks

On Sun, 26 Jul 2009, Ryan Niebur wrote:
> bash jade experimental debhelper $ ls -l 
> /usr/share/perl5/Debian/Debhelper/Buildsystem/waf.pm
> -rw-r--r-- 1 ryan52 root 949 2009-07-26 12:44 
> /usr/share/perl5/Debian/Debhelper/Buildsystem/waf.pm
> bash jade experimental debhelper $ sudo perltidy -io -et 4 -b 
> /usr/share/perl5/Debian/Debhelper/Buildsystem/waf.pm
> bash jade experimental debhelper $ ls -l 
> /usr/share/perl5/Debian/Debhelper/Buildsystem/waf.pm
> -rw-r--r-- 1 root root 949 2009-07-26 12:44 
> /usr/share/perl5/Debian/Debhelper/Buildsystem/waf.pm
> bash jade experimental debhelper $ 

First off, running perltidy as root is not a really good idea.

Second, I'm not convinced that chown'ing the file which is created
when you use -b to the user of the original file is what one would
expect; -b basically behaves as perl's -i.bak does. (It renames to
foo.bak and opens a new file.)

That said, the documentation of -b should probably be fixed to clarify
this. If you agree with my assessment above, I'll adjust the
documentation.


Don Armstrong

-- 
This message brought to you by weapons of mass destruction related
program activities, and the letter G.

http://www.donarmstrong.com              http://rzlab.ucr.edu



-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to