From nobody@FreeBSD.org  Sat Oct  6 22:43:52 2012
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id 92DA51065670
	for <freebsd-gnats-submit@FreeBSD.org>; Sat,  6 Oct 2012 22:43:52 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from red.freebsd.org (red.freebsd.org [IPv6:2001:4f8:fff6::22])
	by mx1.freebsd.org (Postfix) with ESMTP id 7CB2C8FC16
	for <freebsd-gnats-submit@FreeBSD.org>; Sat,  6 Oct 2012 22:43:52 +0000 (UTC)
Received: from red.freebsd.org (localhost [127.0.0.1])
	by red.freebsd.org (8.14.5/8.14.5) with ESMTP id q96MhpOf082833
	for <freebsd-gnats-submit@FreeBSD.org>; Sat, 6 Oct 2012 22:43:51 GMT
	(envelope-from nobody@red.freebsd.org)
Received: (from nobody@localhost)
	by red.freebsd.org (8.14.5/8.14.5/Submit) id q96Mhpil082832;
	Sat, 6 Oct 2012 22:43:51 GMT
	(envelope-from nobody)
Message-Id: <201210062243.q96Mhpil082832@red.freebsd.org>
Date: Sat, 6 Oct 2012 22:43:51 GMT
From: michael copeland <michael@kryptos-security.com>
To: freebsd-gnats-submit@FreeBSD.org
Subject: Building mplayer on 9.1-PRERELEASE errors with host architecture "UNKNOWN"
X-Send-Pr-Version: www-3.1
X-GNATS-Notify: thomas.e.zander@googlemail.com

>Number:         172416
>Category:       ports
>Synopsis:       Building multimedia/mplayer on 9.1-PRERELEASE errors with host architecture "UNKNOWN"
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Sat Oct 06 22:50:03 UTC 2012
>Closed-Date:    Fri Jan 18 15:54:11 EST 2013
>Last-Modified:  Fri Jan 18 21:00:01 UTC 2013
>Originator:     michael copeland
>Release:        9.1-PRERELEASE powerpc
>Organization:
Kryptos Security
>Environment:
FreeBSD mini.kryptos-security.com 9.1-PRERELEASE FreeBSD 9.1-PRERELEASE #0: Sat Oct  6 01:06:47 EDT 2012     michael@mini.kryptos-security.com:/usr/obj/usr/src/sys/miniTV  powerpc
>Description:
Configure script fails when detecting $host_arch.

guilty part is:

 # host's CPU/instruction set
  case "$(uname -m 2>&1)" in
      x86_64|amd64|i[3-9]86*|i86pc|x86|x86pc|k5|k6|k6_2|k6_3|k6-2|k6-3|pentium*|athlon*|i586_i686|i586-i686) host_arch=i386 ;;
      ia64) host_arch=ia64 ;;
      macppc|ppc*|Power*) host_arch=ppc ;;


uname -m gives us powerpc, this isn't here, so add it.


 # host's CPU/instruction set
  case "$(uname -m 2>&1)" in
      x86_64|amd64|i[3-9]86*|i86pc|x86|x86pc|k5|k6|k6_2|k6_3|k6-2|k6-3|pentium*|athlon*|i586_i686|i586-i686) host_arch=i386 ;;
      ia64) host_arch=ia64 ;;
      macppc|ppc*|Power*|powerpc) host_arch=ppc ;;


builds and installs now.
>How-To-Repeat:
cd to /usr/ports/multimedia/mplayer  make install
>Fix:
edit configure script in work/mplayer-export-2012-07-21


find section 
# host's CPU/instruction set

patch section  macppc|ppc*|Power*) host_arch=ppc ;;

with            macppc|ppc*|Power*|powerpc) host_arch=ppc ;;



changing system var UNAME_m to ppc would also work, but this may break other ports.

>Release-Note:
>Audit-Trail:
Class-Changed-From-To: maintainer-update->change-request 
Class-Changed-By: edwin 
Class-Changed-When: Sun Oct 7 00:19:11 UTC 2012 
Class-Changed-Why:  
Fix category (submitter is not maintainer) (via the GNATS Auto Assign 
Tool) 

http://www.freebsd.org/cgi/query-pr.cgi?pr=172416 
State-Changed-From-To: open->feedback 
State-Changed-By: edwin 
State-Changed-When: Sun Oct 7 00:19:23 UTC 2012 
State-Changed-Why:  
Awaiting maintainers feedback (via the GNATS Auto Assign Tool) 

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

From: Edwin Groothuis <edwin@FreeBSD.org>
To: thomas.e.zander@googlemail.com
Cc: bug-followup@FreeBSD.org
Subject: Re: ports/172416: Building multimedia/mplayer on 9.1-PRERELEASE errors with host architecture "UNKNOWN"
Date: Sun, 7 Oct 2012 00:19:19 UT

 Maintainer of multimedia/mplayer,
 
 Please note that PR ports/172416 has just been submitted.
 
 If it contains a patch for an upgrade, an enhancement or a bug fix
 you agree on, reply to this email stating that you approve the patch
 and a committer will take care of it.
 
 The full text of the PR can be found at:
     http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/172416
 
 -- 
 Edwin Groothuis via the GNATS Auto Assign Tool
 edwin@FreeBSD.org

From: Thomas Zander <thomas.e.zander@googlemail.com>
To: bug-followup@freebsd.org
Cc: michael copeland <michael@kryptos-security.com>
Subject: Re: ports/172416: Building multimedia/mplayer on 9.1-PRERELEASE
 errors with host architecture "UNKNOWN"
Date: Mon, 8 Oct 2012 19:23:10 +0200

 --047d7b2e770abff86404cb8f7983
 Content-Type: text/plain; charset=ISO-8859-1
 
 Would you find attached patch to the port acceptable?
 If so, this can be committed.
 
 A question or two, if I may.
 What machine is this you are using mplayer on? There aren't that many
 powerpc machines these days.
 Would you test whether clang compiles mplayer on ppc? To test this,
 just add powerpc to the if clause before MPLAYER_CLANG_SUPPORTED_ARCH
 in Makefile.options.
 
 Best regards
 Riggs
 
 --047d7b2e770abff86404cb8f7983
 Content-Type: text/plain; charset=US-ASCII; name="m.diff.txt"
 Content-Disposition: attachment; filename="m.diff.txt"
 Content-Transfer-Encoding: base64
 X-Attachment-Id: f_h81uhfo90
 
 ZGlmZiAtciA2NzQ4NWEyZjU1MjMgbXBsYXllci9maWxlcy9wYXRjaC1jb25maWd1cmUKLS0tIGEv
 bXBsYXllci9maWxlcy9wYXRjaC1jb25maWd1cmUJTW9uIE9jdCAwOCAxODo1NDo1MyAyMDEyICsw
 MjAwCisrKyBiL21wbGF5ZXIvZmlsZXMvcGF0Y2gtY29uZmlndXJlCU1vbiBPY3QgMDggMTk6MTc6
 NDAgMjAxMiArMDIwMApAQCAtMSw2ICsxLDYgQEAKLS0tLSBjb25maWd1cmUub3JpZwkyMDEyLTAz
 LTE1IDE2OjIwOjAyLjAwMDAwMDAwMCArMDEwMAotKysrIGNvbmZpZ3VyZQkyMDEyLTAzLTIyIDE1
 OjM2OjE2LjU4MzAwNjA3MyArMDEwMAotQEAgLTYzNyw3ICs2MzcsNyBAQAorLS0tIGNvbmZpZ3Vy
 ZS5vcmlnCTIwMTItMDctMjAgMTU6MTE6NDIuMDAwMDAwMDAwICswMjAwCisrKysgY29uZmlndXJl
 CTIwMTItMTAtMDggMTk6MTY6MTkuNjM3Nzk3NDYyICswMjAwCitAQCAtNjQ1LDcgKzY0NSw3IEBA
 CiAgX2l3bW14dD1hdXRvCiAgX210cnI9YXV0bwogIF9hbHRpdmVjPWF1dG8KQEAgLTksNyArOSw3
 IEBACiAgX3JhbmxpYj1yYW5saWIKICBfd2luZHJlcz13aW5kcmVzCiAgX2NjPWNjCi1AQCAtMTQ1
 NSw3ICsxNDU1LDYgQEAKK0BAIC0xNDY3LDcgKzE0NjcsNiBAQAogIAogICAgKikKICAgICAgZWNo
 byAiVW5rbm93biBwYXJhbWV0ZXI6ICRhY19vcHRpb24iCkBAIC0xNyw3ICsxNywxNiBAQAogICAg
 ICA7OwogIAogICAgZXNhYwotQEAgLTE1MzMsNyArMTUzMyw3IEBACitAQCAtMTUxNSw3ICsxNTE0
 LDcgQEAKKyAgIGNhc2UgIiQodW5hbWUgLW0gMj4mMSkiIGluCisgICAgICAgeDg2XzY0fGFtZDY0
 fGlbMy05XTg2KnxpODZwY3x4ODZ8eDg2cGN8azV8azZ8azZfMnxrNl8zfGs2LTJ8azYtM3xwZW50
 aXVtKnxhdGhsb24qfGk1ODZfaTY4NnxpNTg2LWk2ODYpIGhvc3RfYXJjaD1pMzg2IDs7CisgICAg
 ICAgaWE2NCkgaG9zdF9hcmNoPWlhNjQgOzsKKy0gICAgICBtYWNwcGN8cHBjKnxQb3dlciopIGhv
 c3RfYXJjaD1wcGMgOzsKKysgICAgICBtYWNwcGN8cG93ZXJwY3xwcGMqfFBvd2VyKikgaG9zdF9h
 cmNoPXBwYyA7OworICAgICAgIGFscGhhKSBob3N0X2FyY2g9YWxwaGEgOzsKKyAgICAgICBzdW40
 KnxzcGFyYyopIGhvc3RfYXJjaD1zcGFyYyA7OworICAgICAgIHBhcmlzYyp8aHBwYSp8OTAwMCop
 IGhvc3RfYXJjaD1ocHBhIDs7CitAQCAtMTU1NSw3ICsxNTU0LDcgQEAKICBmaQogIAogIGV4dHJh
 X2NmbGFncz0iLUkuIC1JZmZtcGVnICRleHRyYV9jZmxhZ3MiCkBAIC0yNiw3ICszNSw3IEBACiAg
 X3RpbWVyPXRpbWVyLWxpbnV4LmMKICBfZ2V0Y2g9Z2V0Y2gyLmMKICAKLUBAIC0zNTc2LDcgKzM1
 NzUsNyBAQAorQEAgLTM2MDEsNyArMzYwMCw3IEBACiAgaWYgbGludXggOyB0aGVuCiAgICBUSFJF
 QURfQ0ZMQUdTPS1EX1JFRU5UUkFOVAogIGVsaWYgZnJlZWJzZCB8fCBuZXRic2QgfHwgb3BlbmJz
 ZCB8fCBic2RvcyA7IHRoZW4KQEAgLTM1LDcgKzQ0LDcgQEAKICBmaQogIGlmIHRlc3QgIiRfcHRo
 cmVhZHMiID0gYXV0byA7IHRoZW4KICBjYXQgPiAkVE1QQyA8PCBFT0YKLUBAIC0zNTg2LDcgKzM1
 ODUsNyBAQAorQEAgLTM2MTEsNyArMzYxMCw3IEBACiAgRU9GCiAgX3B0aHJlYWRzPW5vCiAgaWYg
 ISBocHV4IDsgdGhlbgpAQCAtNDQsNyArNTMsNyBAQAogICAgICAjIGZvciBjcm9zc2NvbXBpbGF0
 aW9uLCB3ZSBjYW5ub3QgZXhlY3V0ZSB0aGUgcHJvZ3JhbSwgYmUgaGFwcHkgaWYgd2UgY2FuIGxp
 bmsgc3RhdGljYWxseQogICAgICBjY19jaGVjayAkVEhSRUFEX0NGTEFHUyAkbGRfdG1wICYmICh0
 bXBfcnVuIHx8IHRlc3QgIiRsZF9zdGF0aWMiKSAmJiBsZF9wdGhyZWFkPSIkbGRfdG1wIiAmJiBf
 cHRocmVhZHM9eWVzICYmIGJyZWFrCiAgICBkb25lCi1AQCAtNjI0NSw3ICs2MjQ0LDcgQEAKK0BA
 IC02MjcxLDcgKzYyNzAsNyBAQAogIGVjaG9jaGVjayAibGliZ3NtIgogIGlmIHRlc3QgIiRfbGli
 Z3NtIiA9IGF1dG8gOyB0aGVuCiAgICBfbGliZ3NtPW5vCkBAIC01Myw3ICs2Miw3IEBACiAgZmkK
 ICBpZiB0ZXN0ICIkX2xpYmdzbSIgPSB5ZXMgOyB0aGVuCiAgICBkZWZfbGliZ3NtPScjZGVmaW5l
 IENPTkZJR19MSUJHU00gMScKLUBAIC02NzA1LDYgKzY3MDQsMTYgQEAKK0BAIC02NzMzLDYgKzY3
 MzIsMTYgQEAKICAgIG5vbGlicnRtcD1ubwogICAgZGVmX2xpYnJ0bXA9JyNkZWZpbmUgQ09ORklH
 X0xJQlJUTVAgMScKICAgIGlucHV0bW9kdWxlcz0ibGlicnRtcCAkaW5wdXRtb2R1bGVzIgpAQCAt
 NzAsNyArNzksNyBAQAogIGVsc2UKICAgIG5vbGlicnRtcD15ZXMKICAgIF9saWJydG1wPW5vCi1A
 QCAtNzA5Nyw3ICs3MTA2LDcgQEAKK0BAIC03MTI1LDcgKzcxMzQsNyBAQAogIGVjaG9jaGVjayAi
 bWVuY29kZXIiCiAgaWYgdGVzdCAiJF9tZW5jb2RlciIgPSBubyA7IHRoZW4KICAgICMgbXBlZzF2
 aWRlbyBmb3IgdmZfbGF2Yywgc25vdyBmb3IgdmZfdXNwcCAvIHZmX21jZGVpbnQsCkBAIC03OSw3
 ICs4OCw3IEBACiAgZmkKICBlY2hvcmVzICIkX21lbmNvZGVyIgogIAotQEAgLTc0MzUsOCArNzQ0
 NCwxMSBAQAorQEAgLTc0NjMsOCArNzQ3MiwxMSBAQAogIGlmIHRlc3QgIiRfZ3VpIiA9IHllcyA7
 IHRoZW4KICAKICAgICMgUmVxdWlyZWQgbGlicmFyaWVzCkBAIC05Myw3ICsxMDIsNyBAQAogICAg
 ICBkaWUgIlRoZSBHVUkgcmVxdWlyZXMgbGliYXZjb2RlYyB3aXRoIFBORyBzdXBwb3J0IChuZWVk
 cyB6bGliKS4iCiAgICBmaQogICAgdGVzdCAiJF9mcmVldHlwZSIgPSBubyAmJiB0ZXN0ICIkX2Jp
 dG1hcF9mb250IiA9IG5vICYmCi1AQCAtODAwMSw3ICs4MDEzLDcgQEAKK0BAIC04MDI5LDcgKzgw
 NDEsNyBAQAogIEVYRVNVRiAgICAgID0gJF9leGVzdWYKICBFWEVTVUZTX0FMTCA9IC5leGUKICAK
 --047d7b2e770abff86404cb8f7983--

From: Michael Copeland <michael@kryptos-security.com>
To: Thomas Zander <thomas.e.zander@googlemail.com>
Cc: bug-followup@freebsd.org
Subject: Re: ports/172416: Building multimedia/mplayer on 9.1-PRERELEASE
 errors with host architecture "UNKNOWN"
Date: Mon, 8 Oct 2012 17:27:33 -0400

 --047d7bdc8fb4baa34004cb92e33f
 Content-Type: text/plain; charset=ISO-8859-1
 
 On Mon, Oct 8, 2012 at 1:23 PM, Thomas Zander <
 thomas.e.zander@googlemail.com> wrote:
 
 > Would you find attached patch to the port acceptable?
 > If so, this can be committed.
 >
 > A question or two, if I may.
 > What machine is this you are using mplayer on? There aren't that many
 > powerpc machines these days.
 > Would you test whether clang compiles mplayer on ppc? To test this,
 > just add powerpc to the if clause before MPLAYER_CLANG_SUPPORTED_ARCH
 > in Makefile.options.
 >
 > Best regards
 > Riggs
 >
 
 
 Patch is fine. I'm using it several machines. mini g4 1.2GHz radeon 9200,
 dual and quad g5 machines both with nvidia gpu, attempting to play with it
 on a ps3 and I have some ibm machines that I've been screwing around with.
 
 I'll gladly test the build with clang.
 
 Michael
 
 --047d7bdc8fb4baa34004cb92e33f--
State-Changed-From-To: feedback->open 
State-Changed-By: linimon 
State-Changed-When: Wed Oct 10 00:14:35 UTC 2012 
State-Changed-Why:  
Maintainer approved. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=172416 
State-Changed-From-To: open->closed 
State-Changed-By: glarkin 
State-Changed-When: Fri Jan 18 15:53:16 EST 2013 
State-Changed-Why:  
Committed with an additional modification to Makefile to disable asm 
in embedded ffmpeg (compiler errors in altivec .S file).  Thank you for 
your PR! 

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

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/172416: commit references a PR
Date: Fri, 18 Jan 2013 20:52:57 +0000 (UTC)

 Author: glarkin
 Date: Fri Jan 18 20:52:49 2013
 New Revision: 310619
 URL: http://svnweb.freebsd.org/changeset/ports/310619
 
 Log:
   - Fixed OS detection on freebsd-ppc [1]
   - Fixed ppc assembly language build error in embedded ffmpeg (tested on
     committer's Mac G4)
   
   PR:		ports/172416
   Submitted by:	michael copeland <michael@kryptos-security.com>
   Approved by:	Thomas Zander <thomas.e.zander@googlemail.com> (maintainer)
 
 Modified:
   head/multimedia/mplayer/Makefile
   head/multimedia/mplayer/files/patch-configure
 
 Modified: head/multimedia/mplayer/Makefile
 ==============================================================================
 --- head/multimedia/mplayer/Makefile	Fri Jan 18 18:56:04 2013	(r310618)
 +++ head/multimedia/mplayer/Makefile	Fri Jan 18 20:52:49 2013	(r310619)
 @@ -94,6 +94,11 @@ CONFIGURE_ARGS+=	--disable-directfb \
  			--disable-mpg123 \
  			--disable-musepack
  
 +# Fix a problem with unknown assembly opcodes in embedded ffmpeg
 +.if ${ARCH} == "ppc"
 +CONFIGURE_ARGS+=        --disable-asm
 +.endif
 +
  .include "${.CURDIR}/Makefile.options"
  
  .if !defined(WITHOUT_GUI) && !defined(WITHOUT_X11)
 
 Modified: head/multimedia/mplayer/files/patch-configure
 ==============================================================================
 --- head/multimedia/mplayer/files/patch-configure	Fri Jan 18 18:56:04 2013	(r310618)
 +++ head/multimedia/mplayer/files/patch-configure	Fri Jan 18 20:52:49 2013	(r310619)
 @@ -1,6 +1,6 @@
 ---- configure.orig	2012-03-15 16:20:02.000000000 +0100
 -+++ configure	2012-03-22 15:36:16.583006073 +0100
 -@@ -637,7 +637,7 @@
 +--- configure.orig	2012-07-20 15:11:42.000000000 +0200
 ++++ configure	2012-10-08 19:16:19.637797462 +0200
 +@@ -645,7 +645,7 @@
   _iwmmxt=auto
   _mtrr=auto
   _altivec=auto
 @@ -9,7 +9,7 @@
   _ranlib=ranlib
   _windres=windres
   _cc=cc
 -@@ -1455,7 +1455,6 @@
 +@@ -1467,7 +1467,6 @@
   
     *)
       echo "Unknown parameter: $ac_option"
 @@ -17,7 +17,16 @@
       ;;
   
     esac
 -@@ -1533,7 +1533,7 @@
 +@@ -1515,7 +1514,7 @@
 +   case "$(uname -m 2>&1)" in
 +       x86_64|amd64|i[3-9]86*|i86pc|x86|x86pc|k5|k6|k6_2|k6_3|k6-2|k6-3|pentium*|athlon*|i586_i686|i586-i686) host_arch=i386 ;;
 +       ia64) host_arch=ia64 ;;
 +-      macppc|ppc*|Power*) host_arch=ppc ;;
 ++      macppc|powerpc|ppc*|Power*) host_arch=ppc ;;
 +       alpha) host_arch=alpha ;;
 +       sun4*|sparc*) host_arch=sparc ;;
 +       parisc*|hppa*|9000*) host_arch=hppa ;;
 +@@ -1555,7 +1554,7 @@
   fi
   
   extra_cflags="-I. -Iffmpeg $extra_cflags"
 @@ -26,7 +35,7 @@
   _timer=timer-linux.c
   _getch=getch2.c
   
 -@@ -3576,7 +3575,7 @@
 +@@ -3601,7 +3600,7 @@
   if linux ; then
     THREAD_CFLAGS=-D_REENTRANT
   elif freebsd || netbsd || openbsd || bsdos ; then
 @@ -35,7 +44,7 @@
   fi
   if test "$_pthreads" = auto ; then
   cat > $TMPC << EOF
 -@@ -3586,7 +3585,7 @@
 +@@ -3611,7 +3610,7 @@
   EOF
   _pthreads=no
   if ! hpux ; then
 @@ -44,7 +53,7 @@
       # for crosscompilation, we cannot execute the program, be happy if we can link statically
       cc_check $THREAD_CFLAGS $ld_tmp && (tmp_run || test "$ld_static") && ld_pthread="$ld_tmp" && _pthreads=yes && break
     done
 -@@ -6245,7 +6244,7 @@
 +@@ -6271,7 +6270,7 @@
   echocheck "libgsm"
   if test "$_libgsm" = auto ; then
     _libgsm=no
 @@ -53,7 +62,7 @@
   fi
   if test "$_libgsm" = yes ; then
     def_libgsm='#define CONFIG_LIBGSM 1'
 -@@ -6705,6 +6704,16 @@
 +@@ -6733,6 +6732,16 @@
     nolibrtmp=no
     def_librtmp='#define CONFIG_LIBRTMP 1'
     inputmodules="librtmp $inputmodules"
 @@ -70,7 +79,7 @@
   else
     nolibrtmp=yes
     _librtmp=no
 -@@ -7097,7 +7106,7 @@
 +@@ -7125,7 +7134,7 @@
   echocheck "mencoder"
   if test "$_mencoder" = no ; then
     # mpeg1video for vf_lavc, snow for vf_uspp / vf_mcdeint,
 @@ -79,7 +88,7 @@
   fi
   echores "$_mencoder"
   
 -@@ -7435,8 +7444,11 @@
 +@@ -7463,8 +7472,11 @@
   if test "$_gui" = yes ; then
   
     # Required libraries
 @@ -93,7 +102,7 @@
       die "The GUI requires libavcodec with PNG support (needs zlib)."
     fi
     test "$_freetype" = no && test "$_bitmap_font" = no &&
 -@@ -8001,7 +8013,7 @@
 +@@ -8029,7 +8041,7 @@
   EXESUF      = $_exesuf
   EXESUFS_ALL = .exe
   
 _______________________________________________
 svn-ports-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-ports-all
 To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org"
 
>Unformatted:
