Opened 16 years ago

Closed 14 years ago

#8371 closed defect (Fixed)

Firewire recording doesn't work on Intel Macs

Reported by: Ojas Parekh <mythtv@…> Owned by: Nigel
Priority: minor Milestone: unknown
Component: Ports - OSX Version: 0.23rc2
Severity: medium Keywords:
Cc: Ticket locked: no

Description

From the MythTV on OS X wiki:

"(As of mythtv 0.21-fixes-20090316) There is one important note if trying to record via FireWire on an Intel Mac. The MythBackend cannot get a LAM lock signal when recording TV via FireWire. The solution is to check the "Open with Rosetta" checkbox on the Get Info panel of MythBackend.app. See this ​post for information.

Because of this, it's important to open MythBackend.app from the Finder and not from a Terminal window because it won't open with Rosetta. There must be some PPC-specific FireWire code in there."

I believe I've identified and fixed the issue, but I need help testing. With the patch below, I'm able to get a LAM lock on my Intel Mini and record successfully but occasionally have errors with certain channels. I'd like to know if the latter is related to the above bug, cause by a more general firewire (on OS X) bug, or just an issue I have with my particular provider and location. I've also cleaned up some OS X firewire-related header includes.

Attachments (2)

darwinfirewire.patch​ (3.0 KB ) - added by Ojas Parekh <mythtv@…> 16 years ago.
libs/libmythtv/darwinfirewire.cpp patch
darwinfirewire-intel.patch​ (1.8 KB ) - added by Nigel 14 years ago.
Recent trunk

Download all attachments as: .zip

Change History (6)

by Ojas Parekh <mythtv@…>, 16 years ago

Attachment: darwinfirewire.patch​ added

libs/libmythtv/darwinfirewire.cpp patch

comment:1 by robertm, 16 years ago

Component: MythTV - Recording → Ports - OSX
Milestone: → unknown
Owner: changed from danielk to Nigel

comment:2 by robertm, 15 years ago

Status: new → assigned

by Nigel, 14 years ago

Attachment: darwinfirewire-intel.patch​ added

Recent trunk

comment:3 by Github, 14 years ago

Try to fix Firewire recording on Intel Macs. Refs #8371.

Untested by me except for compilation. There might be other data fields that need endian swapping, but hopefully this won't break working PPC setups.

Branch: master Changeset: ef64fad06f941a9f01fe0cf48fcf8eb350dc17ff

comment:4 by Nigel, 14 years ago

Resolution: → Fixed
Status: assigned → closed

Haven't heard any problems, so will close for now.

Note: See TracTickets for help on using tickets.