Bug 240556 – dev-libs/zziplib-0.13.49-r1: broken configure check

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Elias Pipping 2008-10-08 18:34:06 UTC]

-e 's,.*% *,,'

  is called out of nowhere which should obviously be

sed -e 's,.*% *,,'

attaching a patch that takes care of it. As a result of bug #239472,
configure is already patched so that this patch (which requires one
to run eautoreconf afterwards) might be overkill and patching configure
instead might be the better choice here. the line in configure reads

  ac_cv_cflags_no_writable_strings=`echo $ac_arg | -e 's,.*% *,,'` ; break

[Comment 1 Elias Pipping 2008-10-08 18:34:25 UTC]

Created attachment 167642 [details, diff]
patch against the m4 file

[Comment 2 Elias Pipping 2008-10-08 18:38:00 UTC]

Created attachment 167644 [details, diff]
patch against configure

[Comment 3 Wormo (RETIRED) 2008-10-09 06:24:24 UTC]

Good catch, and thanks for both proposed patches so maintainer can use whichever makes more sense. Assigning to maintainers

[Comment 4 SpanKY 2008-10-09 19:45:05 UTC]

thanks, pushed upstream and added to our cvs

http://sources.gentoo.org/dev-libs/zziplib/zziplib-0.13.49-r1.ebuild?r1=1.1&r2=1.2
http://sources.gentoo.org/dev-libs/zziplib/files/zziplib-0.13.49-configure-sed.patch?rev=1.1