Bug#872646: qa.debian.org: [debcheck] Escape some HTML before outputting

2019-12-15 Thread Colin Watson
On Sun, Aug 20, 2017 at 02:53:09PM +0200, gregor herrmann wrote: > On Sun, 20 Aug 2017 10:13:47 +0200, Mattia Rizzolo wrote: > > On Sat, Aug 19, 2017 at 11:20:50AM -0700, Chris Lamb wrote: > > > Updated patch attached, although the last hunk is probably unnecessary > > > anyway. > > > > Although,

Bug#872646: qa.debian.org: [debcheck] Escape some HTML before outputting

2017-08-20 Thread Chris Lamb
Hi Mattia, > Although, I'm not a perl guy so I must ask before applying: […] ACK all those issues. Tbh the patch was more of a demonstration; I'm sure the real author will have a much cleaner solution. :) Regards, -- ,''`. : :' : Chris Lamb `. `'` la...@debian.org /

Bug#872646: qa.debian.org: [debcheck] Escape some HTML before outputting

2017-08-20 Thread gregor herrmann
On Sun, 20 Aug 2017 10:13:47 +0200, Mattia Rizzolo wrote: > On Sat, Aug 19, 2017 at 11:20:50AM -0700, Chris Lamb wrote: > > Updated patch attached, although the last hunk is probably unnecessary > > anyway. > > Although, I'm not a perl guy so I must ask before applying: > * shouldn't that functi

Bug#872646: qa.debian.org: [debcheck] Escape some HTML before outputting

2017-08-20 Thread Mattia Rizzolo
Control: tag -1 moreinfo On Sat, Aug 19, 2017 at 11:20:50AM -0700, Chris Lamb wrote: > Updated patch attached, although the last hunk is probably unnecessary > anyway. Thanks for the patch! Although, I'm not a perl guy so I must ask before applying: * shouldn't that function be `escape_html()`

Bug#872646: qa.debian.org: [debcheck] Escape some HTML before outputting

2017-08-19 Thread Chris Lamb
Hi, > qa.debian.org: [debcheck] Escape some HTML before outputting Updated patch attached, although the last hunk is probably unnecessary anyway. Regards, -- ,''`. : :' : Chris Lamb `. `'` la...@debian.org / chris-lamb.co.uk `- >From b57aea649dd0ee90d6f7e2bf44

Bug#872646: qa.debian.org: [debcheck] Escape some HTML before outputting

2017-08-19 Thread Chris Lamb
Package: qa.debian.org Severity: important Tags: patch Hi, Can someone review and release the attached patch: commit b57aea649dd0ee90d6f7e2bf44f6d6119ed71815 Author: Chris Lamb Date: Sat Aug 19 10:59:07 2017 -0700 debcheck: Escape some HTML before outputting. Disco