Bug 438272

Summary:	app-text/wv2-0.4.2 giochannel.h direct inclusion
Product:	Gentoo Linux	Reporter:	Attila Tóth <atoth>
Component:	Current packages	Assignee:	Gentoo KDE team <kde>
Status:	RESOLVED FIXED
Severity:	normal	CC:	adaptee, tex
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	406437

Description Attila Tóth 2012-10-13 21:29:21 UTC

glib/giochannel.h must be replaced with glib.h (new glib requirement) in src/olestream.h. Can be done using a simple sed line or a patch.

Reproducible: Always

Comment 1 Michael Palimaka (kensington) gentoo-dev

2012-10-16 14:38:52 UTC

Thanks, fixed in CVS.

+  16 Oct 2012; Michael Palimaka <kensington@gentoo.org> +wv2-0.4.2-r1.ebuild,
+  metadata.xml:
+  Add patch from upstream bugtracker to fix build with glib-2.32. Migrate to
+  EAPI 4. Add upstream metadata.

Comment 2 Jekyll Wu 2012-10-16 18:41:13 UTC

(In reply to comment #1)
> Thanks, fixed in CVS.
> 
> +  16 Oct 2012; Michael Palimaka <kensington@gentoo.org>
> +wv2-0.4.2-r1.ebuild,
> +  metadata.xml:
> +  Add patch from upstream bugtracker to fix build with glib-2.32. Migrate to
> +  EAPI 4. Add upstream metadata.

Did you forget to add the patch? I fail to find the patch through http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/app-text/wv2/?hideattic=1

Comment 3 Michael Palimaka (kensington) gentoo-dev

2012-10-17 04:32:04 UTC

(In reply to comment #2)
> Did you forget to add the patch? I fail to find the patch through
> http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/app-text/wv2/
> ?hideattic=1

Oops! Thanks for noticing, fixed now.