Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 310335 - sys-devel/m4-1.4.14: bundled gnulib incorrectly redefines posix spawn values on FreeBSD.
Summary: sys-devel/m4-1.4.14: bundled gnulib incorrectly redefines posix spawn values ...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo/Alt
Classification: Unclassified
Component: FreeBSD (show other bugs)
Hardware: All FreeBSD
: High trivial (vote)
Assignee: Gentoo/BSD Team
URL: http://www.freebsd.org/cgi/query-pr.c...
Whiteboard:
Keywords:
Depends on:
Blocks: 320131
  Show dependency tree
 
Reported: 2010-03-20 09:25 UTC by Javier Villavicencio (RETIRED)
Modified: 2010-05-20 17:51 UTC (History)
2 users (show)

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


Attachments
Patch to the ebuild. (m4_ebuild.patch,2.09 KB, patch)
2010-03-24 04:54 UTC, Javier Villavicencio (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Javier Villavicencio (RETIRED) gentoo-dev 2010-03-20 09:25:31 UTC
And that causes m4's internal macros like "esyscmd" to fail at starting subprocesses, it shows up while trying to emerge subversion with this version of m4.

A patch exists on freebsd ports: 
http://www.freebsd.org/cgi/cvsweb.cgi/ports/devel/m4/files/patch-lib_spawn.in.h
Comment 1 Javier Villavicencio (RETIRED) gentoo-dev 2010-03-20 09:29:47 UTC
Just for tracking, will drop -fbsd keywords on this version, 1.4.13 works okay.
@base-system: up to you to apply the patch or not.
Comment 2 Fabian Groffen gentoo-dev 2010-03-20 10:03:34 UTC
Thanks for the heads up, I just masked this version in Prefix' freebsd profile.
Comment 3 SpanKY gentoo-dev 2010-03-20 16:46:27 UTC
anything that is in lib/ isnt specific to m4; it comes from gnulib.  if the change is already in the upstream gnulib code, feel free to commit whatever you need.
Comment 4 Javier Villavicencio (RETIRED) gentoo-dev 2010-03-23 13:53:42 UTC
Just curious, inheriting eutils to patch stuff here wouldn't mess bootstrapping? (or something else like a circular dep?)
Comment 5 Fabian Groffen gentoo-dev 2010-03-23 13:58:51 UTC
As far as I'm concerned, usage of epatch early in the process is fine.
Comment 6 SpanKY gentoo-dev 2010-03-23 19:58:42 UTC
`patch` should be available in stage1, so there should never be a bootstrapping issue with using `epatch`.  if there is, that's a bug in the release process.
Comment 7 Javier Villavicencio (RETIRED) gentoo-dev 2010-03-24 04:54:15 UTC
Created attachment 225067 [details, diff]
Patch to the ebuild.

Already on upstream gnulib:
http://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=commit;h=88baa1ed437db0f062e49b87cff59002ec8f7ab2

And please check the patch to the ebuild, will commit it later if no objections.

Thanks.
Comment 8 Javier Villavicencio (RETIRED) gentoo-dev 2010-03-28 10:35:10 UTC
Fixed in CVS.