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

Add shebang to setup.py #1078

Merged
merged 2 commits into from
Dec 13, 2024
Merged

Add shebang to setup.py #1078

merged 2 commits into from
Dec 13, 2024

Conversation

Unike267
Copy link
Contributor

Add shebang to the setup.py file to support installation using:

./setup.py install

Without explicitly specifying the interpreter as Python, the script execution fails (for obvious reasons).

Error:

user@machine:~/path/vunit$ sudo ./setup.py install
[sudo] contraseña para user: 
./setup.py: línea 9: $'\nPyPI setup script\n': orden no encontrada
./setup.py: línea 13: from: orden no encontrada
./setup.py: línea 14: from: orden no encontrada
./setup.py: línea 15: from: orden no encontrada
./setup.py: línea 18: error sintáctico cerca del elemento inesperado `0,'
./setup.py: línea 18: `sys.path.insert(0, str(Path(__file__).parent.resolve()))'

By adding the appropriate shebang, the interpreter will be correctly identified, ensuring the script runs as expected.

Cheers!


/cc @umarcor

@LarsAsplund LarsAsplund merged commit 35d1aa6 into VUnit:master Dec 13, 2024
14 checks passed
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.

2 participants