From gvs@road.yandex.ru  Wed Aug  4 14:09:32 2004
Return-Path: <gvs@road.yandex.ru>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 6934E16A4CE
	for <FreeBSD-gnats-submit@freebsd.org>; Wed,  4 Aug 2004 14:09:32 +0000 (GMT)
Received: from road.yandex.ru (road.yandex.ru [213.180.193.165])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 70BFE43D66
	for <FreeBSD-gnats-submit@freebsd.org>; Wed,  4 Aug 2004 14:09:31 +0000 (GMT)
	(envelope-from gvs@road.yandex.ru)
Received: from road.yandex.ru (localhost.yandex.net [127.0.0.1] (may be forged))
	by road.yandex.ru (8.12.11/8.12.11) with ESMTP id i74EDKjt038076
	for <FreeBSD-gnats-submit@freebsd.org>; Wed, 4 Aug 2004 18:13:20 +0400 (MSD)
	(envelope-from gvs@road.yandex.ru)
Received: (from root@localhost)
	by road.yandex.ru (8.12.11/8.12.11/Submit) id i74EDJA7038075;
	Wed, 4 Aug 2004 18:13:19 +0400 (MSD)
	(envelope-from gvs)
Message-Id: <200408041413.i74EDJA7038075@road.yandex.ru>
Date: Wed, 4 Aug 2004 18:13:19 +0400 (MSD)
From: Seva Gluschenko <gvs@rinet.ru>
To: FreeBSD-gnats-submit@freebsd.org
Cc:
Subject: MPlayer-1.0pre5 port can be built with GTK2, finally.
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         70001
>Category:       ports
>Synopsis:       MPlayer-1.0pre5 port can be built with GTK2, finally.
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    vs
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          update
>Submitter-Id:   current-users
>Arrival-Date:   Wed Aug 04 14:10:24 GMT 2004
>Closed-Date:    Mon Aug 16 09:45:31 GMT 2004
>Last-Modified:  Tue Oct 26 22:50:26 GMT 2004
>Originator:     Seva Gluschenko
>Release:        FreeBSD 5.2-CURRENT i386
>Organization:
Yandex LLC
>Environment:
System: FreeBSD road.yandex.ru 5.2-CURRENT FreeBSD 5.2-CURRENT #8: Fri Jun 4 13:37:30 MSD 2004 gvs@road.yandex.ru:/local/obj/local/usr/src/sys/ROAD i386


	
>Description:
	The multimedia/mplayer port has ditto WITH_GTK2 knob which
	does nothing but complains it's unavailable. That's not true.

	I found the patch here:

	http://homepages.ius.edu/CJEZELL/patches/mplayer1.0pre5-gtk2-20040730.patch.bz2

	the description is at

	http://homepages.ius.edu/CJEZELL/projects.html

	I have no clue how to specify master site for extra patchfile,
	so I downloaded it in files/ and adjusted Makefile appropriately.
>How-To-Repeat:
	CVSup and cd to latest multimedia/mplayer. Download the patch
	specified and place it to files/ subdir of multimedia/mplayer port.
	Then apply the patch below:

--- Makefile.orig	Wed Aug  4 18:01:24 2004
+++ Makefile	Wed Aug  4 18:04:23 2004
@@ -43,8 +43,6 @@
 #       gui capabilities.
 #       The GTK2 patches are not official ones, so they should be considered as
 #       experimental.
-# Note: GTK2 patches are not included at the moment. So defining this knob has
-#       no effect until new working patches are available
 #
 # WITHOUT_GUI
 # default: undefined
@@ -372,30 +370,31 @@
 
 .if !defined(WITHOUT_GUI)
 PLIST_SUB+=	GMPLAYER=""
+RUN_DEPENDS+=	${LOCALBASE}/share/mplayer/Skin:${PORTSDIR}/multimedia/mplayer-skins
+CONFIGURE_ARGS+=	--enable-gui
 .else
 PLIST_SUB+=	GMPLAYER="@comment "
 .endif
 
-#.if defined(WITH_GTK2) && !defined(WITHOUT_GUI)
-#USE_GNOME+=	gtk20
-#RUN_DEPENDS+=	${LOCALBASE}/share/mplayer/Skin:${PORTSDIR}/multimedia/mplayer-skins
-#.if defined(PKGNAMESUFFIX)
-#PKGNAMESUFFIX:=	${PKGNAMESUFFIX}-gtk2
-#.else
-#PKGNAMESUFFIX=	-gtk2
-#.endif
-#CONFIGURE_ARGS+=	--enable-gui
-#.endif
+.if defined(WITH_GTK2) && !defined(WITHOUT_GUI)
+USE_GNOME+=	gtk20
+.if defined(PKGNAMESUFFIX)
+PKGNAMESUFFIX:=	${PKGNAMESUFFIX}-gtk2
+.else
+PKGNAMESUFFIX=	-gtk2
+.endif
+.if defined(WITH_LANG)
+WITH_LANG=	""
+.endif
+.endif
 
 .if defined(WITH_GTK1) && !defined(WITHOUT_GUI)
 USE_GNOME+=	gtk12
-RUN_DEPENDS+=	${LOCALBASE}/share/mplayer/Skin:${PORTSDIR}/multimedia/mplayer-skins
 .if defined(PKGNAMESUFFIX)
 PKGNAMESUFFIX:=	${PKGNAMESUFFIX}-gtk
 .else
 PKGNAMESUFFIX=	-gtk
 .endif
-CONFIGURE_ARGS+=	--enable-gui
 .endif
 
 .if defined(WITHOUT_GUI)
@@ -420,7 +419,7 @@
 PLIST_SUB+=	MENCODER="@comment "
 .endif
 
-.if defined(WITH_ARTS) && (!defined(WITH_GTK1) || defined(WITHOUT_GUI))
+.if defined(WITH_ARTS) && (!defined(WITH_GTK1) || defined(WITH_GTK2) || defined(WITHOUT_GUI))
 LIB_DEPENDS+=	artsc.0:${PORTSDIR}/audio/arts
 
 CONFIGURE_ARGS+=	--enable-arts
@@ -623,9 +622,10 @@
 	@${ECHO_MSG} "make WITH_GTK1"
 	@${ECHO_MSG} "builds MPlayer with GTK1-GUI support."
 
-.if defined(WITH_GTK2) && !defined(WITHOUT_GUI)
-	@${ECHO_MSG} "GTK2 plugin is not available at the moment."
-	@${ECHO_MSG} "Falling back to use GTK1 if installed on the system."
+.if defined(WITH_LANG) && defined(WITH_GTK2)
+	@${ECHO_MSG}	"========================== ATTENTION! ============================"
+	@${ECHO_MSG}	"GTK2 has UTF8 compatibility problems. Language selection disabled."
+	@${ECHO_MSG}	"=================================================================="
 .endif
 .if !defined(WITHOUT_GUI)
 	@${ECHO_MSG} "If you want to use the GUI, you can either install"
@@ -635,9 +635,9 @@
 .endif
 
 post-patch:
-#.if defined(WITH_GTK2) && !defined(WITHOUT_GUI)
-#	@cd ${WRKSRC} && ${PATCH} < ${PATCHDIR}/extra-patch-gtk2
-#.endif
+.if defined(WITH_GTK2) && !defined(WITHOUT_GUI)
+	@cd ${WRKSRC} && bzcat ${PATCHDIR}/mplayer1.0pre5-gtk2-20040730.patch.bz2 | ${PATCH} -p1
+.endif
 	@${REINPLACE_CMD} -e \
 		's|libxmms.so.1|libxmms.so|' \
 		${CONFIGURE_WRKSRC}/${CONFIGURE_SCRIPT}
>Fix:

	The Makefile's patch is probably needs to be fixed for using either
	PATCHFILE variable or the patch mentioned needs to be placed to ports
	tree. The known issue of WITH_GTK2 is language problems: trying it
	with WITH_LANG=ru resulted to empty menus and many debug complains to
	the console:

** (<unknown>:24529): WARNING **: Invalid UTF8 string passed to pango_layout_set_text()

	So that I put some message to Makefile and disabled using WITH_LANG
	together with WITH_GTK2.
>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->feedback 
State-Changed-By: vs 
State-Changed-When: Mon Aug 9 14:46:36 GMT 2004 
State-Changed-Why:  
Forwarded PR to maintainer 

http://www.freebsd.org/cgi/query-pr.cgi?pr=70001 

From: "Volker Stolz" <vs@freebsd.org>
To: freebsd-gnats-submit@freebsd.org, gvs@rinet.ru, riggs@rrr.de
Cc:  
Subject: Re: ports/70001: MPlayer-1.0pre5 port can be built with GTK2, finally.
Date: Mon, 09 Aug 2004 16:45:57 +0200

 Dear maintainer, please review the proposed PR!
 
 Cheers,
    Volker
Responsible-Changed-From-To: freebsd-ports-bugs->vs 
Responsible-Changed-By: vs 
Responsible-Changed-When: Mon Aug 16 08:31:49 GMT 2004 
Responsible-Changed-Why:  
I'm on this 

http://www.freebsd.org/cgi/query-pr.cgi?pr=70001 
State-Changed-From-To: feedback->closed 
State-Changed-By: vs 
State-Changed-When: Mon Aug 16 09:45:20 GMT 2004 
State-Changed-Why:  
Committed, thanks! 

http://www.freebsd.org/cgi/query-pr.cgi?pr=70001 

From: "Wolfram A. Kraushaar" <wak@mailsat.dstm.de>
To: freebsd-gnats-submit@FreeBSD.org, vs@FreeBSD.org
Cc:  
Subject: Re: ports/70001: MPlayer-1.0pre5 port can be built with GTK2, finally.
Date: Wed, 27 Oct 2004 00:42:19 +0200

 Just some feedback on mplayer built with gtk2
 
 Mplayer builds fine with gtk2 but dies with signal 11 (SIGSEGV) upon
 clicking ok in the preferences window - all other things work fine
 (command-line - playing movies w/o touching preferences)
 
 uname -a:  FreeBSD a.nonymiz.ed 5.3-STABLE FreeBSD 5.3-STABLE #0: Mon 
 Oct 25 18:42:47 CEST 2004     
 root@a.nonymiz.ed:/usr/obj/usr/src/sys/CUSTOM53  i386
 
 GTK2: gtk-2.4.9_1
 
 mplayer built from ports (cvsuped Oct 25th) with:
 
 -DWITH_OPTIMIZED_CFLAGS
 -DWITHOUT_RUNTIME_CPUDETECTION
 -DWITH_GTK2
 -DWITHOUT_3DNOW
 -DWITHOUT_DSP
 -DWITHOUT_ESOUND
 
 However, building without optimized cflags did the trick for my system, 
 so you
 might consider disabling -DWITH_OPTIMIZED_CFLAGS upon usage of -DWITH_GTK2.
 
 Rgds,
 Wolfram
>Unformatted:
