Bug 240556

Summary:	dev-libs/zziplib-0.13.49-r1: broken configure check
Product:	Gentoo Linux	Reporter:	Elias Pipping <pipping>
Component:	New packages	Assignee:	SpanKY <vapier>
Status:	RESOLVED FIXED
Severity:	normal	CC:	mr_bones_
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
URL:	https://sourceforge.net/tracker/index.php?func=detail&aid=2155649&group_id=6389&atid=306389
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	patch against the m4 file patch against configure

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) gentoo-dev

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 gentoo-dev

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