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,
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 /
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
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()`
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
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
6 matches
Mail list logo