On 2023/07/20 18:19, Antoine Jacoutot wrote:
> On Thu, Jul 20, 2023 at 01:47:42PM +0100, Stuart Henderson wrote:
> > On 2023/07/20 14:27, Antoine Jacoutot wrote:
> > > On Thu, Jul 20, 2023 at 10:24:10AM +0100, Stuart Henderson wrote:
> > > > portroach doesn't reliably use the site handler for cpan, various ports
> > > > (notably those with Foo-Bar-v1.23 format distfile names) fallback to
> > > > directory listing. For example see the "L" instead of "S" for
> > > > p5-Sys-Virt in
> > > > https://portroach.openbsd.org/antoine%20jacoutot%20%3cajacou...@openbsd.org%3E.html
> > > > 
> > > > Also the "is this site handled by this" regex doesn't cover some URL
> > > > formats (though afaict as long as one of the various MASTER_SITES
> > > > entries matches, it looks like it does pick it up anyway).
> > > > 
> > > > I think this diff should help but it's not tested. Is there any good way
> > > > to test portroach without setting up a full installation and having it
> > > > fetch a list of URLs the size of 45 aardvarks?
> > > 
> > > I can put this on portroach.openbsd.org if you want?
> > 
> > If it's not likely to mess anything up too badly for the database, yes
> > please - I don't think it can make things much worse anyway :)
> 
> Running with it.
> portroach.openbsd.org has been updated with the latest ports with this patch
> if you want to check.

It didn't fix existing ones on portroach.openbsd.org but I've done a
quick install on a machine here and it is correctly using the right site
handler during a fetch with debug enabled, and doesn't show incorrect
"new" versions for p5-* things where I know there was a problem (e.g.
p5-Calendar-Simple listed for ports@), so I think the diff is good.

I suspect portroach isn't overwriting an entry where it already thinks
it has a "new" version, so clearing the db should help fix those.

Reply via email to