Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SMI style tags are displayed as is #3017

Closed
trustin opened this issue Apr 3, 2016 · 6 comments
Closed

SMI style tags are displayed as is #3017

trustin opened this issue Apr 3, 2016 · 6 comments

Comments

@trustin
Copy link

@trustin trustin commented Apr 3, 2016

Built from master. Debian 7.

I'm not exactly sure since when, but MPV started to display the style tags such as <FONT COLOR=...> in .SMI subtitle files as is rather than stripping them away or applying styles correctly.

image

@ghost
Copy link

@ghost ghost commented Apr 3, 2016

Works with a sample file I have (in fact the sample file which is used by ffmpeg to test this functionality).

Can you upload a non-working .smi file?

@ghost
Copy link

@ghost ghost commented Apr 3, 2016

Actually, before you do this, against which ffmpeg version is mpv linked?

@ghost ghost added the meta:info-needed label Apr 3, 2016
@trustin
Copy link
Author

@trustin trustin commented Apr 4, 2016

$ ffmpeg -v
ffmpeg version N-79143-g8ff0f6a Copyright (c) 2000-2016 the FFmpeg developers
  built with gcc 4.9.2 (Debian 4.9.2-10)
  configuration: --prefix=/usr --enable-shared --enable-gpl --enable-libmp3lame --enable-libx264 --extra-cflags='-march=native -O3' --extra-cxxflags='-march=native -O3'
  libavutil      55. 19.100 / 55. 19.100
  libavcodec     57. 30.100 / 57. 30.100
  libavformat    57. 29.101 / 57. 29.101
  libavdevice    57.  0.101 / 57.  0.101
  libavfilter     6. 40.102 /  6. 40.102
  libswscale      4.  0.100 /  4.  0.100
  libswresample   2.  0.101 /  2.  0.101
  libpostproc    54.  0.100 / 54.  0.100

I built ffmpeg from ffmpeg master. What branch would you recommend instead?

@trustin
Copy link
Author

@trustin trustin commented Apr 4, 2016

As a workaround, I used this script to convert .smi to .ass:

https://gist.github.com/trustin/e06fadcbca90985b1df4dafebd9f5d25

@ghost
Copy link

@ghost ghost commented Apr 4, 2016

ffmpeg master is fine.

Still, mpv could be possibly linked or configured against another version, so either look at the pmp log for the version, or run mpv CLI without argument.

A sample would still be appreciated. You also can check if this is a ffmpeg issue with ffmpeg -i file.smi out.ass.

@mia-0
Copy link
Member

@mia-0 mia-0 commented Jul 21, 2017

Guess we’re not getting a sample.

@mia-0 mia-0 closed this Jul 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants