Re: [PATCH] Fix bashism in fsck.btrfs for debian/ubuntu dash.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



If we just want to fix the bashism, then

        if [ "$AUTO" = "false" ]; then

is a good solution. No point to use a yoda condition.


But I like the style proposed by Florian best. It is closest to
traditional shell programming.

On Sat, May 16, 2015 at 10:27 AM, Florian Gamböck <ml@xxxxxxxx> wrote:
>
> AUTO=
> # ...
> if [ -z "$AUTO" ]; then

Markus

-- 
Markus Baertschi, Bas du Rossé 16, CH-1163, Etoy, Switzerland
markus@xxxxxxxxxx, +41 (79) 403 1186 (mobile)
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux