[issue9822] windows batch files are dependent on cmd current directory
Zachary Ware added the comment: Fixed by #21907. -- assignee: - zach.ware resolution: - fixed stage: - resolved status: open - closed superseder: - Update Windows build batch scripts versions: +Python 3.5 -Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9822] windows batch files are dependent on cmd current directory
Zachary Ware added the comment: I just stumbled across this issue in looking for another issue, and this turns out to be of interest to me as well. As such, I've updated sorin's patch to apply to the current default branch, added the same kind of change to the new-since-then clean-amd64.bat, and fixed a bug in the initial patch to external-common.bat--it had been doing 'cd /D %~dp0\..' instead of %~dp0\..\..\... It also looks like my editor stripped some trailing whitespace in a couple of files. A much larger change that I made was to switch to using pushd/popd to make the initial dir change, and to return to the calling dir. It strikes me as good practice not to change directory without warning simply by calling a batch file. Alternately, things could be switched back to cd /D instead of pushd, and switch popd to echo Leaving you in %CD%... just to give some notice of the change. I can't test whether the change would affect the buildbots, but it doesn't look like it should. I have tested calling each file from several locations, including the root of the source tree, and everything seems to work as expected. I will also be attaching a patch that also moves external*.bat into PCbuild as Martin suggested, as that does seem a more natural place for them. -- components: +Windows nosy: +zach.ware versions: +Python 3.4 -Python 3.2 Added file: http://bugs.python.org/file29068/issue9822.v2.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9822] windows batch files are dependent on cmd current directory
Zachary Ware added the comment: Here's the patch that moves external*.bat into PCbuild. I also took the opportunity to give the three files more descriptive names: external.bat - get_externals.bat, external-common.bat - get_common_externals.bat, and external-amd64.bat - get_externals-amd64.bat. I'm pretty sure I got all references to them; I know I got everything in PCbuild\readme.txt and in the Tools\buildbot scripts. As an aside, it seems like my last message caused my avast! to freak out about and refuse to load this page, calling it an HTML:Script-inf virus. My apologies if anyone else is similarly affected. -- Added file: http://bugs.python.org/file29069/issue9822.v3.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9822] windows batch files are dependent on cmd current directory
Martin v. Löwis mar...@v.loewis.de added the comment: I did not realize the build instructions recommended using the buildbot tool. It is clearly past time I got around to doing windows build myself. I'd rather prefer a different setup, also: it would be better if the batch files to get the sources lived in PC (or perhaps PCbuild), with Tools/buildbot just referring to them. Please regenerate the diffs from the top level of the checkout, and we'll see if one of the windows devs feels like checking it in. If the buildbots continue to work, it's fine with me. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9822] windows batch files are dependent on cmd current directory
New submission from sorin sorin.sbar...@gmail.com: Currently most batch files from Tools\buildbot do fail to run if you do not call them from the python source root directory. I already have patched files, but the question is against which branch should I make them, considering that this bug is valid for all versions? Do you prefer patch or just the entire files? -- components: Build messages: 116009 nosy: sorin priority: normal severity: normal status: open title: windows batch files are dependent on cmd current directory type: compile error versions: Python 2.6, Python 2.7, Python 3.1, Python 3.2, Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9822] windows batch files are dependent on cmd current directory
Changes by sorin sorin.sbar...@gmail.com: -- keywords: +patch Added file: http://bugs.python.org/file18828/buildbot_curdir.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9822] windows batch files are dependent on cmd current directory
R. David Murray rdmur...@bitdance.com added the comment: The general rule for submitting patches is to make them against development trunk (which is currently the py3k branch). Unified diffs from the top level of the checkout are preferred; whole files are not useful. However, it is not clear what bug you are trying to fix here. Why would the buildbot scripts ever get executed from anywhere except the checkout directory? They are obviously working just fine as they are, since the buildbots are using them successfully. -- nosy: +r.david.murray versions: -Python 2.6, Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9822] windows batch files are dependent on cmd current directory
sorin sorin.sbar...@gmail.com added the comment: In this case I made the patch on the right branch (py3k). These scripts could be called manually and it doesn't make any sense not to make them work in all cases. Not every Windows developer is using Visual Studio to build things, 'some' of them are using the command line, and in this case it does make a sense to be able to call this scripts without these 'strange' limitations. Implementing this would simplify dev setup/usage on Windows. Also I updated the PCbuild/readme.txt file. -- Added file: http://bugs.python.org/file18829/readme.txt.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9822] windows batch files are dependent on cmd current directory
R. David Murray rdmur...@bitdance.com added the comment: I did not realize the build instructions recommended using the buildbot tool. It is clearly past time I got around to doing windows build myself. Please regenerate the diffs from the top level of the checkout, and we'll see if one of the windows devs feels like checking it in. -- nosy: +brian.curtin, loewis, tim.golden type: compile error - feature request versions: -Python 2.7, Python 3.1 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9822] windows batch files are dependent on cmd current directory
Brian Curtin cur...@acm.org added the comment: This is just a guess, but it might have to do with how buildbot works, so I'd be careful of changing this. A lot of things (everything?) in that Tools\buildbot folder depend on being run from the top-level directory in order to work correctly. Another example: If you run Tools\buildbot\external.bat from a directory other than the top-level, the checkouts will end up in a place that pcbuild.sln doesn't know about, thus various subprojects will not build because they can't find the source. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9822] windows batch files are dependent on cmd current directory
sorin sorin.sbar...@gmail.com added the comment: This is true, that this can impact the build process but the changes are assuring that it will always chdir to the right directory. BTW, %~dp0 is the directory when the batch file is located. -- Added file: http://bugs.python.org/file18831/path_for_bug_9822_v1.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9822] windows batch files are dependent on cmd current directory
Changes by sorin sorin.sbar...@gmail.com: Removed file: http://bugs.python.org/file18829/readme.txt.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9822] windows batch files are dependent on cmd current directory
Changes by sorin sorin.sbar...@gmail.com: Removed file: http://bugs.python.org/file18828/buildbot_curdir.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9822 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
