| 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) *  |
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) * |
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) * |
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. |
|
|