Modify

Ticket #8371 (closed defect: Fixed)

Opened 3 years ago

Last modified 16 months ago

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

darwinfirewire.patch (3.0 KB) - added by Ojas Parekh <mythtv@…> 3 years ago.
libs/libmythtv/darwinfirewire.cpp patch
darwinfirewire-intel.patch (1.8 KB) - added by nigel 20 months ago.
Recent trunk

Change History

Changed 3 years ago by Ojas Parekh <mythtv@…>

libs/libmythtv/darwinfirewire.cpp patch

comment:1 Changed 3 years ago by robertm

  • Owner changed from danielk to nigel
  • Component changed from MythTV - Recording to Ports - OSX
  • Milestone set to unknown

comment:2 Changed 3 years ago by robertm

  • Status changed from new to assigned

Changed 20 months ago by nigel

Recent trunk

comment:3 Changed 20 months ago by Github

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 Changed 16 months ago by nigel

  • Status changed from assigned to closed
  • Resolution set to Fixed

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

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.