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

Use console_scripts instead of scripts #88

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

depau
Copy link

@depau depau commented Nov 7, 2024

uv seems to not be able to run scripts installed via scripts= in setup.py on Windows; it doesn't seem to accept the provided .bat script (see astral-sh/uv#8888)

This change bypasses the issue by installing awslocal as a console_script, which produces an exe when installed on Windows.

You can verify that python3 -m venv venv; venv\Scripts\activate.bat; pip install . produces venv\Scripts\awslocal.exe when ran on Windows with my changes.

This also fixes the issue with uv: uv tool install . installs awslocal.exe in the binaries directory.

This still does not fix the issue in #84, so the program still segfaults, but at least it's installed properly.

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.

1 participant