Bug#760588: html2text: UTF8 patch table is incomplete
Hi Marcos, On Sonntag, 7. September 2014, Marcos Marado wrote: I'll try to give it a look and test it on Monday. ok, cool. Notice that while most of the entries came from that source, some weren't there, so I googled for them (various other sources, which I didn't take note of). ack. please reword / append as appropriate then. cheers, Holger signature.asc Description: This is a digitally signed message part.
Bug#760588: html2text: UTF8 patch table is incomplete
Hi there, On Sun, Sep 7, 2014 at 2:37 PM, Holger Levsen hol...@layer-acht.org wrote: Hi Marcos, On Sonntag, 7. September 2014, Marcos Marado wrote: I'll try to give it a look and test it on Monday. ok, cool. Found some time today, after all. The branch looks fine. Notice that while most of the entries came from that source, some weren't there, so I googled for them (various other sources, which I didn't take note of). ack. please reword / append as appropriate then. Slight rewording, then, here: https://github.com/marado/html2text/commit/371a1d3baa38c89321d6f19b2928478db55d99cb Best regards, -- Marcos Marado -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#760588: html2text: UTF8 patch table is incomplete
Hi, On Sonntag, 7. September 2014, Marcos Marado wrote: Found some time today, after all. The branch looks fine. cool. Slight rewording, then, here: https://github.com/marado/html2text/commit/371a1d3baa38c89321d6f19b2928478d b55d99cb great, thanks. guess I will upload this soon then! :) do you think this will fix some other bugs from https://bugs.debian.org/html2text as well? ;-) cheers, Holger signature.asc Description: This is a digitally signed message part.
Bug#760588: html2text: UTF8 patch table is incomplete
Hi there, On Sun, Sep 7, 2014 at 4:05 PM, Holger Levsen hol...@layer-acht.org wrote: Hi, On Sonntag, 7. September 2014, Marcos Marado wrote: Found some time today, after all. The branch looks fine. cool. Slight rewording, then, here: https://github.com/marado/html2text/commit/371a1d3baa38c89321d6f19b2928478d b55d99cb great, thanks. guess I will upload this soon then! :) Thank you for maintaining this package, and for your prompt replies. do you think this will fix some other bugs from https://bugs.debian.org/html2text as well? ;-) I don't think so - I've checked all open bugs before submiting this new one. I can't be sure, tho, some of those bug reports don't have much info... Cheers, -- Marcos Marado -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#760588: html2text: UTF8 patch table is incomplete
Hi, On Sonntag, 7. September 2014, Marcos Marado wrote: I don't think so - I've checked all open bugs before submiting this new one. I can't be sure, tho, some of those bug reports don't have much info... ok, cool! thanks! cheers, Holger signature.asc Description: This is a digitally signed message part.
Bug#760588: html2text: UTF8 patch table is incomplete
Hey there, On Sep 6, 2014 6:03 AM, Holger Levsen hol...@layer-acht.org wrote: Hi Marcos, On Freitag, 5. September 2014, Marcos Marado wrote: Sure, here it is: https://github.com/marado/html2text/commit/2e7b95138bb932261fed0863d2d28058 cfdacaa3 great, thanks! Some remarks/questions: 900-complete-utf8-entities-table.patch modifies the Makefile, this seems unneccesary, can you fix this, please? Yes, I noticed that after I sent you the link, please look at the latest version of that patch, instead of looking into the commit. And why modify tests/pkgreport.html.supposed at all? Well, if you see the patch for 'supposed' you'll notice that it actually has stuff like quot;thisquot; , when it should be this. Cheers, cheers, Holger -- Marcos Marado
Bug#760588: html2text: UTF8 patch table is incomplete
Hi, On Samstag, 6. September 2014, Marcos Marado wrote: Yes, I noticed that after I sent you the link, please look at the latest version of that patch, instead of looking into the commit. where is the latest version? cheers, Holger signature.asc Description: This is a digitally signed message part.
Bug#760588: html2text: UTF8 patch table is incomplete
https://github.com/marado/html2text/blob/master/debian/patches/900-complete-utf8-entities-table.patch On Sep 6, 2014 12:34 PM, Holger Levsen hol...@layer-acht.org wrote: Hi, On Samstag, 6. September 2014, Marcos Marado wrote: Yes, I noticed that after I sent you the link, please look at the latest version of that patch, instead of looking into the commit. where is the latest version? cheers, Holger
Bug#760588: html2text: UTF8 patch table is incomplete
Hi Marcos, On Samstag, 6. September 2014, Marcos Marado wrote: And why modify tests/pkgreport.html.supposed at all? Well, if you see the patch for 'supposed' you'll notice that it actually has stuff like quot;thisquot; , when it should be this. yes, but how is this related? cheers, Holger signature.asc Description: This is a digitally signed message part.
Bug#760588: html2text: UTF8 patch table is incomplete
Hi Marcos, I've now rebased your branch, cleaned up the two commits from removing the Makefile and remove the debian/changelog entry, plus added the source of the table to the patch description... Can you please take a look at the branch bug760588 and confirm whether this is good / correct to merge into master as it is? Thanks cheers, Holger signature.asc Description: This is a digitally signed message part.
Bug#760588: html2text: UTF8 patch table is incomplete
I'll try to give it a look and test it on Monday. Notice that while most of the entries came from that source, some weren't there, so I googled for them (various other sources, which I didn't take note of). On Sep 6, 2014 11:22 PM, Holger Levsen hol...@layer-acht.org wrote: Hi Marcos, I've now rebased your branch, cleaned up the two commits from removing the Makefile and remove the debian/changelog entry, plus added the source of the table to the patch description... Can you please take a look at the branch bug760588 and confirm whether this is good / correct to merge into master as it is? Thanks cheers, Holger
Bug#760588: html2text: UTF8 patch table is incomplete
Package: html2text Version: 1.3.2a-17 Severity: normal Dear Maintainer, $ echo quot;helloQUOT;, nu; rang; oelig;|html2text quot;helloQUOT;, ν rang; oelig; As you can see, nu; is correctly converted, while the rest is not. That is because the table in the utf8 patch is quite incomplete. Completing it (for instance, using this[1] as a data source, it's still not fully complete, but almost) would fix this issue. [1] http://lists.gnu.org/archive/html/groff/2006-01/msg00113.html Best regards, Marcos Marado -- System Information: Debian Release: jessie/sid APT prefers testing APT policy: (500, 'testing') Architecture: amd64 (x86_64) Kernel: Linux 3.14-2-amd64 (SMP w/4 CPU cores) Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Versions of packages html2text depends on: ii libc6 2.19-10 ii libgcc1 1:4.9.1-4 ii libstdc++6 4.9.1-4 html2text recommends no packages. Versions of packages html2text suggests: ii curl 7.37.1-1 ii wget 1.15-1+b1 -- no debconf information -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#760588: html2text: UTF8 patch table is incomplete
Hi Marcos, thanks for your bug report! On Freitag, 5. September 2014, Marcos Marado wrote: As you can see, nu; is correctly converted, while the rest is not. That is because the table in the utf8 patch is quite incomplete. Completing it (for instance, using this[1] as a data source, it's still not fully complete, but almost) would fix this issue. [1] http://lists.gnu.org/archive/html/groff/2006-01/msg00113.html could you maybe provide a patch for this? cheers, Holger signature.asc Description: This is a digitally signed message part.
Bug#760588: html2text: UTF8 patch table is incomplete
Sure, here it is: https://github.com/marado/html2text/commit/2e7b95138bb932261fed0863d2d28058cfdacaa3 On Fri, Sep 5, 2014 at 5:08 PM, Holger Levsen hol...@layer-acht.org wrote: Hi Marcos, thanks for your bug report! On Freitag, 5. September 2014, Marcos Marado wrote: As you can see, nu; is correctly converted, while the rest is not. That is because the table in the utf8 patch is quite incomplete. Completing it (for instance, using this[1] as a data source, it's still not fully complete, but almost) would fix this issue. [1] http://lists.gnu.org/archive/html/groff/2006-01/msg00113.html could you maybe provide a patch for this? cheers, Holger -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#760588: html2text: UTF8 patch table is incomplete
Hi Marcos, On Freitag, 5. September 2014, Marcos Marado wrote: Sure, here it is: https://github.com/marado/html2text/commit/2e7b95138bb932261fed0863d2d28058 cfdacaa3 great, thanks! Some remarks/questions: 900-complete-utf8-entities-table.patch modifies the Makefile, this seems unneccesary, can you fix this, please? And why modify tests/pkgreport.html.supposed at all? cheers, Holger signature.asc Description: This is a digitally signed message part.