Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 208326 - sci-mathematics/axiom-200711: gcl-2.6.7 configure has quoting error
Summary: sci-mathematics/axiom-200711: gcl-2.6.7 configure has quoting error
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Science Mathematics related packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-31 17:50 UTC by Martin Väth
Modified: 2008-02-01 17:52 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 Martin Väth 2008-01-31 17:50:30 UTC
The archive silver/zips/gcl-2.6.7 contains a configure-new.ac (and a corresponding configure.in and configure) which all have the false line
   system=MP-RAS-`${AWK} '{print $3}' /etc/.relid'`
(the last ' is obviously false).
For some reason, the line is "eaten" by bash (presumably with a false interpretation) but building definitely fails if /bin/sh is a symlink to dash (it dies with the expected "unterminated quoted string" error).
Comment 1 Martin Väth 2008-01-31 18:46:58 UTC
Now I see what is going wrong:

Apparently there was a patch for this already by gentoo and by upstream, and the gentoo fix was removed because of the upstream patch.

Unfortunately, the upstream patch is wrong, because it applies to "configure.in" without calling autotools, i.e. "configure" is still wrong.

So the clean solution would probably be to call autotools for regenerating configure or to patch configure as well. But perhaps it is enough to make the upstream patch apply to "configure" instead of configure.in. So as a quick fix, the following will do in src_unpack():
        sed -e "s:configure.in:configure:" \
                -i zips/gcl-2.6.7.configure.in.patch
Comment 2 Markus Dittrich (RETIRED) gentoo-dev 2008-02-01 17:52:56 UTC
Thanks much for pointing this out. This should be fixed
in portage cvs now. 

cheers,
Markus