338353 – net-dns/bind: init script contains bashisms

Bug 338353 - net-dns/bind: init script contains bashisms

Summary: net-dns/bind: init script contains bashisms

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	[OLD] Server (show other bugs)
Hardware:	All Linux

Importance:	High enhancement (vote)
Assignee:	Christian Ruppert (idl0r)

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2010-09-22 11:37 UTC by Peter Volkov (RETIRED)
Modified:	2010-10-09 21:26 UTC (History)
CC List:	2 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Peter Volkov (RETIRED) gentoo-dev

2010-09-22 11:37:23 UTC

Hi. openrc is shell agnostic and thus bind should not use bashisms in init script. Only POSIX constructions should be there. net-dns/bind-9.7.1_p2 has them.
Setting Severity to major since this is regression in the stable tree.

Comment 1 Christian Ruppert (idl0r) gentoo-dev

2010-09-22 12:01:33 UTC

I used "dash -n" to verify it is POSIX conform. Can you please give me some more information?

Comment 2 Peter Volkov (RETIRED) gentoo-dev

2010-09-22 14:47:58 UTC

[[ is reserved and thus dash -n does not issue error but behavior of [[ command is not specified. So, don't use [[ for tests.

http://www.mail-archive.com/bug-bash@gnu.org/msg07882.html

Comment 3 Peter Volkov (RETIRED) gentoo-dev

2010-09-22 14:48:35 UTC

Changing Severity since it looks like [[ should work at least in some shels.

Comment 4 SpanKY gentoo-dev

2010-10-09 21:21:48 UTC

POSIX compliance is not a requirement

Comment 5 Christian Ruppert (idl0r) gentoo-dev

2010-10-09 21:26:19 UTC

Already fixed (hopefully) in 9.4.3_p5-r1, 9.6.2_p2-r1, 9.7.1_p2-r1 and above.