On 29-01-2018 00:11, Markus Koschany wrote:
>
> I noticed that you had to import apt but reportbug does not depend on
> python3-apt. After I had installed this package it worked. I also
> believe you don't need to check for the upstream changelog.gz file, the
> Debian changelog should be sufficien
Hi,
On Mon, Jan 29, 2018 at 01:29:59PM +0100, Markus Koschany wrote:
> [sorry forgot to CC the rest]
>
> Hello,
>
> Am 29.01.2018 um 09:30 schrieb Guido Günther:
> > Hi Markus,
> > thanks for pursuing this! Some minor nitpicks:
> >
> > On Mon, Jan 29, 2018 at 12:11:03AM +0100, Markus Koschany wro
Hi,
On Mon, Jan 29, 2018 at 01:34:51PM +0100, Markus Koschany wrote:
>
> >> This will not catch 404 or similar http status codes. If you don't care
> >> about the type of error you can just do …
> >>
> >>r.raise_for_status()
>
> Ok, got it. So adding r.raise_for_status
>> This will not catch 404 or similar http status codes. If you don't care
>> about the type of error you can just do …
>>
>>r.raise_for_status()
Ok, got it. So adding r.raise_for_status() below the request line should
be sufficient because requests.exceptions.RequestE
[sorry forgot to CC the rest]
Hello,
Am 29.01.2018 um 09:30 schrieb Guido Günther:
> Hi Markus,
> thanks for pursuing this! Some minor nitpicks:
>
> On Mon, Jan 29, 2018 at 12:11:03AM +0100, Markus Koschany wrote:
> [..snip..]
>> +if utils.is_security_update(package, pkgversion):
>> +
Hi Markus,
thanks for pursuing this! Some minor nitpicks:
On Mon, Jan 29, 2018 at 12:11:03AM +0100, Markus Koschany wrote:
[..snip..]
> +if utils.is_security_update(package, pkgversion):
> +if ui.yes_no('Do you want to report a regression because of
> a security update