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

fix CircleCI shared lib issue with pypy 3.9 & 3.10 #726

Merged
merged 1 commit into from
Dec 6, 2023

Conversation

ikappaki
Copy link
Contributor

@ikappaki ikappaki commented Dec 5, 2023

Hi,

can you please review patch to fix circleci tests failing for pypy 3.9 and 3.10. It fails with the following error during poetry installation:

Installing Poetry (1.7.1): An error occurred. Removing partial environment.
Poetry installation failed.
See /root/project/poetry-installer-error-s__g4uo2.log for error logs.

Exited with code exit status 127

I guess there was a poetry change that somehow affected the virtual environment created by poetry during its installation. It is missing the pypy shared libraries and fails the installation with the following message (as taken from the log)

See /root/project/poetry-installer-error-24g4cn6z.log for error logs.
/root/.local/share/pypoetry/venv/bin/python3: error while loading shared libraries: libffi.so.6: cannot open shared object file: No such file or directory

Traceback:

  File "<stdin>", line 923, in main
  File "<stdin>", line 562, in run

I couldn't find a mention of this issue on the Internet. I've opted to sym link the pypy shared libraries to the venv and add it to the LD_LIBRARY_PATH so they are available at runtime.

Thanks

@ikappaki ikappaki changed the title fix shared lib issue with pypy 3.9 & 3.10 fix CircleCI shared lib issue with pypy 3.9 & 3.10 Dec 5, 2023
Copy link
Member

@chrisrink10 chrisrink10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very bizarre. Thanks for fixing that.

@chrisrink10 chrisrink10 enabled auto-merge (squash) December 5, 2023 22:47
@chrisrink10 chrisrink10 disabled auto-merge December 6, 2023 12:24
@chrisrink10 chrisrink10 merged commit 8a42054 into basilisp-lang:main Dec 6, 2023
7 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