On Fri, 2008-03-14 at 20:13 -0600, Raphael Geissert wrote:
> Attached is a simple hack that adds an option (which might probably be 
> renamed 
> to something more meaningful) that makes checkbashisms also display variable 
> assignations.

They're generally referred to assignments, as assignation has an
additional meaning which I suspect wasn't intended here. ;-)

> Example:
> [...]
> $ ./checkbashisms.pl -x test
> possible bashism in test line 3 ($RANDOM):
> RANDOM=foo
> possible bashism in test line 4 ($RANDOM):
> echo $RANDOM
> 
> This feature will be very helpful (specially when performing the archive wide 
> check) to detect scripts which fall back to something else when the expected 
> bash behaviour isn't detected.

I've implemented this as --extra (still -x) and moved the $BASH check to
only be flagged in --extra mode ($BASH_foo are still in the main set of
checks).

Regards,

Adam




-- 
To unsubscribe, send mail to [EMAIL PROTECTED]

Reply via email to