[CFT] mplayer / mencoder port update

2010-11-07 Thread Thomas Zander
Hi,

I have prepared a recent mplayer snapshot for a possible update for
mplayer and mencoder ports.
You can find the tarball here:

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

Since I had to tweak it on several occasions to get it to build and I
have tested it only on amd64 so far, please test whether it builds and
works as intended for you. Of course, I appreciate any form of
constructive feedback.

Thanks in advance,
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 / mencoder port update

2010-11-07 Thread Anonymous
Thomas Zander  writes:

> Hi,
>
> I have prepared a recent mplayer snapshot for a possible update for
> mplayer and mencoder ports.
> You can find the tarball here:
>
> http://www.rrr.de/~riggs/mplayer/m20101106.tar.bz2
>
> Since I had to tweak it on several occasions to get it to build and I
> have tested it only on amd64 so far, please test whether it builds and
> works as intended for you. Of course, I appreciate any form of
> constructive feedback.

Can you include ariff's patch for VPC (Volume Per Channel)?

  http://people.freebsd.org/~ariff/ports/multimedia_mplayer/patch-xzz
  http://forums.freebsd.org/showpost.php?p=29564&postcount=31

I've been using it for about a year and it *works* unlike
the one in uoti's repo.

  http://repo.or.cz/w/mplayer.git/commitdiff/91d0d3a

Also, did you miss -ac mpg123 with previous update intentionally?
It decodes faster than -ac mp3lib on my 2yo amd64 box.
___
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 / mencoder port update

2010-11-07 Thread Stefan Ehmann
On Sunday 07 November 2010 12:50:48 Thomas Zander wrote:
> Hi,
> 
> I have prepared a recent mplayer snapshot for a possible update for
> mplayer and mencoder ports.
> You can find the tarball here:
> 
> http://www.rrr.de/~riggs/mplayer/m20101106.tar.bz2
> 
> Since I had to tweak it on several occasions to get it to build and I
> have tested it only on amd64 so far, please test whether it builds and
> works as intended for you. Of course, I appreciate any form of
> constructive feedback.

Built fine here. Unfortunately, x264 support in mencoder is broken.

Apparently, mencoder wants X264_BUILD >= 99, but multimedia/x264 is at build 
98.
___
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 / mencoder port update

2010-11-08 Thread Thomas Zander
On Sun, Nov 7, 2010 at 19:27, Stefan Ehmann  wrote:

> Built fine here. Unfortunately, x264 support in mencoder is broken.
>
> Apparently, mencoder wants X264_BUILD >= 99, but multimedia/x264 is at build
> 98.

True, x264-devel is already at 104, so in the near future this problem
will be eliminated.
Nevertheless, I'll see that it gets fixed in the next iteration.
Merci for the heads up
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 / mencoder port update

2010-11-08 Thread Thomas Zander
On Sun, Nov 7, 2010 at 14:17, Anonymous  wrote:

> Can you include ariff's patch for VPC (Volume Per Channel)?

I haven't used it myself yet, but it seems quite useful. I'll check it
out, thanks!

> Also, did you miss -ac mpg123 with previous update intentionally?

You mean the laste update to hit the ports tree in July/August? No.
Does it work now as you expect?

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 / mencoder port update

2010-11-08 Thread Dmitry Marakasov
* Thomas Zander (thomas.e.zan...@googlemail.com) wrote:

> I have prepared a recent mplayer snapshot for a possible update for
> mplayer and mencoder ports.
> You can find the tarball here:
> 
> http://www.rrr.de/~riggs/mplayer/m20101106.tar.bz2
> 
> Since I had to tweak it on several occasions to get it to build and I
> have tested it only on amd64 so far, please test whether it builds and
> works as intended for you. Of course, I appreciate any form of
> constructive feedback.

It requires yasm in BUILD_DEPENDS:

http://people.freebsd.org/~amdmi3/mplayer-sdl-1.0.r20101106.log

I'll also run it in a tinderbox with various options flipped to
nondefault values and report.

-- 
Dmitry Marakasov   .   55B5 0596 FF1E 8D84 5F56  9510 D35A 80DD F9D2 F77D
amd...@amdmi3.ru  ..:  jabber: amd...@jabber.ruhttp://www.amdmi3.ru
___
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 / mencoder port update

2010-11-08 Thread Anonymous
Thomas Zander  writes:

> On Sun, Nov 7, 2010 at 14:17, Anonymous  wrote:
>
>> Can you include ariff's patch for VPC (Volume Per Channel)?
>
> I haven't used it myself yet, but it seems quite useful. I'll check it
> out, thanks!
>
>> Also, did you miss -ac mpg123 with previous update intentionally?
>
> You mean the laste update to hit the ports tree in July/August?

-ac mpg123 was added in June 30, r31590

> No. Does it work now as you expect?

Now? It works with 2010-07-17 snapshot, too. I don't use the port[*].

Anyway, it shouldn't automatically link against libmpg123.so when
audio/mpg123 is installed without adjusting LIB_DEPENDS.

[*] it lacks ffmpeg-mt, -ordered-chapters and some vdpau features,
e.g. output_surfaces
___
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 / mencoder port update

2010-11-09 Thread Dmitry Marakasov
* Dmitry Marakasov (amd...@hades.panopticon) wrote:

> I'll also run it in a tinderbox with various options flipped to
> nondefault values and report.

Done, all good.

-- 
Dmitry Marakasov   .   55B5 0596 FF1E 8D84 5F56  9510 D35A 80DD F9D2 F77D
amd...@amdmi3.ru  ..:  jabber: amd...@jabber.ruhttp://www.amdmi3.ru
___
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 / mencoder port update

2010-11-10 Thread Christian Weisgerber
Thomas Zander  wrote:

> I have prepared a recent mplayer snapshot for a possible update for
> mplayer and mencoder ports.
> You can find the tarball here:
> 
> http://www.rrr.de/~riggs/mplayer/m20101106.tar.bz2

The default lavf demuxer is still broken for Ogg streams.

Try this:
$ mplayer http://icecast.version6.net:/okul.ogg

It will usually work for the first song, but at the transition
between songs it will stop and just continue to complain
"Cache not filling!".

A workaround is to use -demuxer ogg.
(This has the side effect of causing the ffvorbis decoder to choke
on a partial packet and a fall-through to the next Vorbis decoder
in line, tremor in our case.  Sigh.)

I don't expect the port to deal with this, I'm mentioning it in the
hope of generating some backpressure on upstream to finally fix
this.

-- 
Christian "naddy" Weisgerber  na...@mips.inka.de

___
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 / mencoder port update

2010-11-11 Thread Thomas Zander
Hi,

I have prepared a new tarball dealing with the issues reported so far.
You can find it here:

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

Most notable changes:
- includes Ariff's volume-per-channel patch
- uses x264-devel for mencoder
- does not link to external mpg123

Please continue testing using this tarball. I'd like for the new port
to hit the ports tree soon!

@Martin: Do you have plans to update x264 to the level of -devel, or
should I keep mencoder using -devel?

Have fun
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 / mencoder port update

2010-11-12 Thread Gary Jennejohn
On Thu, 11 Nov 2010 23:13:14 +0100
Thomas Zander  wrote:

> I have prepared a new tarball dealing with the issues reported so far.
> You can find it here:
> 
> http://www.rrr.de/~riggs/mplayer/m2010.tar.bz2
> 
> Most notable changes:
> - includes Ariff's volume-per-channel patch
> - uses x264-devel for mencoder
> - does not link to external mpg123
> 
> Please continue testing using this tarball. I'd like for the new port
> to hit the ports tree soon!
> 
> @Martin: Do you have plans to update x264 to the level of -devel, or
> should I keep mencoder using -devel?
> 

This has, in my opinion, a major problem - it requires libmpcdec-1.2.6
which conflicts with musepack-2009.03.01_1 which is required by vlc.

That means that, with this new version. a user can't have both mplayer
and vlc installed together :(  IMHO not acceptable.

-- 
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 / mencoder port update

2010-11-12 Thread Gary Jennejohn
On Fri, 12 Nov 2010 11:24:01 +0100
Gary Jennejohn  wrote:

> On Thu, 11 Nov 2010 23:13:14 +0100
> Thomas Zander  wrote:
> 
> > I have prepared a new tarball dealing with the issues reported so far.
> > You can find it here:
> > 
> > http://www.rrr.de/~riggs/mplayer/m2010.tar.bz2
> > 
> > Most notable changes:
> > - includes Ariff's volume-per-channel patch
> > - uses x264-devel for mencoder
> > - does not link to external mpg123
> > 
> > Please continue testing using this tarball. I'd like for the new port
> > to hit the ports tree soon!
> > 
> > @Martin: Do you have plans to update x264 to the level of -devel, or
> > should I keep mencoder using -devel?
> > 
> 
> This has, in my opinion, a major problem - it requires libmpcdec-1.2.6
> which conflicts with musepack-2009.03.01_1 which is required by vlc.
> 
> That means that, with this new version. a user can't have both mplayer
> and vlc installed together :(  IMHO not acceptable.
> 

Woops.  Let me modify this to note that musepack can be turned off in vlc.

It could be that I changed some options when I installed the new mplayer,
so my complaint may be invalid, but I'm not sure.  All I can say for sure
is that I did have both vlc and mplayer installed, and vlc was using musepack.

-- 
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 / mencoder port update

2010-11-12 Thread Stefan Ehmann
On Friday 12 November 2010 11:24:01 Gary Jennejohn wrote:
> On Thu, 11 Nov 2010 23:13:14 +0100
> 
> Thomas Zander  wrote:
> > I have prepared a new tarball dealing with the issues reported so far.
> > You can find it here:
> > 
> > http://www.rrr.de/~riggs/mplayer/m2010.tar.bz2
> > 
> > Most notable changes:
> > - includes Ariff's volume-per-channel patch
> > - uses x264-devel for mencoder
> > - does not link to external mpg123
> > 
> > Please continue testing using this tarball. I'd like for the new port
> > to hit the ports tree soon!
> > 
> > @Martin: Do you have plans to update x264 to the level of -devel, or
> > should I keep mencoder using -devel?

x264 works again in mencoder, but x264-devel conflicts with x264 (which is 
also required by vlc and other ports). Upgrade will be much smoother if 
multimedia/x264 is upgraded.

> This has, in my opinion, a major problem - it requires libmpcdec-1.2.6
> which conflicts with musepack-2009.03.01_1 which is required by vlc.

mplayer no longer depends on libmpcdec (AFAIK it uses the builtin decoder). 
libmpcdec is deprecated anyway
___
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 / mencoder port update

2010-11-12 Thread Christian Weisgerber
Thomas Zander  wrote:

> I have prepared a new tarball dealing with the issues reported so far.
> You can find it here:
> 
> http://www.rrr.de/~riggs/mplayer/m2010.tar.bz2

Not good.  When I play a video, sound file, or stream where the
audio is decoded by mplayer (MP3, AAC, Vorbis) and pause playback,
I get audio noise during the pause.

7.3-STABLE/amd64.
AC3 pass-through over S/PDIF is not affected.

> - includes Ariff's volume-per-channel patch

This is suspicious...  Yes, reverting files/patch-volume-per-channel
fixes it.

-- 
Christian "naddy" Weisgerber  na...@mips.inka.de

___
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 / mencoder port update

2010-11-12 Thread Anonymous
na...@mips.inka.de (Christian Weisgerber) writes:

> Thomas Zander  wrote:
>
>> I have prepared a new tarball dealing with the issues reported so far.
>> You can find it here:
>> 
>> http://www.rrr.de/~riggs/mplayer/m2010.tar.bz2
>
> Not good.  When I play a video, sound file, or stream where the
> audio is decoded by mplayer (MP3, AAC, Vorbis) and pause playback,
> I get audio noise during the pause.
>
> 7.3-STABLE/amd64.
> AC3 pass-through over S/PDIF is not affected.
>
>> - includes Ariff's volume-per-channel patch
>
> This is suspicious...  Yes, reverting files/patch-volume-per-channel
> fixes it.

Can you try to replace

  #ifdef __FreeBSD__

with

  #ifdef __FreeBSD__ && __FreeBSD_version >= 800097

Alternatively, just hide the patch under ${OSVERSION} >= 800097.
___
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 / mencoder port update

2010-11-12 Thread Michal Varga
On Sat, 2010-11-13 at 03:22 +0300, Anonymous wrote:
> Can you try to replace
> 
>   #ifdef __FreeBSD__
> 
> with
> 
>   #ifdef __FreeBSD__ && __FreeBSD_version >= 800097
> 
> Alternatively, just hide the patch under ${OSVERSION} >= 800097.

Just my 2 cents:

Wouldn't be easier to simply made the patch optional via OPTIONS
(defaulting to off) and forget about it?

For example, I too have no use for it, as I don't prefer applications
messing with my mixer and thus run mplayer with -sotfvol. And seriously,
even then, for the past 10 years of pretty heavy mplayer usage - I
changed volume in it for like.. three times, ever (probably by
accident).

So for me, this is a patch that's not part of $Upstream, does absolutely
nothing I need, does even less for 4Front OSS users, and is already
known to break stuff for someone (the last point being somewhat
significant).

Please, don't include it by default for everyone.

m.

-- 
Michal Varga,
Stonehenge (Gmail account)


___
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 / mencoder port update

2010-11-12 Thread Anonymous
Michal Varga  writes:

> On Sat, 2010-11-13 at 03:22 +0300, Anonymous wrote:
>> Can you try to replace
>> 
>>   #ifdef __FreeBSD__
>> 
>> with
>> 
>>   #ifdef __FreeBSD__ && __FreeBSD_version >= 800097
>> 
>> Alternatively, just hide the patch under ${OSVERSION} >= 800097.
>
> Just my 2 cents:
>
> Wouldn't be easier to simply made the patch optional via OPTIONS
> (defaulting to off) and forget about it?
>
> For example, I too have no use for it, as I don't prefer applications
> messing with my mixer and thus run mplayer with -sotfvol. And seriously,
> even then, for the past 10 years of pretty heavy mplayer usage - I
> changed volume in it for like.. three times, ever (probably by
> accident).

VPC is smth like an application-specific mixer. And you can even change
volume for apps that don't normally have any volume control, e.g. audio/xmp.

  http://people.freebsd.org/~ariff/utils/appsmixer

OTOH, -softvol is mplayer-specific.

>
> So for me, this is a patch that's not part of $Upstream, does absolutely
> nothing I need, does even less for 4Front OSS users, and is already
> known to break stuff for someone (the last point being somewhat
> significant).

It was an attempt to make volume control more user-friendly on 8.x packages.
For example

  1. user opens a file and finds out volume is too high
  3. so, he reduces it using volume control buttons (`/' and `*')
  4. after a bit of time he pauses playback (e.g. to get more snacks)
  5. then resumes playback only to find out volume is back to high

And the above doesn't happen on 6.x or 7.x. But on 8.x one has to dig
mplayer manpage for -softvol even if he previously didn't use it.

>
> Please, don't include it by default for everyone.

If it's not gonna be included by default there should be a note in
PKGMESSAGE about how VPC on 8.x affects volume control in mplayer
and a suggestion to use -softvol.
___
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 / mencoder port update

2010-11-12 Thread Anonymous
Anonymous  writes:

>>
>> Please, don't include it by default for everyone.
>
> If it's not gonna be included by default there should be a note in
> PKGMESSAGE about how VPC on 8.x affects volume control in mplayer
> and a suggestion to use -softvol.

Oops, hw.snd.vpc_autoreset=0 is probably better.
___
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 / mencoder port update

2010-11-14 Thread Anonymous
na...@mips.inka.de (Christian Weisgerber) writes:

> Thomas Zander  wrote:
>
>> I have prepared a recent mplayer snapshot for a possible update for
>> mplayer and mencoder ports.
>> You can find the tarball here:
>> 
>> http://www.rrr.de/~riggs/mplayer/m20101106.tar.bz2
>
> The default lavf demuxer is still broken for Ogg streams.
>
> Try this:
> $ mplayer http://icecast.version6.net:/okul.ogg
>
> It will usually work for the first song, but at the transition
> between songs it will stop and just continue to complain
> "Cache not filling!".

It can play a bit longer with internet_radio_hack

  http://repo.or.cz/w/mplayer.git/commitdiff/393bb7b

but eventually it runs out of lavf streams.

>
> A workaround is to use -demuxer ogg.
> (This has the side effect of causing the ffvorbis decoder to choke
> on a partial packet and a fall-through to the next Vorbis decoder
> in line, tremor in our case.  Sigh.)
>
> I don't expect the port to deal with this, I'm mentioning it in the
> hope of generating some backpressure on upstream to finally fix
> this.
___
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 / mencoder port update

2010-11-17 Thread Thomas Zander
Hi,

I have prepared a new tarball, for download at the usual location:

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

Most notable changes:
- remove volume-per-channel patch, instead pkg-message explains the
issue and the solutions
- remove RUN_DEPENDS for live media

@Martin: Do you have plans for an x264 update? I'd like to commit
mplayer after we have a suitable x264 port.

Have fun
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 / mencoder port update

2010-11-30 Thread Martin Matuska
I have just updated x264 to 0.110.1804
We use now original x264 versioning scheme for PORTVERSION.

Dňa 17.11.2010 21:44, Thomas Zander wrote / napísal(a):
> Hi,
>
> I have prepared a new tarball, for download at the usual location:
>
> http://www.rrr.de/~riggs/mplayer/m20101117.tar.bz2
>
> Most notable changes:
> - remove volume-per-channel patch, instead pkg-message explains the
> issue and the solutions
> - remove RUN_DEPENDS for live media
>
> @Martin: Do you have plans for an x264 update? I'd like to commit
> mplayer after we have a suitable x264 port.
>
> Have fun
> 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 / mencoder port update

2010-11-30 Thread Thomas Zander
On Tue, Nov 30, 2010 at 11:59, Martin Matuska  wrote:
> I have just updated x264 to 0.110.1804

Great!

Merci,
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"