Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 236868 - sys-apps/sandbox-1.2.18.1: &> in configure script
Summary: sys-apps/sandbox-1.2.18.1: &> in configure script
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Sandbox (show other bugs)
Hardware: All Linux
: High normal
Assignee: Sandbox Maintainers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-09-06 15:12 UTC by Harald van Dijk (RETIRED)
Modified: 2008-11-09 07:31 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 Harald van Dijk (RETIRED) gentoo-dev 2008-09-06 15:12:02 UTC
Somehow I never got around to reporting this before... Please don't use &> in configure scripts. It breaks horribly with dash as my CONFIG_SHELL, which runs $CC in the background to standard output/error, and creates libctest.log as an empty file.

--- sandbox-1.2.18.1/configure.ac
+++ sandbox-1.2.18.1/configure.ac
@@ -131 +131 @@
-$CC $CFLAGS -Wl,-verbose -o libctest libctest.c &> libctest.log
+$CC $CFLAGS -Wl,-verbose -o libctest libctest.c >libctest.log 2>&1
Comment 1 SpanKY gentoo-dev 2008-11-08 14:42:01 UTC
this was fixed in svn a while ago.  if you really care about 1.2.18.1, feel free to commit a patch to the package itself.
Comment 2 Harald van Dijk (RETIRED) gentoo-dev 2008-11-08 20:14:54 UTC
I don't care about 1.2.18.1 in particular, but it isn't fixed in the alphas either. I'd be happy to modify the sandbox ebuild to fix up the configure script, but it requires a modification to configure.ac, which requires either eautoreconf or direct modification of the generated configure script. I don't know if the former would cause any problems (sandbox being a system package and all, quite possibly installed before autotools are), and while I have no problems with the latter, I know at least one rather vocal dev has strong objections to that... Any insights here?
Comment 3 SpanKY gentoo-dev 2008-11-08 22:23:42 UTC
i was referring to the svn repo, not any particular version in the tree

a simple sed on the configure script would fix things up i imagine ... feel free to commit to the tree
Comment 4 Harald van Dijk (RETIRED) gentoo-dev 2008-11-09 07:26:48 UTC
Alrighty, done.
Comment 5 SpanKY gentoo-dev 2008-11-09 07:31:08 UTC
looks good, thanks