Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Python scripts: define entry_points.console_scripts to have launcher shell/exe scripts #8815

Merged
merged 4 commits into from
Nov 25, 2023

Conversation

rouault
Copy link
Member

@rouault rouault commented Nov 24, 2023

Fixes #8811

(on top of PR #8812)

@rouault rouault added this to the 3.9.0 milestone Nov 24, 2023
@rouault rouault force-pushed the py_exe_installation_scripts branch from 66cc1aa to 45380dd Compare November 24, 2023 15:49
@coveralls
Copy link
Collaborator

Coverage Status

coverage: 67.912%. remained the same
when pulling 45380dd on rouault:py_exe_installation_scripts
into fe649a8 on OSGeo:master.

@rouault rouault merged commit 07ecc07 into OSGeo:master Nov 25, 2023
30 checks passed
rouault added a commit to rouault/gdal that referenced this pull request Nov 25, 2023
…to keep .py scripts (fixes OSGeo#8811)

This is a follow-up of OSGeo#8815 for Windows, and particularly to unbreak conda-forge builds that
test the presence of the .py scripts.

Before this PR, "python setup.py install" would cause the following
logs:
```
Installing gdal2tiles.py script to C:\Miniconda3\envs\test\conda-bld\gdal-split_1700841951001\_h_env\Scripts
[...]
Deleting C:\Miniconda3\envs\test\conda-bld\gdal-split_1700841951001\_h_env\Scripts\gdal2tiles.py
Installing gdal2tiles-script.py script to C:\Miniconda3\envs\test\conda-bld\gdal-split_1700841951001\_h_env\Scripts
Installing gdal2tiles.exe script to C:\Miniconda3\envs\test\conda-bld\gdal-split_1700841951001\_h_env\Scripts
```

This PR hacks around easy_install to avoid the deletiong of the .py
script
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Python utilities: provide launcher scripts
2 participants