Opened 18 years ago
Closed 18 years ago
#1737 closed defect (fixed)
Update mtd to build and run on Mac OS X
Reported by: | Owned by: | Anduin Withers | |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | mythdvd | Version: | |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
mtd needs to some small updates to build and run on Mac OS X.
libmythdvdnav also needs a small update to correctly the load the CSS library if it exists too.
Attachments (2)
Change History (10)
Changed 18 years ago by
Attachment: | dvdprobe.cpp.patch added |
---|
Changed 18 years ago by
Attachment: | dvd_input.c.patch added |
---|
Patch to libs/libmythdvdnav/dvd_input.c loads CSS library if present on DYLD_LIBRARY_PATH on Mac OS X
comment:1 Changed 18 years ago by
With the attached patches and setting the DVD Device to /dev/rdisk1 I can rip VOBs and also transcode to other forrmats (using standard transcode settings and build per mythdvd instructions). Ripping to ISO image appears to still be broken however.
comment:2 Changed 18 years ago by
Owner: | changed from Isaac Richards to Anduin Withers |
---|
comment:3 Changed 18 years ago by
Andrew, this looks good (although I haven't actually tested it yet :-) Some minor thoughts:
- It should probably be dependant on #ifdef Q_OS_MACX instead of Q_WS_MACX - IOKit is an OS feature, and if anyone builds an X11 MythTV on Darwin, we want the to use your code too.
- I think it might be a little cleaner if this Darwin/Mach?-specific stuff goes into a separate file. Maybe somewhere in libmyth, so that other programs (like mfd/mfe?) could use it.
- I have been meaning to implement libmyth/mythcdrom-darwin.cpp for a year or so. Do you know anything about the Disk Arbitration framework?
comment:4 Changed 18 years ago by
- Having to set the DVD Device manually seems very un-Mac-like, and will be wrong if there are extra disks on the Mac. I wonder if we could have an auto-fillin on this the first time the mythdvd plugin loads, or maybe auto-detect a mounted DVD if a particular device isn't specified
comment:5 Changed 18 years ago by
I'm in conversation with Anduin to clean up the patch to get it to fit style guidelines etc. etc.
I'm not personally familiar with the Disk Arbitration framework - but I've been writing Mac software for a long time, I don't expect it would take long to 'work it out'. I'll take a look at what libmyth does/needs with removable media and see what help I may be able to give.
I agree about improving the UI to autodetect the device - the code in the patch can be used to find a mounted DVD and I'll probably change the patch to reflect that an empty device path in the call to probe will find the first mounted DVD (there's no real way from the UI to choose between multiple mounted DVDs). However the current default value for that field is /dev/dvdrom and I don't know how to change that from the UI perspective only on Mac OS.
comment:7 Changed 18 years ago by
comment:8 Changed 18 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Since mtd does compile and run, I will close this ticket. Future OS X improvements deserve their own ticket (like #1753). Setting the DVD device to /dev/rdisk1 or /dev/rdisk2 means that mtd finds the partitions, and the default (biggest?) one is displayed in the mythdvd screens.
Patch to mythdvd/mtd/dvdprobe.cpp supports DVD media detection and changes.