Re: CFT: mplayer and mencoder updates

2011-12-30 Thread Jan Beich
Zhihao Yuan lich...@gmail.com writes:

 Zhihao Yuan lich...@gmail.com writes:

 On Thu, Dec 29, 2011 at 6:07 PM, Thomas Zander
 thomas.e.zan...@googlemail.com wrote:
 Hi,

 thank you all for your comments on the first iteration of the ports.
 A heavily revised version can be found on
 http://www.rrr.de/~riggs/mplayer/m20111230.tar.bz2

 Changes:
 [...]
 - use newer gcc and binutils thus enabling significant speed-ups (and
 also, since most mplayer and ffmpeg development is done on newer gcc
 and binutils than we have in the base system, it's slowly getting
 tedious to ensure compatibility with older toolchain)

 So, with the new toolchain --disable-ssse3 is dropped but why
 BROKEN_RELOCATIONS is still there?


 GJ. Tested with clang, no problem either.

 Even though the port now explicitly uses gcc, ignoring CC from make.conf
 or Makefile.local ? And on clang with -integrated-as being default gnu as
 is not used, e.g. try to remove /usr/bin/as, it'd still build fine.

 It's not ignoring CC. It sets USE_GCC=, which can be overwrite by make 
 CC=clang.

Have you actually tried ? It's a known blind spot for clang -exp runs.

  $ echo CC=clang Makefile.local
  $ make -V CC
  gcc46
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: CFT: mplayer and mencoder updates

2011-12-30 Thread Gary Jennejohn
On Fri, 30 Dec 2011 01:07:58 +0100
Thomas Zander thomas.e.zan...@googlemail.com wrote:

 Hi,
 
 thank you all for your comments on the first iteration of the ports.
 A heavily revised version can be found on
 http://www.rrr.de/~riggs/mplayer/m20111230.tar.bz2
 
 Changes:
 - reduce the number of explicit --enable-feature where it seemed
 reasonable, hence deviate less from the intention of the configure
 script
 - remove patching and reinplacing homegrown tweaks whenever possible,
 hence deviate less from the intention of the configure script
 - fix the reported pthread issue
 - always use internal libass
 - adopt patches to support 24 bit OSS output and keep volume during seek
 - use newer gcc and binutils thus enabling significant speed-ups (and
 also, since most mplayer and ffmpeg development is done on newer gcc
 and binutils than we have in the base system, it's slowly getting
 tedious to ensure compatibility with older toolchain)
 
 Caveats:
 - bluray does not work right now - keep it/turn it off in OPTIONS for now
 
 I hope the refactoring (removing mostly) of the tweaks to the
 configure and build environment does not impose noticeable changes for
 most users, but we may encounter difficulties in some configurations I
 couldn't check.
 Please give it a try!
 
 As always, comments, suggestions and especially patches are more than welcome.
 

I see that some Makefiles under mplayer explicitly use gcc46.  Is that
really necessary?

I like mplayer but I'm not interested in installing an entire gcc46
tool chain just to get the latest version.

-- 
Gary Jennejohn
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: CFT: mplayer and mencoder updates

2011-12-30 Thread Zhihao Yuan
On Fri, Dec 30, 2011 at 3:27 AM, Jan Beich jbe...@tormail.net wrote:
 Zhihao Yuan lich...@gmail.com writes:

 Zhihao Yuan lich...@gmail.com writes:

 On Thu, Dec 29, 2011 at 6:07 PM, Thomas Zander
 thomas.e.zan...@googlemail.com wrote:
 Hi,

 thank you all for your comments on the first iteration of the ports.
 A heavily revised version can be found on
 http://www.rrr.de/~riggs/mplayer/m20111230.tar.bz2

 Changes:
 [...]
 - use newer gcc and binutils thus enabling significant speed-ups (and
 also, since most mplayer and ffmpeg development is done on newer gcc
 and binutils than we have in the base system, it's slowly getting
 tedious to ensure compatibility with older toolchain)

 So, with the new toolchain --disable-ssse3 is dropped but why
 BROKEN_RELOCATIONS is still there?


 GJ. Tested with clang, no problem either.

 Even though the port now explicitly uses gcc, ignoring CC from make.conf
 or Makefile.local ? And on clang with -integrated-as being default gnu as
 is not used, e.g. try to remove /usr/bin/as, it'd still build fine.

 It's not ignoring CC. It sets USE_GCC=, which can be overwrite by make 
 CC=clang.

 Have you actually tried ? It's a known blind spot for clang -exp runs.

  $ echo CC=clang Makefile.local
  $ make -V CC
  gcc46

To specify a var in command has different effect. You can  try it,
make CC=clang in mplayer dir.

-- 
Zhihao Yuan, nickname lichray
The best way to predict the future is to invent it.
___
4BSD -- http://4bsd.biz/
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: CFT: mplayer and mencoder updates

2011-12-30 Thread Thomas Zander
Hi all,

and another quick iteration of the port, no big changes.
Since it is still December 30 today, the URL stays the same, simply
the content has been updated:
http://www.rrr.de/~riggs/mplayer/m20111230.tar.bz2

Changes:
- Remove the patch for BROKEN_RELOCATIONS (thanks Jan for the heads-up)
- bluray actually does work with
http://www.freebsd.org/cgi/query-pr.cgi?pr=163721 (not a real change,
but noteworthy)

Regarding the dependency on gcc46. I'll make some tests whether to
keep supporting builds with the base system gcc for those who don't
wish to install gcc from ports is doable.

Best regards,
Riggs
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: CFT: mplayer and mencoder updates

2011-12-30 Thread Thomas Zander
And here we go again.

On Fri, Dec 30, 2011 at 20:44, Thomas Zander
thomas.e.zan...@googlemail.com wrote:

 Regarding the dependency on gcc46. I'll make some tests whether to
 keep supporting builds with the base system gcc for those who don't
 wish to install gcc from ports is doable.

http://www.rrr.de/~riggs/mplayer/m20111231.tar.bz2

Changes:
- Introduce OPTION variable OTCHAIN defaulting to on. Sets
USE_GCC?=4.6 and depends on recent binutils so that all optimisations
can be used. clang-friends can tweak that to their desire. If this
switch is turned off, the Makefile assumes the base system gcc should
be used and applies extra-patch-base_system_gcc to allow build (minus
a few optimisations).

Caveat:
- It's late. I have tested this with some content on amd64 and i386 on
8/stable but YMMV. Please test and report problems.

Best regards
Riggs
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: CFT: mplayer and mencoder updates

2011-12-29 Thread Thomas Zander
Hi,

thank you all for your comments on the first iteration of the ports.
A heavily revised version can be found on
http://www.rrr.de/~riggs/mplayer/m20111230.tar.bz2

Changes:
- reduce the number of explicit --enable-feature where it seemed
reasonable, hence deviate less from the intention of the configure
script
- remove patching and reinplacing homegrown tweaks whenever possible,
hence deviate less from the intention of the configure script
- fix the reported pthread issue
- always use internal libass
- adopt patches to support 24 bit OSS output and keep volume during seek
- use newer gcc and binutils thus enabling significant speed-ups (and
also, since most mplayer and ffmpeg development is done on newer gcc
and binutils than we have in the base system, it's slowly getting
tedious to ensure compatibility with older toolchain)

Caveats:
- bluray does not work right now - keep it/turn it off in OPTIONS for now

I hope the refactoring (removing mostly) of the tweaks to the
configure and build environment does not impose noticeable changes for
most users, but we may encounter difficulties in some configurations I
couldn't check.
Please give it a try!

As always, comments, suggestions and especially patches are more than welcome.

Best regards
Riggs
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: CFT: mplayer and mencoder updates

2011-12-29 Thread Jan Beich
Zhihao Yuan lich...@gmail.com writes:

 On Thu, Dec 29, 2011 at 6:07 PM, Thomas Zander
 thomas.e.zan...@googlemail.com wrote:
 Hi,

 thank you all for your comments on the first iteration of the ports.
 A heavily revised version can be found on
 http://www.rrr.de/~riggs/mplayer/m20111230.tar.bz2

 Changes:
[...]
 - use newer gcc and binutils thus enabling significant speed-ups (and
 also, since most mplayer and ffmpeg development is done on newer gcc
 and binutils than we have in the base system, it's slowly getting
 tedious to ensure compatibility with older toolchain)

So, with the new toolchain --disable-ssse3 is dropped but why
BROKEN_RELOCATIONS is still there?


 GJ. Tested with clang, no problem either.

Even though the port now explicitly uses gcc, ignoring CC from make.conf
or Makefile.local ? And on clang with -integrated-as being default gnu as
is not used, e.g. try to remove /usr/bin/as, it'd still build fine.
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: CFT: mplayer and mencoder updates

2011-12-29 Thread Zhihao Yuan
On Thu, Dec 29, 2011 at 8:01 PM, Jan Beich jbe...@tormail.net wrote:
 Zhihao Yuan lich...@gmail.com writes:

 On Thu, Dec 29, 2011 at 6:07 PM, Thomas Zander
 thomas.e.zan...@googlemail.com wrote:
 Hi,

 thank you all for your comments on the first iteration of the ports.
 A heavily revised version can be found on
 http://www.rrr.de/~riggs/mplayer/m20111230.tar.bz2

 Changes:
 [...]
 - use newer gcc and binutils thus enabling significant speed-ups (and
 also, since most mplayer and ffmpeg development is done on newer gcc
 and binutils than we have in the base system, it's slowly getting
 tedious to ensure compatibility with older toolchain)

 So, with the new toolchain --disable-ssse3 is dropped but why
 BROKEN_RELOCATIONS is still there?


 GJ. Tested with clang, no problem either.

 Even though the port now explicitly uses gcc, ignoring CC from make.conf
 or Makefile.local ? And on clang with -integrated-as being default gnu as
 is not used, e.g. try to remove /usr/bin/as, it'd still build fine.

It's not ignoring CC. It sets USE_GCC=, which can be overwrite by make CC=clang.

-- 
Zhihao Yuan, nickname lichray
The best way to predict the future is to invent it.
___
4BSD -- http://4bsd.biz/
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: CFT: mplayer and mencoder updates

2011-12-21 Thread Yamagi Burmeister
Hi :)

1. The BLURAY is option is broken:

cc -MD -MP -Wundef -Wall -Wno-switch -Wno-parentheses -Wpointer-arith
-Wredundant-decls -Wstrict-prototypes -Wmissing-prototypes
-Wdisabled-optimization -Wno-pointer-sign -Wdeclaration-after-statement
-std=gnu99 -Werror-implicit-function-declaration -O4 -march=x86-64
-mtune=generic -pipe -ffast-math -fomit-frame-pointer
-fno-tree-vectorize -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64
-D_LARGEFILE64_SOURCE -Ilibdvdread4 -I. -Iffmpeg  -I/usr/local/include
-I/usr/local/include  -I/usr/local/include/freetype2
-I/usr/local/include -I/usr/local/include   -I/usr/local/include
-I/usr/local/include/schroedinger-1.0 -I/usr/local/include/orc-0.4
-D_THREAD_SAFE -D_REENTRANT -I/usr/local/include/gtk-2.0
-I/usr/local/lib/gtk-2.0/include -I/usr/local/include/atk-1.0
-I/usr/local/include/cairo -I/usr/local/include/gdk-pixbuf-2.0
-I/usr/local/include/pango-1.0 -I/usr/local/include/gio-unix-2.0/
-I/usr/local/include -I/usr/local/include/glib-2.0
-I/usr/local/include/pixman-1 -I/usr/local/include/freetype2
-I/usr/local/include/drm   -c -o stream/stream_bluray.o

stream/stream_bluray.c: In function 'bluray_stream_control':
stream/stream_bluray.c:119: error: too many arguments to function
'bd_get_title_info' 

stream/stream_bluray.c:140: error: too many
arguments to function 'bd_get_title_info' 

stream/stream_bluray.c:159:
error: too many arguments to function 'bd_get_title_info'

stream/stream_bluray.c:178: error: too many arguments to function
'bd_get_title_info' 

stream/stream_bluray.c: In function
'bluray_stream_open': 

stream/stream_bluray.c:239: error: too many
arguments to function 'bd_get_titles' 

stream/stream_bluray.c:253: error: too many arguments to function
'bd_get_title_info' 

stream/stream_bluray.c:287: error: too many
arguments to function 'bd_get_title_info stream/stream_bluray.c 



2. ASS subtitles are distorted and missaligned when playing in
fullscreen on multi monitor setups. This is not the case with 
the plain, unpatched upstream source build from svn. A screenshot
says more than 1000 words:

http://deponie.yamagi.org/freebsd/debug/mplayer/mplayer_ass.png

I'll try to track this down to a patch or option, but it'll have to
wait until early january. 



The mplayer internal sound volume (that one that's adjusted with the /
and * keys) is reseted to the default value when the stream is
reinitialized. This happends at least at:
- Fast forwarding or backwarding
- Resuming after pause
- DVD chapter change
Especially the last one is very annoying. Patching this is easy by
saving the current volume level before closing the OSS device and 
set it again after reopening it. An updated patch-libao2-ao_oss.c is
attached, in case that it's eaten my the mailing list a copy can be
found here:
http://deponie.yamagi.org/freebsd/patches/patch-libao2-ao_oss.c

Thanks,
Yamagi :)

On Mon, 19 Dec 2011 23:35:42 +0100 Thomas Zander
thomas.e.zan...@googlemail.com wrote:

 on http://www.rrr.de/~riggs/mplayer/m20111219.tar.bz2 you can find the
 draft of updates to the mplayer and mencoder ports.
 
 Please have a look, play with it and let me know what you think.
 I tested it on 8.2 STABLE i386 and amd64 only so far, so I hope it
 works for 9 and 10 as well.
 
 As always, comments, suggestions and especially patches are more than welcome.
 
 Best regards
 Riggs
 ___
 freebsd-multime...@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/freebsd-multimedia
 To unsubscribe, send any mail to freebsd-multimedia-unsubscr...@freebsd.org
 


-- 
Homepage:  www.yamagi.org
XMPP:  yam...@yamagi.org
GnuPG/GPG: 0xEFBCCBCB


pgpMZe2OSQMDB.pgp
Description: PGP signature


Re: CFT: mplayer and mencoder updates

2011-12-21 Thread Tom Evans
On Mon, Dec 19, 2011 at 10:35 PM, Thomas Zander
thomas.e.zan...@googlemail.com wrote:
 Hi list,

 on http://www.rrr.de/~riggs/mplayer/m20111219.tar.bz2 you can find the
 draft of updates to the mplayer and mencoder ports.

 Please have a look, play with it and let me know what you think.
 I tested it on 8.2 STABLE i386 and amd64 only so far, so I hope it
 works for 9 and 10 as well.

 As always, comments, suggestions and especially patches are more than welcome.

 Best regards
 Riggs

Hi Riggs

I notice in Makefile.shared there is no longer a dependency on
binutils, and there is a patch to define BROKEN_RELOCATIONS.

If you do this, CABAC H264 decoding is ~40% slower on amd64 boxes, and
you lose other speed benefits (SSE2+).

In order to build the best performing, bug free, tested version of
mplayer, you should build with lang/gcc46 and devel/binutils. Both my
ffmpeg and mplayer builds come straight from git master, and work
without patches - in fact, the only thing I must do to build them is
pass configure:
  --with-cc=/usr/local/bin/gcc46 \
  --with-as=/usr/local/bin/as \
  --extra-libs=-L/usr/local/lib \
  --extra-cflags=-I/usr/local/include \
  --prefix=/usr/local --mandir=/usr/local/man

In a similar vein, mplayer's configure does not check for existence of
a feature if you explicitly bake it in. For instance, the libbluray in
freebsd ports is not up to date with the version required for mplayer,
but selecting bluray from OPTIONS forces mplayer to build it in.
If it was not forced, mplayer's configure would detect the library is
there, but of an older version, and it would not attempt to compile in
bluray support, and you would not get build errors.

I'm going to go through the patches in files directory and see what
can be pushed back. I think most of them fix issues with libraries I
don't use (gsm, udp, arts) or with features that I don't use
(libao-oss - well, I use SPDIF).


Cheers

Tom
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: CFT: mplayer and mencoder updates

2011-12-20 Thread Thomas Zander
On Tue, Dec 20, 2011 at 00:43, Zhihao Yuan lich...@gmail.com wrote:
 stable/8:

 gcc:

 ffmpeg/libavformat/libavformat.a(udp.o): In function `udp_open':
 udp.c:(.text+0xf8e): undefined reference to `pthread_create'
 ffmpeg/libavcodec/libavcodec.a(pthread.o): In function `ff_thread_init':
 pthread.c:(.text+0x178c): undefined reference to `pthread_create'
 pthread.c:(.text+0x1894): undefined reference to `pthread_create'
 collect2: ld returned 1 exit status

How did you trigger this build problem? What OPTIONS did you use?
I'll look into your patch set as soon as I can!
Until then, thanks for testing and patching!

Best regards
Riggs
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: CFT: mplayer and mencoder updates

2011-12-20 Thread Zhihao Yuan
On Tue, Dec 20, 2011 at 3:02 PM, Thomas Zander
thomas.e.zan...@googlemail.com wrote:
 On Tue, Dec 20, 2011 at 00:43, Zhihao Yuan lich...@gmail.com wrote:
 stable/8:

 gcc:

 ffmpeg/libavformat/libavformat.a(udp.o): In function `udp_open':
 udp.c:(.text+0xf8e): undefined reference to `pthread_create'
 ffmpeg/libavcodec/libavcodec.a(pthread.o): In function `ff_thread_init':
 pthread.c:(.text+0x178c): undefined reference to `pthread_create'
 pthread.c:(.text+0x1894): undefined reference to `pthread_create'
 collect2: ld returned 1 exit status

 How did you trigger this build problem? What OPTIONS did you use?

OCFLAGS, X11, X11XV, VDPAU, MAD, LIBCDIO.

The following is the options recorded by config.mak:

CONFIGURATION = --cc=clang --host-cc=clang --disable-bitmap-font --disab
le-liba52 --disable-ssse3 --disable-directfb --disable-faac --disable-faad --dis
able-libdca --disable-twolame --disable-x264 --disable-xvid --disable-libdirac-l
avc --disable-mencoder --disable-mpg123 --disable-alsa --disable-musepack --disa
ble-inet6 --disable-gif --disable-libopenjpeg --disable-libdv --disable-theora -
-disable-libvpx-lavc --disable-libschroedinger-lavc --disable-speex --disable-li
ve --disable-jack --disable-nas --disable-openal --disable-pulse --disable-blura
y --disable-smb --disable-fribidi --disable-cdparanoia --disable-ladspa --disabl
e-liblzo --disable-win32dll --disable-qtx --disable-real --disable-libopencore_a
mrnb --disable-libopencore_amrwb --disable-libgsm --disable-tv-v4l1 --disable-tv
-v4l2 --disable-librtmp --disable-mng --enable-ass --enable-ass-internal --disab
le-enca --disable-dga1 --disable-dga2 --disable-gl --disable-xinerama --disable-
vm --disable-rtc --disable-arts --disable-caca --disable-lirc --disable-sdl --di
sable-svga --disable-aa --disable-esd --disable-joystick --x-libraries=/usr/loca
l/lib --x-includes=/usr/local/include --prefix=/usr/local --mandir=/usr/local/ma
n --build=amd64-portbld-freebsd8.2

And, there is no -pthread in this file.

 I'll look into your patch set as soon as I can!
 Until then, thanks for testing and patching!

 Best regards
 Riggs



-- 
Zhihao Yuan, nickname lichray
The best way to predict the future is to invent it.
___
4BSD -- http://4bsd.biz/
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: CFT: mplayer and mencoder updates

2011-12-19 Thread Zhihao Yuan
Hi:

Could you take a look at this (support FreeBSD OSS 24-bit output)?

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

I unified the some patches here:

https://gist.github.com/1362097

1. Disable ALSA explicitly;
2. Allow internal libass if ASS is tuned off;
3. OSS 24-bit output;
4. FreeBSD's log2  log2f in libc.

Let me test the port...

stable/8:

gcc:

ffmpeg/libavformat/libavformat.a(udp.o): In function `udp_open':
udp.c:(.text+0xf8e): undefined reference to `pthread_create'
ffmpeg/libavcodec/libavcodec.a(pthread.o): In function `ff_thread_init':
pthread.c:(.text+0x178c): undefined reference to `pthread_create'
pthread.c:(.text+0x1894): undefined reference to `pthread_create'
collect2: ld returned 1 exit status

clang:

clang: warning: argument unused during compilation:
'-rpath=/usr/lib:/usr/local/lib'
ffmpeg/libavformat/libavformat.a(udp.o): In function `udp_open':
libavformat/udp.c:(.text+0x907): undefined reference to `pthread_create'
ffmpeg/libavcodec/libavcodec.a(pthread.o): In function `ff_thread_init':
libavcodec/pthread.c:(.text+0xded): undefined reference to `pthread_create'
libavcodec/pthread.c:(.text+0x104e): undefined reference to `pthread_create'
clang: error: linker command failed with exit code 1 (use -v to see invocation)

The original config.mak has no -pthread, so the replacement in
port-configure fails to work.

The program works fine. Thanks.

On Mon, Dec 19, 2011 at 4:35 PM, Thomas Zander
thomas.e.zan...@googlemail.com wrote:
 Hi list,

 on http://www.rrr.de/~riggs/mplayer/m20111219.tar.bz2 you can find the
 draft of updates to the mplayer and mencoder ports.

 Please have a look, play with it and let me know what you think.
 I tested it on 8.2 STABLE i386 and amd64 only so far, so I hope it
 works for 9 and 10 as well.

 As always, comments, suggestions and especially patches are more than welcome.

 Best regards
 Riggs
 ___
 freebsd-multime...@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/freebsd-multimedia
 To unsubscribe, send any mail to freebsd-multimedia-unsubscr...@freebsd.org



-- 
Zhihao Yuan, nickname lichray
The best way to predict the future is to invent it.
___
4BSD -- http://4bsd.biz/
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org