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

pamm compiled as a mac executable when installed trough setup.py #26

Open
Iximiel opened this issue Dec 10, 2021 · 2 comments · May be fixed by #27
Open

pamm compiled as a mac executable when installed trough setup.py #26

Iximiel opened this issue Dec 10, 2021 · 2 comments · May be fixed by #27
Assignees
Labels
bug Something isn't working

Comments

@Iximiel
Copy link

Iximiel commented Dec 10, 2021

as title
if I install gmplabtools with python setup.py install the pamm executable that is put un the directory ../gmplabtools-0.0.0-py3.9.egg/gmplabtools/pamm/bin is a mac executable:

$  ./pamm
-bash: ./pamm: cannot execute binary file: Exec format error
$  file pamm
pamm: Mach-O 64-bit x86_64 executable, flags:<NOUNDEFS|DYLDLINK|TWOLEVEL|PIE>
@Iximiel Iximiel added the bug Something isn't working label Dec 10, 2021
@giovannidoni
Copy link
Contributor

I think the reason why that happens is that you need to run python setup.py compile first, that creates the executable, while in your case, you are probably using the executable found in the repo (which should probably not be there).

To fix this you can try to delete the exec in the .gmplabtools/pamm/bin folder and the compile with the command above.

To make the process clearer and avoid confusion:

  • delete the exec in the .gmplabtools/pamm/bin folder and add an empty .gitkeep file;
  • add the "./gmplabtools/pamm/bin/*" folder here as a part of cleanup directories

@Iximiel Iximiel self-assigned this Dec 10, 2021
@Iximiel
Copy link
Author

Iximiel commented Dec 10, 2021

Ok, I'm creating a pull request to solve the issue.
I think the problem is if you find a user like me that goes for setup.py install without reading first the README.

The idea of shipping a precompiled pamm exec is not a bad idea tough. pamm depends on blas and lapack for compiling. If blas and lapack are not on the system a non sudoer o a non expert user can have a hard time in compiling pamm

@Iximiel Iximiel linked a pull request Dec 10, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants