[issue14546] lll.py can't handle multiple parameters correctly

STINNER Victor added the comment: Thanks Carton He for the bug report, thanks Zackery Spytz for the fix, thanks Serhiy Storchaka for the review ;-) -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue14546] lll.py can't handle multiple parameters correctly

STINNER Victor added the comment: New changeset 7c2c01f02a1821298a62dd16ecc3a12da663e14b by Victor Stinner (Zackery Spytz) in branch '2.7': [2.7] bpo-14546: Fix the argument handling in Tools/scripts/lll.py (GH-13026) (GH-13063) https://github.com/python/cpython/commit/7c2c01f02a1821298a62dd

[issue14546] lll.py can't handle multiple parameters correctly

STINNER Victor added the comment: New changeset e85ba1e69288e3fce400ed4cdbefab58934b5904 by Victor Stinner (Miss Islington (bot)) in branch '3.7': [3.7] bpo-14546: Fix the argument handling in Tools/scripts/lll.py (GH-13026) (GH-13060) https://github.com/python/cpython/commit/e85ba1e69288e3f

[issue14546] lll.py can't handle multiple parameters correctly

Change by Zackery Spytz : -- pull_requests: +12980 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://ma

[issue14546] lll.py can't handle multiple parameters correctly

Change by miss-islington : -- pull_requests: +12977 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://m

[issue14546] lll.py can't handle multiple parameters correctly

STINNER Victor added the comment: > test_tools: test_lll_multiple_dirs() failed on AMD64 Windows7 SP1 3.x: see > bpo-36776. I guess that the test should be decorated by @support.skip_unless_symlink or something like that. -- ___ Python tracker

[issue14546] lll.py can't handle multiple parameters correctly

STINNER Victor added the comment: test_tools: test_lll_multiple_dirs() failed on AMD64 Windows7 SP1 3.x: see bpo-36776. -- nosy: +vstinner ___ Python tracker ___ ___

[issue14546] lll.py can't handle multiple parameters correctly

Serhiy Storchaka added the comment: New changeset c4e78b116f9a4299f3b3bfbbd18ef49782bb1143 by Serhiy Storchaka (Zackery Spytz) in branch 'master': bpo-14546: Fix the argument handling in Tools/scripts/lll.py (GH-13026) https://github.com/python/cpython/commit/c4e78b116f9a4299f3b3bfbbd18ef4978

[issue14546] lll.py can't handle multiple parameters correctly

Zackery Spytz added the comment: I've created a PR for this issue (with a test). -- nosy: +ZackerySpytz versions: +Python 3.7, Python 3.8 -Python 3.2, Python 3.3 ___ Python tracker __

[issue14546] lll.py can't handle multiple parameters correctly

Change by Zackery Spytz : -- pull_requests: +12948 stage: test needed -> patch review ___ Python tracker ___ ___ Python-bugs-list ma

[issue14546] lll.py can't handle multiple parameters correctly

R. David Murray added the comment: Thanks for the patch. Do you have any interest in writing a test for this? Tests for tools go in Lib/test/test_tools.py. -- nosy: +r.david.murray stage: -> test needed type: -> behavior versions: +Python 3.2, Python 3.3 -Python 3.4 __

[issue14546] lll.py can't handle multiple parameters correctly

New submission from Carton He : Space errors in calling of lll(arg) of main() cause it only applies to the last parameter instead of all parameters. Patch attached for Python 3.1 -- components: Demos and Tools files: lll.py.patch keywords: patch messages: 158036 nosy: carton priority: