Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

casing in googlecode urls #107

Closed
keszybz opened this issue Mar 2, 2015 · 8 comments
Closed

casing in googlecode urls #107

keszybz opened this issue Mar 2, 2015 · 8 comments
Labels

Comments

@keszybz
Copy link

keszybz commented Mar 2, 2015

I added https://release-monitoring.org/project/5737/ for BioNetGen, but the check fails because the url has lowercase bionetgen:
bad: https://code.google.com/p/BioNetGen/downloads/list?sort=releasedate
good: https://code.google.com/p/bionetgen/downloads/list?sort=releasedate

Upstream camelcases the tarball names, so the camelcased name seems to be the right one to use. It would be really nice if I could use the standard googlecode pattern, but allow the name to be lowercase.

@pypingou
Copy link
Member

pypingou commented Mar 4, 2015

So this is an easyfix in the google backend

@cicku
Copy link

cicku commented Mar 13, 2015

Just a friendly note, Google Code will be shutdown in 2016.

@pypingou
Copy link
Member

Hi Christopher, thanks for the heads up, do you have a link for this?

-------- Message d'origine --------
De : Christopher Meng notifications@github.com
Date :13/03/2015 05:52 (GMT+01:00)
À : fedora-infra/anitya anitya@noreply.github.com
Cc : Pierre-Yves Chibon pingou@pingoured.fr
Objet : Re: [anitya] casing in googlecode urls (#107)

Just a friendly note, Google Code will be shutdown in 2016.


Reply to this email directly or view it on GitHub.

@cicku
Copy link

cicku commented Mar 14, 2015

http://google-opensource.blogspot.com/2015/03/farewell-to-google-code.html

devel list has already advised.

Anyway, about this issue, is it ok to just use lower() to handle the case like bionetgen?

@pypingou
Copy link
Member

@cicku good idea yes

@Aayush-Kasurde
Copy link
Contributor

I would like fix this issue, Can anyone please help me or guide me to fix it ?

@Aayush-Kasurde
Copy link
Contributor

Please see my PR - #113 and provide comments.

@pypingou
Copy link
Member

This has been fixed by #114

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants