Re: Issue 337 in specto: 0.4 setup.py installs incorrectly from tarballs

[spe...@googlecode.com]

Updates:
Summary: 0.4 setup.py installs incorrectly from tarballs

Comment #1 on issue 337 by nekoh...@gmail.com: 0.4 setup.py installs
incorrectly from tarballs
http://code.google.com/p/specto/issues/detail?id=337

If you naïvely put a VERSION file in the resulting tarball, you hit another
traceback:

File "setup.py", line 106, in <module>
with open(os.path.join ("spectlib/constants.py.in"), "rt") as file_in:
IOError: [Errno 2] No such file or directory: 'spectlib/constants.py.in'

[spe...@googlecode.com]

Updates:
Status: Started

Comment #2 on issue 337 by Francesco.Marella: 0.4 setup.py installs

incorrectly from tarballs
http://code.google.com/p/specto/issues/detail?id=337

(No comment was entered for this change.)

[spe...@googlecode.com]

Comment #3 on issue 337 by Francesco.Marella: 0.4 setup.py installs

incorrectly from tarballs
http://code.google.com/p/specto/issues/detail?id=337

I'll revert to the old setup.py script for now.
The VERSION file it's useless anyway and I will not bring it back. The
other issue is with the prefix that we used to save for later use, I will
hardcode it again.

[spe...@googlecode.com]

Comment #4 on issue 337 by Francesco.Marella: 0.4 setup.py installs

incorrectly from tarballs
http://code.google.com/p/specto/issues/detail?id=337

I'll revert to the old setup.py script for now.

The VERSION file its useless anyway and I will not bring it back. The other

[spe...@googlecode.com]

Comment #5 on issue 337 by Francesco.Marella: 0.4 setup.py installs

incorrectly from tarballs
http://code.google.com/p/specto/issues/detail?id=337

I think I've sorted out all the problems with the setup script and without
reverting to the old one.

The main change is that the version string for specto is located at
spectlib/__init__.py. Getting the version it's just 'from spectlib import
__pkg_version__' and you're done.

nekohayo, could you checkout this branch
lp:~francesco-marella/specto/fix-sdist
and test it?

[spe...@googlecode.com]

Comment #6 on issue 337 by vinyviny...@gmail.com: 0.4 setup.py installs

incorrectly from tarballs
http://code.google.com/p/specto/issues/detail?id=337

He had the same problem, the branch decided for me.

Thanks

-----------------------------------
Translate by Google Translate
Original message:

Estava com o mesmo problema, o branch resolveu para mim.

Agradeço

[spe...@googlecode.com]

Comment #7 on issue 337 by nekoh...@gmail.com: 0.4 setup.py installs

incorrectly from tarballs
http://code.google.com/p/specto/issues/detail?id=337

Hi Francesco, sorry for making you wait (I was at guadec). I just tested
your branch, I did:

python setup.py sdist

Then uncompressed the resulting tarball to a different folder, and in that
folder I ran "sudo setup.py install", which failed:

running install_scripts
copying build/scripts-2.7/specto -> /usr/bin
changing mode of /usr/bin/specto to 755
running install_data
error: can't copy 'specto.desktop': doesn't exist or not a regular file

jeff@kusanagi:~/dev/specto/main/dist/specto-0.4.1$ sudo python setup.py
uninstall
running uninstall
warning: uninstall: Could not read installed files list installed_files

[spe...@googlecode.com]

Comment #8 on issue 337 by Francesco.Marella: 0.4 setup.py installs

incorrectly from tarballs
http://code.google.com/p/specto/issues/detail?id=337

Hi,

run `python setup.py build` before install please.

[spe...@googlecode.com]

Updates:
Status: Committed
Labels: -Milestone-Release0.4 Milestone-Release0.4.1

Comment #9 on issue 337 by nekoh...@gmail.com: 0.4 setup.py installs

incorrectly from tarballs
http://code.google.com/p/specto/issues/detail?id=337

Oops, you are correct, I should add that to the instructions :) I'll merge
the branch then. Thanks!

[spe...@googlecode.com]

Updates:
Status: Fixed

Comment #10 on issue 337 by nekoh...@gmail.com: 0.4 setup.py installs

incorrectly from tarballs
http://code.google.com/p/specto/issues/detail?id=337

Pushed out a new tarball, so marking as fixed.