[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2017-03-16 Thread Gert van den Berg
I haven't reported it to Debian, I did not have a test system to confirm that it is present there as well (I'm quite sure it would be present, it is the same package source) (And https://www.debian.org/Bugs/Reporting for manual reporting seems quite scary...) -- You received this bug notification

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2017-03-16 Thread Roland Giesler
Confirmed on Debian 8: $ curl -s https://launchpadlibrarian.net/250515419/test.html | html2text; echo $? Input recoding failed due to invalid input sequence. Unconverted part of text follows. “Test��?) 0 -- You received this bug notification because you are a member of Desktop Packages, whi

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2017-03-16 Thread Roland Giesler
Gert, you're right, my bad. I get the full picture now. Have you reported this to the Debian bugtracker? -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to html2text in Ubuntu. https://bugs.launchpad.net/bugs/1564317 Title: Successful r

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2017-03-10 Thread Gert van den Berg
@Roland: Incorrect upstream, that is python-html2text / python3-html2text. This is from http://www.mbayer.de/html2text/ (And that won't have the issue, since it was introduced in Debian's UTF-8 patch) (Debian might also be affected though) -- You received this bug notification because you are a m

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2017-03-10 Thread Roland Giesler
I logged this upstream at https://github.com/Alir3z4/html2text/issues/156 as well. ** Bug watch added: github.com/Alir3z4/html2text/issues #156 https://github.com/Alir3z4/html2text/issues/156 -- You received this bug notification because you are a member of Desktop Packages, which is subscrib

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2017-03-10 Thread Roland Giesler
I can confirm the same incorrect response on Ubuntu 14.04.5 LTS with all updates installed. -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to html2text in Ubuntu. https://bugs.launchpad.net/bugs/1564317 Title: Successful return code on e

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2017-03-10 Thread Launchpad Bug Tracker
Status changed to 'Confirmed' because the bug affects multiple users. ** Changed in: html2text (Ubuntu) Status: New => Confirmed -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to html2text in Ubuntu. https://bugs.launchpad.net/bugs/

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2017-03-02 Thread Gert van den Berg
Also affects 1.3.2a-18 on Ubuntu 16.04. ** Description changed: When running html2text on a file with incorrect encoding (e.g. indicated as us-ascii, but actually UTF-8), it fails with an error message but returns a successful result code (0). Actual output: - $ htm

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2016-04-04 Thread Gert van den Berg
Note: This is probably also present in Debian, since it seems like the Ubuntu package is mostly unmodified. ** Also affects: hundredpapercuts Importance: Undecided Status: New -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to htm

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2016-04-01 Thread Ubuntu Foundations Team Bug Bot
The attachment "Patch to modify patches to return an error code on recoding errors" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team. [This is an automated message p

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2016-03-31 Thread Gert van den Berg
My attempt at an actual patch. The other method, of actually changing all the error handling to save that there have been an error and returning the code at the end might still make sense for later. ** Patch added: "Patch to modify patches to return an error code on recoding errors" https://

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2016-03-31 Thread Gert van den Berg
Diffs for patch files: (for simple fix - return 2 for recoding error, don't attempt to continue to other files) (The problem seems to be present to at least the precise version) --- 611-recognize-input-encoding.patch.orig 2016-03-31 18:25:01.914598768 +0200 +++ 611-recognize-input-encoding.pa

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2016-03-31 Thread Gert van den Berg
The simplest fix is to replace the "continue" in the recode error handlers with "exit(1)" (Patched html2text.C lines 595 and 555). (Using a different value for encoding errors might be useful) This makes the error handling consistent with other cases, like failing to open a file. (See line 482) A

[Desktop-packages] [Bug 1564317] Re: Successful return code on encoding error

2016-03-31 Thread Gert van den Berg
Quick method to test: sudo apt-get install html2text curl -s https://launchpadlibrarian.net/250515419/test.html | html2text; echo $? -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to html2text in Ubuntu. https://bugs.launchpad.net/bugs/1564