root/status @ 413

Revision 413, 0.6 kB (checked in by lincoln, 6 years ago)

seperate out call_prog's overloading of $retval into two variables.
this was causing expressions which expected a numeric $retval to succeed
where they shouldn't have.

e.g. a shepherd.log from a user indicated shepherd was accepting the output from a failed plugin even though it shouldn't have:

SHEPHERD: Using postprocessor: augment_timezone
SHEPHERD: Excuting command: /home/mythtv/.shepherd/postprocessors/augment_timezone/augment_timezone --region 73 --channels_file /home/mythtv/.shepherd/channels.conf --output /home/mythtv/.shepherd/postprocessors/augment_timezone/output.xmltv --days 7 /home/mythtv/.shepherd/reconcilers/reconciler_mk2/output.xmltv

:::::: Output from augment_timezone
: adjust_timezone v0.04
: Not a SCALAR reference at /home/mythtv/.shepherd/postprocessors/augment_timezone/augment_timezone line 110.
:::::: End output from augment_timezone

SHEPHERD: Warning: plugin 'augment_timezone' output file '/home/mythtv/.shepherd/postprocessors/augment_timezone/output.xmltv' does not exist
SHEPHERD: accepting output from postprocessor augment_timezone, feeding it into next stage

whereas it should have rejected it up-front.

resolves bugs introduced in [401], [406] and [410].

Line 
1application     shepherd            0.4.29
2grabber         yahoo7widget        1.71
3grabber         rex                 3.3.8
4grabber         abc_website         2.03
5grabber         abc2_website        2.03
6grabber         sbsnews_website     0.04
7grabber         oztivo              0.13
8grabber         jrobbo              0.06
9grabber         d1                  0.6.2.4-r3
10grabber         ninemsn             0.04
11grabber         yahoo7web           0.04
12reconciler      reconciler_mk2      0.17
13postprocessor   imdb_augment_data   0.06
14postprocessor   augment_timezone    0.05
Note: See TracBrowser for help on using the browser.