GNU bug report logs - #39430
Use CRAN’s canonical URL as home-page

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 39430 in the body.
You can then email your comments to 39430 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Lars-Dominik Braun <ldb <at> leibniz-psychology.org>
To: guix-patches <at> gnu.org
Subject: Use CRAN’s canonical URL as home-page
Date: Wed, 5 Feb 2020 08:49:33 +0100

[Message part 1 (text/plain, inline)]

Hi,

I noticed for packages not specifying any URL the CRAN importer emits home-page
URLs like https://cran.r-project.org/web/packages/<package> (without trailing
slash). These are permanent redirects, thus the linter complains about them.
One option would be adding a trailing slash or use the canonical URL, as
provided by CRAN (on the bottom of the description page). The attached patch
does the latter.

Lars

[0001-import-cran-Use-CRAN-s-canonical-URL-as-home-page.patch (text/x-diff, attachment)]

Message #8 received at 39430 <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: Lars-Dominik Braun <ldb <at> leibniz-psychology.org>
Cc: 39430 <at> debbugs.gnu.org
Subject: Re: [bug#39430] Use CRAN’s canonical URL as
 home-page
Date: Sun, 13 Dec 2020 13:17:11 +0000

[Message part 1 (text/plain, inline)]

Lars-Dominik Braun <ldb <at> leibniz-psychology.org> writes:

> I noticed for packages not specifying any URL the CRAN importer emits home-page
> URLs like https://cran.r-project.org/web/packages/<package> (without trailing
> slash). These are permanent redirects, thus the linter complains about them.
> One option would be adding a trailing slash or use the canonical URL, as
> provided by CRAN (on the bottom of the description page). The attached patch
> does the latter.

Hi,

Sorry for the delay in following up on this.

I gave this patch a try, testing using gProfileR as just a random
package I picked.

So, my take on this is that the canonical URLs still redirect, but using
a 303 redirect, which the linter doesn't complain about. Given I see [1]
on the page, I guess it makes sense to use these URLs.

1: Please use the canonical form https://CRAN.R-project.org/package=gProfileR to link to this page.

I'll leave it a day or two for others to comment, as I'm not
particularly familiar with CRAN, but I'm happy to merge this.

Thanks,

Chris

[signature.asc (application/pgp-signature, inline)]

Message #13 received at 39430-done <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: Christopher Baines <mail <at> cbaines.net>
Cc: 39430-done <at> debbugs.gnu.org, Lars-Dominik Braun <ldb <at> leibniz-psychology.org>
Subject: Re: [bug#39430] Use CRAN’s canonical URL as
 home-page
Date: Tue, 15 Dec 2020 22:35:29 +0000

[Message part 1 (text/plain, inline)]

Christopher Baines <mail <at> cbaines.net> writes:

> Lars-Dominik Braun <ldb <at> leibniz-psychology.org> writes:
>
>> I noticed for packages not specifying any URL the CRAN importer emits home-page
>> URLs like https://cran.r-project.org/web/packages/<package> (without trailing
>> slash). These are permanent redirects, thus the linter complains about them.
>> One option would be adding a trailing slash or use the canonical URL, as
>> provided by CRAN (on the bottom of the description page). The attached patch
>> does the latter.
>
> Hi,
>
> Sorry for the delay in following up on this.
>
> I gave this patch a try, testing using gProfileR as just a random
> package I picked.
>
> So, my take on this is that the canonical URLs still redirect, but using
> a 303 redirect, which the linter doesn't complain about. Given I see [1]
> on the page, I guess it makes sense to use these URLs.
>
> 1: Please use the canonical form https://CRAN.R-project.org/package=gProfileR to link to this page.
>
> I'll leave it a day or two for others to comment, as I'm not
> particularly familiar with CRAN, but I'm happy to merge this.

I've gone ahead and pushed this as
f14bb29defb36051039e977ba3791b542d90578a. I tweaked one of the variable
names.

Thanks again,

Chris

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.