Bug 151166 - [patch] converters/fribidi fails to build if devel/glib20 is installed
Summary: [patch] converters/fribidi fails to build if devel/glib20 is installed
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: freebsd-gnome (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-03 03:30 UTC by John Hein
Modified: 2010-10-04 02:50 UTC (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description John Hein 2010-10-03 03:30:05 UTC
--with-glib=auto is the default 'configure' behavior.  But 0.19.2
fails to link if glib is detected at the time configure is run:

../lib/.libs/libfribidi.so: undefined reference to `g_assert'

The reason is that g_assert is not defined because 
glib/gtestutils.h is not included.

This was noticed with glib-2.24.2, but likely occurs
with other versions of glib, too.

It was fixed in cvs upstream ...
http://lists.freedesktop.org/archives/fribidi/2010-February/000598.html

... but no new release has been made.

For 0.19.2 in ports, just use --without-glib (which is effectively
how pointyhat builds it since glib is not a dependency).

The patch below does that.  It also fixes the URL in pkg-descr.

Fix: --xWFuU/5WGn
Content-Type: text/plain; name="patch-without-glib"
Content-Description: fix fribidi 0.19.2 if glib installed
Content-Disposition: inline;
	filename="patch-without-glib"
Content-Transfer-Encoding: 7bit



--xWFuU/5WGn----69OTQMRukXmzr97Lu2wXhsZzIybupxsgGyR7Sd1Itxnx0Q0J
Content-Type: text/plain; name="file.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="file.diff"

Index: Makefile
===================================================================
RCS file: /base/FreeBSD-CVS/ports/converters/fribidi/Makefile,v
retrieving revision 1.22
diff -u -p -r1.22 Makefile
--- Makefile	23 Aug 2009 17:17:33 -0000	1.22
+++ Makefile	3 Oct 2010 01:18:30 -0000
@@ -42,6 +42,8 @@ MAN3=		fribidi_charset_to_unicode.3 frib
 CONFIGURE_ARGS=	--disable-debug
 .endif
 
+CONFIGURE_ARGS+=	--without-glib
+
 post-patch:
 	@${SED} -e 's|%%LOCALBASE%%|${LOCALBASE}|g' < \
 	    	${FILESDIR}/fribidi-config > ${WRKDIR}/fribidi-config
Index: pkg-descr
===================================================================
RCS file: /base/FreeBSD-CVS/ports/converters/fribidi/pkg-descr,v
retrieving revision 1.3
diff -u -p -r1.3 pkg-descr
--- pkg-descr	12 Sep 2005 07:29:08 -0000	1.3
+++ pkg-descr	3 Oct 2010 01:23:56 -0000
@@ -17,4 +17,4 @@ every Unicode character will be treated 
 specification. The same is true for the mirroring of characters, which also
 works for all the characters listed as mirrorable in the Unicode specification.
 
-WWW: http://fribidi.org/wiki/
+WWW: http://fribidi.org/
How-To-Repeat: 
Install devel/glib20.
Then try to build converters/fribidi.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2010-10-03 03:30:14 UTC
Responsible Changed
From-To: freebsd-ports-bugs->gnome

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 swell.k 2010-10-03 04:34:13 UTC
I think it affects only default LOCALBASE. With non-default one
configure script won't find pkg-config and won't check glib version.
Comment 3 dfilter service freebsd_committer freebsd_triage 2010-10-03 16:45:42 UTC
mezz        2010-10-03 15:45:37 UTC

  FreeBSD ports repository

  Modified files:
    converters/fribidi   Makefile pkg-descr 
  Log:
  -Disable glib for it will not enable if it exists in the system.
  -Update the WWW.
  
  PR:             ports/151166
  Submitted by:   John Hein <jhein@symmetricom.com>
  
  Revision  Changes    Path
  1.23      +3 -2      ports/converters/fribidi/Makefile
  1.4       +1 -1      ports/converters/fribidi/pkg-descr
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 4 Jeremy Messenger freebsd_committer freebsd_triage 2010-10-03 16:45:57 UTC
State Changed
From-To: open->closed

Committed, thanks!
Comment 5 John Hein 2010-10-03 16:53:19 UTC
Anonymous wrote at 07:34 +0400 on Oct  3, 2010:
 > I think it affects only default LOCALBASE. With non-default one
 > configure script won't find pkg-config and won't check glib version.

Sure.  Putting it another way, if pkg-config is not in your path or
glib-2.0.pc is not in PKG_CONFIG_PATH and/or the pkg-config default
search path, then fribidi's configure will not detect glib and the
build of fribidi will not fail.

But in most cases I think it's accurate to say that if someone has
devel/glib20 installed, fribidi's build will fail.
Comment 6 swell.k 2010-10-03 18:02:53 UTC
John Hein <jhein@symmetricom.com> writes:

> Anonymous wrote at 07:34 +0400 on Oct  3, 2010:
>  > I think it affects only default LOCALBASE. With non-default one
>  > configure script won't find pkg-config and won't check glib version.
>
> Sure.  Putting it another way, if pkg-config is not in your path or

FYI, pkg-config *is* in my PATH. If I move PKG_CHECK_MODULES out of
if-then-else condition it works, it also works if I define PKG_CONFIG in
environment.
Comment 7 John Hein 2010-10-04 02:48:45 UTC
For the sake of PR posterity, upstream comments on the need (or not)
for glib...

http://lists.freedesktop.org/archives/fribidi/2010-October/000605.html

Basically, it's currently not needed (but I imagine for some platforms
might help with portability issues).