Opened 16 years ago

Closed 15 years ago

#5406 closed defect (invalid)

Fixed a bug in sa3250ch-BR

Reported by: hadees Owned by: Rob Smith
Priority: minor Milestone: unknown
Component: contrib Version: unknown
Severity: medium Keywords:
Cc: Ticket locked: no

Description

The original sa3250ch script was hard coded to use node 0 however Benton Roberts <benton@…> added support to input your own node and/or port via command line options. He did almost most everything to switch it except actually switching the hard coded zero for the command line node number. He did it for the ports already and it looks like he did some other stuff with different kind of boxes but i'm not sure what. He posted this to the mailing list but doesn't seem to ever have submitted it to trac.

The relevent sections for what I did are in the calls to the function avc1394_transaction_block.

Here is him posting it to the mailing list ​http://www.gossamer-threads.com/lists/mythtv/users/266768#266768

Direct link to his original patch: ​http://bentonroberts.com/personal/media-server/code/sa3250ch-BR.c

Attachments (1)

multiple_sa3250_box_support.path​ (3.5 KB) - added by hadees 16 years ago.
pathc for sa3250ch.c

Download all attachments as: .zip

Change History (5)

Changed 16 years ago by hadees

Attachment: multiple_sa3250_box_support.path​ added

pathc for sa3250ch.c

comment:1 Changed 16 years ago by Rob Smith

Owner: changed from Isaac Richards to Rob Smith
Status: new → accepted

comment:2 Changed 16 years ago by Rob Smith

Status: accepted → infoneeded

Can you just upload the full file, the patch doesn't apply cleanly to my copy of -BR.c

comment:3 Changed 16 years ago by stuartm

Alternatively please provide a unified diff (svn diff OR diff -U).

comment:4 Changed 15 years ago by danielk

Resolution: → invalid
Status: infoneeded → closed

broken patch, no response to request for working one for 10 mos...

Note: See TracTickets for help on using tickets.