Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 457776 - eutils.eclass: make_wrapper creates non-POSIX shell scripts
Summary: eutils.eclass: make_wrapper creates non-POSIX shell scripts
Status: RESOLVED INVALID
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-15 23:27 UTC by Julian Ospald
Modified: 2013-02-20 03:00 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Julian Ospald 2013-02-15 23:27:53 UTC
running checkbashisms on a wrapper script created by make_wrapper yields:


# checkbashisms /usr/games/bin/grimrock
possible bashism in /usr/games/bin/grimrock line 5 (export foo=bar should be foo=bar; export foo):
		export LD_LIBRARY_PATH="${LD_LIBRARY_PATH}:/opt/grimrock/lib"
possible bashism in /usr/games/bin/grimrock line 7 (export foo=bar should be foo=bar; export foo):
		export LD_LIBRARY_PATH="/opt/grimrock/lib"
Comment 1 SpanKY gentoo-dev 2013-02-18 05:50:46 UTC
as i've told people before, this stupid checkbashisms script is broken.  the export syntax used here (and in many other places) is correct, and people trying to "fix" that is doing it wrong.
Comment 2 Julian Ospald 2013-02-18 13:52:45 UTC
shouldn't we fix the checkbashisms script then?
Comment 3 SpanKY gentoo-dev 2013-02-19 01:41:42 UTC
feel free to take it up with them.  i have nothing to do with that project.
Comment 4 Diego Elio Pettenò (RETIRED) gentoo-dev 2013-02-19 01:42:47 UTC
Would be nice if it also stopped complaining about `kill -HUP` — as that's by far not a bashism, kill is its own command...