Bug 381503 – sys-apps/openrc-0.9.1: mtab uses -l instead of -L

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Helmuth Schmelzer 2011-09-01 23:38:51 UTC]

Use Openrc and runs but, despites a error:

/etc/init.d/mtab: línea 15: [: -l: se esperaba un operador unario
 * Updating /etc/mtab ... [ ok ]


Reproducible: Always

Steps to Reproduce:
1. echo "sys-apps/openrc" >> /etc/portage/package.keywords
2. emerge openrc
3. etc-update 
4. /etc/init.d/mtab restart and shows...
etc/init.d/mtab: línea 15: [: -l: se esperaba un operador unario
 * Updating /etc/mtab ... [ ok ]
Actual Results:  
Mtab init script have a syntax error 

/etc/init.d/mtab: línea 15: [: -l: se esperaba un operador unario
 * Updating /etc/mtab ... [ ok ]

[Comment 1 Helmuth Schmelzer 2011-09-01 23:39:26 UTC]

Created attachment 285297 [details]
rc.log

Log of openrc

[Comment 2 Helmuth Schmelzer 2011-09-01 23:42:53 UTC]

Created attachment 285299 [details]
openrc-0.9.0-r1.ebuild

Ebuild of openrc, with included openrc-0.9.0-mtab_fix_operator.patch

[Comment 3 Helmuth Schmelzer 2011-09-01 23:43:42 UTC]

Created attachment 285301 [details, diff]
openrc-0.9.0-mtab_fix_operator.patch

Patch for fixing missing operator in init script of openrc

[Comment 4 William Hubbs 2011-09-02 03:13:28 UTC]

The "! -w" portion of this patch is not correct because we only want to
test for whether or not /etc/mtab is a symbolic link. If it is, we do
not update it. Also, the comment is incorrect for the same reason.

[Comment 5 William Hubbs 2011-09-02 07:17:24 UTC]

The issue was the use of -l instead of -L in the if statement.
This has been fixed in commit 7a6dbed and will be included in the next
release of openrc.

Thanks for the report.