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.

Sign up for GitHub

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

Jump to bottom

mpv crashes on quit (Segmentation fault: 11) MBP 2016 #3864

Closed
SimonAth opened this issue Dec 3, 2016 · 3 comments
Closed

mpv crashes on quit (Segmentation fault: 11) MBP 2016 #3864

SimonAth opened this issue Dec 3, 2016 · 3 comments
Labels
os:mac

Comments

@SimonAth
Copy link

SimonAth commented Dec 3, 2016

mpv version and platform

/usr/local/Cellar/mpv/HEAD-ec74a79_1 (66 files, 38.6M) *
Built from source on 2016-12-02 at 14:17:56 with: --with-bundle --with-libdvdread
From: https:/Homebrew/homebrew-core/blob/master/Formula/mpv.rb

Mac OSX Sierra on a 2016 MacBook Pro with Radeon Pro 460

Reproduction steps

Quit the program (using q or cmd+c) whilst playing a video in fullscreen

Expected behavior

Program closes quietly.

Actual behavior

Program closes with Segmentation fault: 11, and throws an error "mpv quit unexpectedly"

Log file

Make a log file made with -v or --log-file=output.txt, paste it to
http://sprunge.us/YagA

OSX problem details and system configuration
http://sprunge.us/JgTV

Sample files

same issue on any file when played in mpv's fullscreen mode.
(mpv is fine when not in fullscreen, or in OSX native fullscreen mode.)
@ghost ghost added the os:mac label Dec 3, 2016
@Akemi Akemi mentioned this issue Dec 11, 2016
Merged
@Akemi
Copy link
Member

Akemi commented Dec 11, 2016

my PR #3873 will most likely fix this issue. would be nice if you could confirm it.

@SimonAth
Copy link
Author

Yes, #3873 fixes this issue.
Cheers

Akemi added a commit to Akemi/mpv that referenced this issue Dec 12, 2016
@Akemi
cocoa: fullscreen refactoring
45d1d05 
this replaces the old fullscreen with the native
macOS fullscreen.

Fixes mpv-player#2857 mpv-player#3272 mpv-player#1352 mpv-player#2062 mpv-player#3864
Akemi added a commit to Akemi/mpv that referenced this issue Dec 13, 2016
@Akemi
cocoa: fullscreen refactoring
699730f 
this replaces the old fullscreen with the native
macOS fullscreen.

Fixes mpv-player#2857 mpv-player#3272 mpv-player#1352 mpv-player#2062 mpv-player#3864
Akemi added a commit to Akemi/mpv that referenced this issue Dec 14, 2016
@Akemi
cocoa: fullscreen refactoring
769913e 
this replaces the old fullscreen with the native
macOS fullscreen. the fs-black-out-screens
option was removed because the used API
was changed. it can possibly be readded
if wanted.

Fixes mpv-player#2857 mpv-player#3272 mpv-player#1352 mpv-player#2062 mpv-player#3864
Akemi added a commit to Akemi/mpv that referenced this issue Dec 14, 2016
@Akemi
cocoa: fullscreen refactoring
ca2be7a 
this replaces the old fullscreen with the native
macOS fullscreen.

Fixes mpv-player#2857 mpv-player#3272 mpv-player#1352 mpv-player#2062 mpv-player#3864
Akemi added a commit to Akemi/mpv that referenced this issue Dec 14, 2016
@Akemi
cocoa: fullscreen refactoring
1132648 
this replaces the old fullscreen with the native
macOS fullscreen. additional the
--fs-black-out-screens was removed since the new
API doesn't support it in a way the old one did.
it can possibly be re-added if done manually.

Fixes mpv-player#2857 mpv-player#3272 mpv-player#1352 mpv-player#2062 mpv-player#3864
Akemi added a commit to Akemi/mpv that referenced this issue Dec 15, 2016
@Akemi
cocoa: fullscreen refactoring
2b7f6b9 
this replaces the old fullscreen with the native
macOS fullscreen. additional the
--fs-black-out-screens was removed since the new
API doesn't support it in a way the old one did.
it can possibly be re-added if done manually.

Fixes mpv-player#2857 mpv-player#3272 mpv-player#1352 mpv-player#2062 mpv-player#3864
Akemi added a commit to Akemi/mpv that referenced this issue Dec 15, 2016
@Akemi
cocoa: fullscreen refactoring
132bd6d 
this replaces the old fullscreen with the native
macOS fullscreen. additional the
--fs-black-out-screens was removed since the new
API doesn't support it in a way the old one did.
it can possibly be re-added if done manually.

Fixes mpv-player#2857 mpv-player#3272 mpv-player#1352 mpv-player#2062 mpv-player#3864
Akemi added a commit that referenced this issue Dec 15, 2016
@Akemi
cocoa: fullscreen refactoring
a8347eb 
this replaces the old fullscreen with the native
macOS fullscreen. additional the
--fs-black-out-screens was removed since the new
API doesn't support it in a way the old one did.
it can possibly be re-added if done manually.

Fixes #2857 #3272 #1352 #2062 #3864
@Akemi
Copy link
Member

Akemi commented Dec 15, 2016

fixed in a8347eb

@Akemi Akemi closed this as completed Dec 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
os:mac
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Akemi @SimonAth