[issue12152] Parser/asdl_c.py relies on mercurial repository revision

2011-05-31 Thread Ralf Schmitt
Ralf Schmitt sch...@gmail.com added the comment: issue 12225 contains a patch. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12152 ___ ___

[issue12152] Parser/asdl_c.py relies on mercurial repository revision

2011-05-23 Thread Matthias Klose
New submission from Matthias Klose d...@debian.org: when building without an hg repository present, the build fails with: ./Parser/asdl_c.py -h ../Include ../Parser/Python.asdl Traceback (most recent call last): File ../Parser/asdl_c.py, line 1214, in module main(args[0]) File

[issue12152] Parser/asdl_c.py relies on mercurial repository revision

2011-05-23 Thread Éric Araujo
Changes by Éric Araujo mer...@netwok.org: -- nosy: +benjamin.peterson ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12152 ___ ___ Python-bugs-list

[issue12152] Parser/asdl_c.py relies on mercurial repository revision

2011-05-23 Thread R. David Murray
R. David Murray rdmur...@bitdance.com added the comment: asdl_c.py should not be run during a non-checkout build, since the files it builds should already have been (re)built and checked in by the time a release is rolled. IIUC The release building script is supposed to ensure the timestamps

[issue12152] Parser/asdl_c.py relies on mercurial repository revision

2011-05-23 Thread Benjamin Peterson
Benjamin Peterson benja...@python.org added the comment: Indeed, Include/Python-ast.c and Python/Python-ast.c should be touched to avoid this. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12152

[issue12152] Parser/asdl_c.py relies on mercurial repository revision

2011-05-23 Thread Matthias Klose
Matthias Klose d...@debian.org added the comment: I'm building a snapshot package on a buildd, where a checkout is used (without the .hg directory). buildds don't have access to the network, and I don't want to package the .hg directory. So I assume I have to create a release tarball on my

[issue12152] Parser/asdl_c.py relies on mercurial repository revision

2011-05-23 Thread Benjamin Peterson
Benjamin Peterson benja...@python.org added the comment: It's true that Python use to be the only dependency for asdl_c.py, but now Mercurial is, too. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12152

[issue12152] Parser/asdl_c.py relies on mercurial repository revision

2011-05-23 Thread R. David Murray
R. David Murray rdmur...@bitdance.com added the comment: And like Benjamin said, it should be enough to touch those two files. You don't have to actually build a release tarball. -- ___ Python tracker rep...@bugs.python.org

[issue12152] Parser/asdl_c.py relies on mercurial repository revision

2011-05-23 Thread Matthias Klose
Matthias Klose d...@debian.org added the comment: ok, thanks. good enough -- resolution: - invalid status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12152 ___