This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Distutils2: add logging message to report successful installation
Type: behavior Stage: resolved
Components: Distutils2 Versions: Python 3.3, 3rd party
process
Status: closed Resolution: out of date
Dependencies: Superseder:
Assigned To: eric.araujo Nosy List: alex.gronholm, alexis, eric.araujo, tarek
Priority: normal Keywords:

Created on 2012-03-17 19:55 by alex.gronholm, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
d2log.txt alex.gronholm, 2012-03-17 19:55 Full installation log
Messages (7)
msg156196 - (view) Author: Alex Grönholm (alex.gronholm) * Date: 2012-03-17 19:55
Running distutils2 hg tip (changeset 2cec52b682a9):

The command "pysetup install" fails with:
u'Python': u'2.7.2+' is not a valid version (field 'Version')

Not that it should matter, but this is Ubuntu 11.10.

Also, distutils2 installs fine using setup.py.
msg156211 - (view) Author: Éric Araujo (eric.araujo) * (Python committer) Date: 2012-03-17 23:30
Your log file seems to tell that the installation goes fine.  The last message is just a warning.  So what makes you say that d2 doesn’t install?
msg156212 - (view) Author: Alex Grönholm (alex.gronholm) * Date: 2012-03-17 23:33
Being new to d2, I wouldn't know that the installation was completed successfully or if there was something left out.

Regardless, this needs to be fixed.
msg156213 - (view) Author: Éric Araujo (eric.araujo) * (Python committer) Date: 2012-03-17 23:35
I would agree, if you told me *what* needs to be fixed :)  First, the unhelpful warning will go (that’s another bug); second, would you be satisfied if I added a logging message to tell that the installation completed successfully?  The tradition of unix command-line applications is to say nothing if everything goes fine, but if we’re not working in quiet mode then a final logging message would be better.
msg156214 - (view) Author: Alex Grönholm (alex.gronholm) * Date: 2012-03-17 23:38
It doesn't say "Warning" there, so how was I supposed to determine that it's not an error which terminated the installation process?

If it is indeed just a warning, just getting rid of it would fix this for me. However, will this cause trouble with "requires-python" in setup.cfg?
msg156215 - (view) Author: Éric Araujo (eric.araujo) * (Python committer) Date: 2012-03-17 23:44
> It doesn't say "Warning" there
Right, so that’s a problem with our logging config.

> However, will this cause trouble with "requires-python" in setup.cfg?
I’m not sure why you’re asking that, as d2’s setup.cfg does not use requires-python and does produce the warning, so it’s unrelated.  (The warning comes from d2.database.)
msg156217 - (view) Author: Alex Grönholm (alex.gronholm) * Date: 2012-03-17 23:50
> I’m not sure why you’re asking that, as d2’s setup.cfg does not use requires-python and does produce the warning, so it’s unrelated.  (The warning comes from d2.database.)

I was concerned that maybe the version comparator would barf on a version like 2.7.2+ and refuse to process the "requires-python" directive, but if you say this is not the case then I'll just leave it at that.
History
Date User Action Args
2022-04-11 14:57:28adminsetgithub: 58560
2014-03-13 02:31:54eric.araujosetstatus: open -> closed
resolution: out of date
stage: resolved
2012-03-17 23:50:16alex.gronholmsetmessages: + msg156217
2012-03-17 23:44:32eric.araujosetmessages: + msg156215
title: Distutils2 won't install on Ubuntu -> Distutils2: add logging message to report successful installation
2012-03-17 23:38:23alex.gronholmsetmessages: + msg156214
2012-03-17 23:35:22eric.araujosetmessages: + msg156213
2012-03-17 23:33:05alex.gronholmsetmessages: + msg156212
2012-03-17 23:30:07eric.araujosetmessages: + msg156211
versions: + 3rd party
2012-03-17 19:55:49alex.gronholmcreate