On Thu, May 21, 2015 at 09:19:59AM +0100, Dimitri John Ledkov wrote: > On 15 May 2015 at 21:28, Dimitri John Ledkov <[email protected]> > wrote: > > Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=784911 > > Signed-off-by: Dimitri John Ledkov <[email protected]> > > --- > > fsck.btrfs | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/fsck.btrfs b/fsck.btrfs > > index f056a7f..3a92804 100755 > > --- a/fsck.btrfs > > +++ b/fsck.btrfs > > @@ -31,7 +31,7 @@ if [ ! -e $DEV ]; then > > echo "$0: $DEV does not exist" > > exit 8 > > fi > > -if [ "$AUTO" == "false" ]; then > > +if [ "false" = "$AUTO" ]; then > > echo "If you wish to check the consistency of a BTRFS filesystem or" > > echo "repair a damaged filesystem, see btrfs(8) subcommand 'check'." > > fi > > There was a lot of discussion about this trivial patch, mostly because > it's so trivial and has many ways to fix. > > I hope btrfs maintainers could implement & push any of the proposed > ways to resolve this bashism to rule them all =)
If you're ok to do if ! $AUTO... with your signed-off, then I'll add the patch. I was not comfortable to do it right away. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
