[issue34765] Update install-sh

Michael Felt added the comment: The AIX build-bots thank you. Back to "failed-test" status. 1721...failed test (failure) 1720...failed compile (failure) -- ___ Python tracker

[issue34765] Update install-sh

STINNER Victor added the comment: New changeset ed271b2350486b1fa031fa70e1e99d0d9b2b2133 by Victor Stinner in branch 'master': bpo-34765: install-sh is executable (GH-10225) https://github.com/python/cpython/commit/ed271b2350486b1fa031fa70e1e99d0d9b2b2133 --

[issue34765] Update install-sh

STINNER Victor added the comment: I made install-sh executable again. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue34765] Update install-sh

Michael Felt added the comment: FYI: On my manual build server I have coreutils "install" installed, and it seems install-sh is not called in that case (which is why I never saw with manual builds) FYI: There is also an issue (I hope side-effect) that pyexpat is not building while

[issue34765] Update install-sh

Change by STINNER Victor : -- pull_requests: +9539 stage: resolved -> patch review ___ Python tracker ___ ___ Python-bugs-list

[issue34765] Update install-sh

Change by STINNER Victor : -- resolution: fixed -> status: closed -> open ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue34765] Update install-sh

Michael Felt added the comment: This is closed, however, since this was merged the AIX buildbots have failed. This is because the file mode bits lack the -x root@x066:[/data/prj/python/git/cpython-master]find . -name install-sh -ls 148833829 16 -rw-r--r-- 1 root felt 15368

[issue34765] Update install-sh

Charalampos Stratakis added the comment: Alright the static scanner warns in the case of a typo being made where the variable is actually used, but since the variable is indeed not used, that doesn't not make it a bug. -- ___ Python tracker

[issue34765] Update install-sh

Charalampos Stratakis added the comment: Generic reference on the issue: https://github.com/koalaman/shellcheck/wiki/SC2034 -- ___ Python tracker ___

[issue34765] Update install-sh

Charalampos Stratakis added the comment: Adding to this, the reason I initially caught up with that, was due to a coverity scan. More explicitly: Error: SHELLCHECK_WARNING: [#def1] /usr/lib64/python3.6/config-3.6dm-x86_64-linux-gnu/install-sh:63:1: warning: transform_arg appears unused.

[issue34765] Update install-sh

STINNER Victor added the comment: Thanks Charalampos Stratakis for the change! > If there's no actual bug, it certainly does not need to be done on > maintenance branches. I concur. Since install-sh has been updated in the master branch, I close the issue. -- resolution: -> fixed

[issue34765] Update install-sh

STINNER Victor added the comment: New changeset b7ad31c8926aad2fdd0d4661373c25cc9d753a40 by Victor Stinner (stratakis) in branch 'master': bpo-34765: Update the install-sh file (GH-9592) https://github.com/python/cpython/commit/b7ad31c8926aad2fdd0d4661373c25cc9d753a40 -- nosy:

[issue34765] Update install-sh

Change by Charalampos Stratakis : -- keywords: +patch pull_requests: +8990 stage: -> patch review ___ Python tracker ___ ___

[issue34765] Update install-sh

Zachary Ware added the comment: If there's no actual bug, it certainly does not need to be done on maintenance branches. -- nosy: +zach.ware versions: -Python 2.7, Python 3.6, Python 3.7 ___ Python tracker

[issue34765] Update install-sh

New submission from Charalampos Stratakis : The install-sh file that python uses for autotools is horribly outdated. Last update was 16 years ago and it's being copied from automake's source code. Updating it to modern standards could potentially fix issues for systems that use autotools,