Linux version of PMS with statically linked ffmpeg/mencoder

[slooff]

thanks for all the work in this script; truly valuable for me! I tried this version but initially it didn't detect maven in the correct way for me (opensuse 11.4), then it failed in the final steps:

Code: Select all

CONFLICT (delete/modify): tests/fate2.mak deleted in 48706f41e1a17e04b0aa09852eff021833708233 and modified in HEAD. Version HEAD of tests/fate2.mak left in tree.
Auto-merging tests/ref/lavfi/pixdesc
Auto-merging tests/ref/lavfi/pixfmts_copy
Auto-merging tests/ref/lavfi/pixfmts_null
Auto-merging tests/ref/lavfi/pixfmts_scale
Auto-merging tests/ref/lavfi/pixfmts_vflip
Auto-merging tests/ref/seek/lavf_mxf
CONFLICT (content): Merge conflict in tests/ref/seek/lavf_mxf
Automatic merge failed; fix conflicts and then commit the result.
Fatal error occurred, aborting build.


As I am only interested in mencoder/mplayer, it works for me, but doesn't build all the stuff though.

[happy.neko]

That's because I'm still not using new fancy git/maven combo. So it remains untested. You can compile PS3MS old-fashioned way with ant+subversion with build_ps3mediaserver from previews scripts. Or just grub compiled binaries from target/bin like you did.

Raptor399, please consider updating VERSION_MPLAYER to 34582 because of the SSA style override bug. More info: https://code.google.com/p/ps3mediaserve ... id=1235#c6

[Raptor399]

Raptor399, please consider updating VERSION_MPLAYER to 34582 because of the SSA style override bug. More info: https://code.google.com/p/ps3mediaserve ... id=1235#c6

Okay, I'll have a look at it tomorrow.
First some sleep.

[happy.neko]

Ok.

And now I see that slooff's problem is probably related to ffmpeg. I don't know why I blamed poor maven slooff, are you trying to compile with existing ffmpeg source folder? You should delete at least mplayer and ffmpeg folders before running build-pms-binaries.sh.

[Raptor399]

I have tested with VERSION_MPLAYER 34582.
It builds correct, but a test movie clip with ASS subtitles shows the subs in different fonts now.
Looks like some fixed-width font like Courier is used instead of the font that was used before. Reverting back to the SB21 build of mencoder fixed the problem.

These are the contents of the .ass file:

Code: Select all

[Script Info]

;-----------------------------------;
; Advanced Sub Station Alpha script ;
;-----------------------------------;
;                                   ;
; This script has been created with ;
;    MEDUSA - Subtitling Station    ;
;-----------------------------------;

Title:
Original Script:
Original Translation:
Original Editing:
Original Timing:
Synch Point:
Script Updated By:
Update Details:
ScriptType: v4.00+
Collisions: Normal
PlayResX: 800
PlayResY: 600
PlayDepth: 0
Timer: 100.0000
WrapStyle: 0

[V4+ Styles]
Format: Name, Fontname, Fontsize, PrimaryColour, SecondaryColour, OutlineColour, BackColour, Bold, Italic, Underline, StrikeOut, ScaleX, ScaleY, Spacing, Angle, BorderStyle, Outline, Shadow, Alignment, MarginL, MarginR, MarginV, Encoding
Style: Daikuuji (KGNE Omake),Albert,36,&H0059E3F7,&H0059E3F7,&H0003373F,&HA003373F,0,0,0,0,100,100,0,0,1,2.9,3,2,8,8,8,0
Style: Mayu (KGNE Omake),Albert,36,&H00AC9F95,&H00AC9F95,&H001B1714,&HA01B1714,0,0,0,0,100,100,0,0,1,2.9,3,2,8,8,8,0
Style: Takayuki (KGNE),ResPublica,36,&H00C9F0F8,&H00C9F0F8,&H0013161A,&HA013161A,-1,0,0,0,100,100,0,0,1,2.9,3,2,8,8,8,0
Style: Hayase (KGNE Omake),Albert,36,&H00FCB6C8,&H00FCB6C8,&H002D020D,&HA02D020D,0,0,0,0,100,100,0,0,1,2.9,3,2,8,8,8,0

[Events]
Format: Layer, Start, End, Style, Actor, MarginL, MarginR, MarginV, Effect, Text
Dialogue: 0,0:00:00.35,0:00:03.44,Daikuuji (KGNE Omake),,0000,0000,0000,,That fart... on an insignificant thing like that!
Dialogue: 0,0:00:04.48,0:00:05.23,Daikuuji (KGNE Omake),,0000,0000,0000,,Was there something?
Dialogue: 0,0:00:07.32,0:00:08.98,Takayuki (KGNE),,0000,0000,0000,,It's not your fault, Akane-chan.
Dialogue: 0,0:00:09.99,0:00:10.78,Mayu (KGNE Omake),,0000,0000,0000,,Senpai!


[slooff]

happy.neko, I was running build-pms-binaries.sh in a new /clean directory without any existing stuff in it.

[happy.neko]

Raptor399, мay be you have "Use ASS default style" box unchecked?

[happy.neko]

Raptor399?

[Raptor399]

Sorry, had a busy last few days, no time to test.
However, I have tested it now!

I have updated the binaries to SB22 and the same regression happens: different fonts are used for the subtitles.
Examples make the regression easier to explain.

MEncoder-SB20 ("Use ASS default style" not selected), MEncoder-SB20 ("Use ASS default style" selected) and MEncoder-SB22 ("Use ASS default style" selected):


MEncoder-SB22, "Use ASS default style" not selected:


In other words: checking the "Use ASS default style" indeed fixes the problem: the original fonts of SB20 return.

I'm not sure if this is a regression or an intentional fix in MEncoder.
But whatever it was, it changes the default behavior of PMS.

I think we should change the default value of configuration option "mencoder_ass_defaultstyle" to true to keep the default PMS behavior the same.

Agreed?

[happy.neko]

In other words: checking the "Use ASS default style" indeed fixes the problem: the original fonts of SB20 return.

I'm not sure if this is a regression or an intentional fix in MEncoder.
But whatever it was, it changes the default behavior of PMS.

I think we should change the default value of configuration option "mencoder_ass_defaultstyle" to true to keep the default PMS behavior the same.

Agreed?

I think gui checkbox label is misleading. In my patch I renamed it from "Use ASS default style" to "Use embedded ASS style". When it is ticked PMS uses style from SSA/ASS subtitle file, when unticked overrides it with user-entered values. Very useful for SRT subtitles, not so much for SSA. I also patched in font change for custom style.

As for default config value I'm using "mencoder_ass_defaultstyle = true" in my deb from the beginning because it makes more sense. Who wants SSA subtitles with broken styles?
But this revision of mencoder is the first one where style overriding actually works, before it was ignored so config value made no difference.