Opened 9 years ago

Closed 9 years ago

#9194 closed patch (Fixed)

Improved xmltv error reporting

Reported by: Sverker Wiberg <sverkerw@…> Owned by: beirdo
Priority: minor Milestone: 0.25
Component: MythTV - MythSystem Version: Master Head
Severity: low Keywords:
Cc: Ticket locked: no

Description

This patch parses xmltv's termination status (from system()) into exit status (from main()) or signal number as applicable, and logging its findings -- instead of simply logging the unparsed termination status (still a last resort).

This should simplify xmltv troubleshooting; and I'd like to think that ​http://www.google.com/search?q=xmltv+error+256 shows that this would be very useful.

Attachments (1)

mythfilldatabase.exit-status.diff​ (2.5 KB) - added by Sverker Wiberg <sverkerw@…> 9 years ago.

Download all attachments as: .zip

Change History (6)

Changed 9 years ago by Sverker Wiberg <sverkerw@…>

Attachment: mythfilldatabase.exit-status.diff​ added

comment:1 Changed 9 years ago by Sverker Wiberg <sverkerw@…>

In short: The patch changes 

2010-11-07 15:50:30.122 FillData, Error: xmltv returned error code 256

into 

2010-11-07 15:55:00.331 FillData, Error: xmltv exited with non-zero exit status 1.

comment:2 Changed 9 years ago by beirdo

Owner: changed from stuartm to beirdo
Status: new → assigned

comment:3 Changed 9 years ago by beirdo

This is a prime candidate for replacing with myth_system in the next round of upgrades to it.

comment:4 Changed 9 years ago by beirdo

Component: MythTV - Mythfilldatabase → MythTV - MythSystem

comment:5 Changed 9 years ago by beirdo

Milestone: unknown → 0.25
Resolution: → Fixed
Status: assigned → closed

This area of code was reworked in the mythsystem-rewrite branch, which has been merged. This patch is no longer needed.

Note: See TracTickets for help on using tickets.