Control: tag -1 + pending

On 29 September 2017 at 05:00, astian <ast...@eclipso.at> wrote:
> I think that commit 27ac5129ce187c6f571cac25ef70553bb9c9d475 broke the
> error message dman used to produce when it failed to fetch some page.
> It says "not found: " but it no longer says what it didn't find.

Indeed, there is an error in the script. I have fixed it in GIT.

> I actually run into this several weeks ago and was ready to send a patch
> back then, but then I ended up rewriting most when I decided to
> implement caching.

The script you provided looks quite nice. I am going to submit it as a
separate bug report.

> I can't remember where I read that the maintainer doesn't like
> maintaining scripts longer than a couple hundred lines, and this one
> grew to about 390 so in the end I didn't send it and forgot about the
> bug report.  Today I remembered and here it is.

It might have been in #861611, where Antoine Beaupré said it. I am not
against scripts >100 lines and your replacement has some nice
improvements that could be worth including. I'll wait until the other
co-maintainers review it before committing it, though.

Regards

Javier

Reply via email to