Re: rpms/mtools/devel mtools-3.9.10-sh.patch,NONE,1.1

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

 



Patrice Dumas napsal(a):
On Fri, May 11, 2007 at 04:54:43AM -0400, Adam Tkac wrote:
Author: atkac

mtools-3.9.10-sh.patch:

I may be completly wrong, but in my recollection using
arithmetic/conditionals like

+if [[ $# -eq 0 ]] ; then

is not sh compatible but requires some bash extensions.
It should be, instead, along

if [ $# = 0 ]; then

== also in [ ] is not portable.

I am not sure and this would benefit from double checking. In any
case, for fedora it should be fine even with the bash extensions, and
the shebang also may be changed to #! /bin/bash to be safe, but if it is to be submitted upstream (as it should be)
you should really take care that no bash extensions are used.

--
Pat

Ah, I've tested it on my rawhide machine and I forgot that sh is symlink to bash ( => script works for me) :)

I'm going to change /bin/sh to /bin/bash immediately.

Regards, -A-

--
fedora-devel-list mailing list
fedora-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-devel-list

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux