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

error: can't copy 'build/temp.linux-x86_64-cpython-311/symengine_install/lib/python3.11/site-packages/symengine/lib/symengine_wrapper.cpython-311-x86_64-linux-gnu.so': doesn't exist or not a regular file #330

Closed
KOLANICH opened this issue May 22, 2023 · 4 comments
Labels
bug Something isn't working

Comments

@KOLANICH
Copy link

KOLANICH commented May 22, 2023

Describe the bug
The whole build/temp.linux-x86_64-cpython-311/symengine_install/lib/python3.11 doesn't exist.

find shows that the file is are in
build/temp.linux-x86_64-cpython-311/symengine_install/local/lib/python3.11/dist-packages/symengine/lib/symengine_wrapper.cpython-311-x86_64-linux-gnu.so

To Reproduce
Steps to reproduce the behavior, e.g.:

  1. python3 -m build -nwx .

Expected behavior
Should be built.

Environment (please complete the following information):

  • OS and version: Debian sid
  • Python version 3.11
  • SymForce Version c3d2f3c
@jackpittenger
Copy link

I am also facing this issue with python310. Did you have any luck getting it built? @KOLANICH

aaron-skydio added a commit that referenced this issue Jun 2, 2023
So that it works on different platforms that e.g. use `local/lib` or
`site-packages`.

We were doing this correctly for editable installs but not non-editable
installs

Longer-term I'm leaning more and more towards requiring building
our copy of symengine separately first, but this is an easy strict
improvement for now

Fixes #330

Topic: sf-wrapper-glob
Reviewers: bradley,nathan,chao,ryan-b
@aaron-skydio
Copy link
Member

Try #342 ?

aaron-skydio added a commit that referenced this issue Jun 2, 2023
So that it works on different platforms that e.g. use `local/lib` or
`site-packages`.

We were doing this correctly for editable installs but not non-editable
installs

Longer-term I'm leaning more and more towards requiring building
our copy of symengine separately first, but this is an easy strict
improvement for now

Fixes #330

Topic: sf-wrapper-glob
Reviewers: bradley,nathan,chao,ryan-b
aaron-skydio added a commit that referenced this issue Jun 2, 2023
So that it works on different platforms that e.g. use `local/lib` or
`site-packages`.

We were doing this correctly for editable installs but not non-editable
installs

Longer-term I'm leaning more and more towards requiring building
our copy of symengine separately first, but this is an easy strict
improvement for now

Fixes #330

Topic: sf-wrapper-glob
Reviewers: bradley,nathan,chao,ryan-b
aaron-skydio added a commit that referenced this issue Jun 2, 2023
So that it works on different platforms that e.g. use `local/lib` or
`site-packages`.

We were doing this correctly for editable installs but not non-editable
installs

Longer-term I'm leaning more and more towards requiring building
our copy of symengine separately first, but this is an easy strict
improvement for now

Fixes #330

Topic: sf-wrapper-glob
Reviewers: bradley,nathan,chao,ryan-b
@KOLANICH
Copy link
Author

KOLANICH commented Jun 3, 2023

Did you have any luck getting it built?

Yes, I have built it. But I have done it semi-manually, somewhere editing CMakeCache.txt, somewhere replacing pieces of files with own code, somewhere hardcoding paths, somewhere using patchelf utility. It took me a lot of time to just build this "masterpiece of marvelous engineering" into a wheel that can be installed with pip and with which the examples from the ReadMe work. And I had to unload X11 and almost all the system services in order just to have enough RAM to build native code part of this package. The mere invocations of the compiler took a lot of time in my system. Since it is so problematic for me to rebuild the package, I'm not going to try #342 unless I really have to do the painful rebuilding process again.

@KOLANICH
Copy link
Author

KOLANICH commented Jun 8, 2023

@aaron-skydio, @bradley-solliday-skydio, @nathan-skydio, @chao-qu-skydio, @ryan-brott-skydio, thank you for fixing this. I have not tested this yet, but I hope that it works.

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.

3 participants