FS#15775 - [nfs-utils] start-statd is not valid /bin/sh script

Attached to Project: Arch Linux
Opened by Dan McGee (toofishes) - Tuesday, 04 August 2009, 00:54 GMT
Last edited by Tobias Powalowski (tpowa) - Sunday, 10 January 2010, 16:55 GMT
Task Type Bug Report
Category Packages: Core
Status Closed
Assigned To Tobias Powalowski (tpowa)
Thomas Bächler (brain0)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

When starting the netfs "daemon", you can see errors like this when /bin/sh is linked to something other than bash:

$ sudo /etc/rc.d/netfs restart
:: Unmounting Network Filesystems [DONE]
:: Mounting Network Filesystems [BUSY]
/bin/sh: Illegal option -p
mount.nfs: rpc.statd is not running but is required for remote locking.
mount.nfs: Either use '-o nolock' to keep locks local, or start statd.
[FAIL]


This is due to the /usr/sbin/start-statd program having the opening line "#!/bin/sh -p", and -p is not a valid POSIX option.

Do you guys know where to contact upstream and/or leave a bug report? The mailing list on sourceforge seems awfully full of spam. Anyway, as we patch this file in Arch to use conf.d, I've attached a working start-statd script that is fully sh-compatible.
   start-statd (0.5 KiB)
This task depends upon

Closed by  Tobias Powalowski (tpowa)
Sunday, 10 January 2010, 16:55 GMT
Reason for closing:  Fixed
Additional comments about closing:  1.2.1-3
Comment by Gerardo Exequiel Pozzi (djgera) - Tuesday, 04 August 2009, 01:38 GMT
In the patch for Arch Linux [#1] the -p is not present. Seems that is based in another version.

[#1] http://repos.archlinux.org/viewvc.cgi/nfs-utils/repos/core-i686/start-statd.patch?view=co
Comment by Dan McGee (toofishes) - Tuesday, 04 August 2009, 02:57 GMT
I know, the -p is in the upstream version, thus the reason I really want to talk to upstream but couldn't find the relevant place to do so.
Comment by Tobias Powalowski (tpowa) - Sunday, 10 January 2010, 09:43 GMT
Dan isn't this fixed in latest nfs-utils package? there is no /bin/sh -p at the start line of start-statd
Comment by Dan McGee (toofishes) - Sunday, 10 January 2010, 16:03 GMT
Am I missing something then?

dmcgee@galway ~
$ head -n1 /usr/sbin/start-statd
#!/bin/sh -p
Comment by Tobias Powalowski (tpowa) - Sunday, 10 January 2010, 16:17 GMT
ups, sorry was just looking at tom's startd patch there is no -p, shall i patch it out?
Comment by Dan McGee (toofishes) - Sunday, 10 January 2010, 16:21 GMT
That would be great. I reported another bug to them upstream the other day after I found the ML, I wonder if I should report this one too...

Loading...