Skip to content

BUG: fix download link in setup.py (which is shown on PyPi). #2707

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

Merged
merged 1 commit into from
Nov 14, 2012

Conversation

rgommers
Copy link
Member

@rgommers rgommers commented Nov 5, 2012

Closes Trac ticket 2243.

@certik: should be backported to 1.7.x

I'll upload a new PKGINFO to PyPi for 1.6.2 when this is merged.

MAJOR = 1
MINOR = 8
MAJOR = 1
MINOR = 8
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to be changed to 7 for the backport, right?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed. I can fix that and backport this myself; Ondrej is busy IIRC.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might as well commit it then. I checked the URL and it looks good to me. I assume PyPi can figure out the versions from there?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Guess so, but I'll check when I upload a fix for 1.6.2 to PyPi.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eh, sorry, replied too fast. Your question doesn't parse. PyPi doesn't figure out versions. Do you mean pip?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On Fri, Nov 9, 2012 at 9:56 AM, Ralf Gommers notifications@github.comwrote:

In setup.py:

LICENSE = 'BSD'
CLASSIFIERS = filter(None, CLASSIFIERS.split('\n'))
AUTHOR = "Travis E. Oliphant, et.al."
AUTHOR_EMAIL = "oliphant@enthought.com"
PLATFORMS = ["Windows", "Linux", "Solaris", "Mac OS-X", "Unix"]
-MAJOR = 1
-MINOR = 8
+MAJOR = 1
+MINOR = 8

Eh, sorry, replied too fast. Your question doesn't parse. PyPi doesn't
figure out versions. Do you mean pip?

Actually, I don't know what I'm talking about ;) I just used your PR
comment.

@certik
Copy link
Contributor

certik commented Nov 14, 2012

@rgommers, can you do this? I don't want to make a mistake while backporting this.

rgommers pushed a commit that referenced this pull request Nov 14, 2012
BUG: fix download link in setup.py (which is shown on PyPi).
@rgommers rgommers merged commit 067b5f8 into numpy:master Nov 14, 2012
@rgommers
Copy link
Member Author

OK, merging this. Will backport too.

@rgommers
Copy link
Member Author

Backported in 98ca13d.

@stefanv
Copy link
Contributor

stefanv commented Nov 14, 2012

BTW, that should also probably be "et al."
On Nov 14, 2012 12:32 PM, "Ralf Gommers" notifications@github.com wrote:

Backported in 98ca13d 98ca13d.


Reply to this email directly or view it on GitHubhttps://github.com//pull/2707#issuecomment-10383224.

@rgommers
Copy link
Member Author

"that" = ?

@stefanv
Copy link
Contributor

stefanv commented Nov 14, 2012

Sorry, "Travis Oliphant, et. al" -> "Travis Oliphant et al." in the AUTHOR field.

@rgommers
Copy link
Member Author

Yeah, it's been et al for a while. New PR?

@stefanv
Copy link
Contributor

stefanv commented Nov 14, 2012

My most glorious PR yet: #2744 :-)

@certik
Copy link
Contributor

certik commented Nov 14, 2012

Thanks!

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

Successfully merging this pull request may close these issues.

4 participants