[Xmltv-devel] [ xmltv-Bugs-3075400 ] ParseOptions does not handle --output as documented

[SourceForge.net]

Bugs item #3075400, was opened at 2010-09-25 10:21
Message generated for change (Tracker Item Submitted) made by dekarl
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=3075400&group_id=39046

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: XMLTV.pm
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Karl Dietz (dekarl)
Assigned to: Nobody/Anonymous (nobody)
Summary: ParseOptions does not handle --output as documented

Initial Comment:
documentated behaviour:
If the --output option is specified, STDOUT will be redirected to the specified file.

actual behaviour:
no redirection, so every grabbers handles it themselves


----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=3075400&group_id=39046

[SourceForge.net]

Bugs item #3075400, was opened at 2010-09-25 10:21
Message generated for change (Comment added) made by dekarl
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=3075400&group_id=39046

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: XMLTV.pm
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Karl Dietz (dekarl)
Assigned to: Nobody/Anonymous (nobody)
Summary: ParseOptions does not handle --output as documented

Initial Comment:
documentated behaviour:
If the --output option is specified, STDOUT will be redirected to the specified file.

actual behaviour:
no redirection, so every grabbers handles it themselves


----------------------------------------------------------------------

Comment By: Karl Dietz (dekarl)
Date: 2010-09-25 16:21

Message:
The patch changes ParseOptions from selecting the default output to
redirecting STDOUT and selecting it as default output.
With this patch XMLTV::Writer can write to where ParseOptions wants it to
write without additional STDOUT redirection in every single grabber.

test_grabbers is currently running, will update patch once everything
works again.

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=3075400&group_id=39046

[Karl Dietz]

Post by SourceForge.net
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=3075400&group_id=39046

...

Post by SourceForge.net
The patch changes ParseOptions from selecting the default output to
redirecting STDOUT and selecting it as default output.
With this patch XMLTV::Writer can write to where ParseOptions wants it to
write without additional STDOUT redirection in every single grabber.

Writing _pt_meo made me aware of that bug. I'd like to commit a fix
this week unless there are objections.

Post by SourceForge.net
test_grabbers is currently running, will update patch once everything
works again.

Patches to _ee and _il have been added.
I'd like to remove the (now) duplicate output redirection from more
grabbers, but these two break with the fixed API.

Regards,
Karl

[SourceForge.net]

Bugs item #3075400, was opened at 2010-09-25 10:21
Message generated for change (Comment added) made by dekarl
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=3075400&group_id=39046

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: XMLTV.pm
Group: None

Status: Closed
Resolution: Fixed

Priority: 5
Private: No
Submitted By: Karl Dietz (dekarl)
Assigned to: Nobody/Anonymous (nobody)
Summary: ParseOptions does not handle --output as documented

Initial Comment:
documentated behaviour:
If the --output option is specified, STDOUT will be redirected to the specified file.

actual behaviour:
no redirection, so every grabbers handles it themselves


----------------------------------------------------------------------

Comment By: Karl Dietz (dekarl)

Date: 2010-10-01 18:33

Message:
patch has been commited

----------------------------------------------------------------------

Comment By: Karl Dietz (dekarl)
Date: 2010-09-25 16:21

Message:
The patch changes ParseOptions from selecting the default output to
redirecting STDOUT and selecting it as default output.
With this patch XMLTV::Writer can write to where ParseOptions wants it to
write without additional STDOUT redirection in every single grabber.

test_grabbers is currently running, will update patch once everything
works again.

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=3075400&group_id=39046