bug#39430: closed (Use CRAN’s canonical URL as home-page)

[GNU bug Tracking System]

Your message dated Tue, 15 Dec 2020 22:35:29 +0000
with message-id <87lfdy3dfy.fsf@cbaines.net>
and subject line Re: [bug#39430] Use CRAN’s canonical URL as home-page
has caused the debbugs.gnu.org bug report #39430,
regarding Use CRAN’s canonical URL as home-page
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
39430: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=39430
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: Use CRAN’s canonical URL as home-page Date: Wed, 5 Feb 2020 08:49:33 +0100 User-agent: Mutt/1.10.1 (2018-07-13)

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
Description: Text Data

--- End Message ---

--- Begin Message --- Subject: Re: [bug#39430] Use CRAN’s canonical URL as home-page Date: Tue, 15 Dec 2020 22:35:29 +0000 User-agent: mu4e 1.4.13; emacs 27.1

Christopher Baines <mail@cbaines.net> writes:

> Lars-Dominik Braun <ldb@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
Description: PGP signature

--- End Message ---