On Fri, Sep 01 2017, Sean Whitton wrote:
> Shouldn't there be a '$' in the first one, too?
Er, I was too tired when I wrote this.
Patch LGTM, though of course needs testing. Thanks again.
--
Sean Whitton
signature.asc
Description: PGP signature
Hello Osamu,
Thank you for working on this bug!
In the case there is a query string you have:
> + $options{'pgpsigurlmangle'} = [ 's/\?/.' . $suffix .
> '?/', ];
And in the case with no query string, you use the old code:
> +
Hi,
I made a quick fix to address this bug as attached against git head:
a830277fec6cea2b4c6cba86ca3a45ceea309b06
This should not only works for default but also work for auto.
Sean Whitton, what do you think of this?
I will test this later this week when I find some time.
Regards,
Osamu
3 matches
Mail list logo