-
-
Notifications
You must be signed in to change notification settings - Fork 18.2k
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 libpthread not found during local installation #40706
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are you sure? this will likely break other linux arches
and not even clear why this is actually needed
Hey @jreback, to be honest I don't know in general, I had to do this to get my development environment working. |
sure would take that as a note (eg you may have to do this in some platforms) |
@cgarciae Thanks for checking out pandas and your first PR! Now, I agree we shouldn't just add this. And even instead of adding a note about it, I would first try to understand why this is needed or what's going wrong (eg I am also on Ubuntu 20.04, and using conda environments, but didn't need this). It could also point to something wrong with your conda env (does your conda env have |
Hey @jorisvandenbossche! I do have |
This is an odd problem indeed. The conda gcc compiler itself will add
even though Pandas doesn't use any multi-threading explicitly (I believe). Can you add a full build log that shows the error in the PR description @cgarciae? |
@@ -299,6 +299,11 @@ We'll now kick off a three-step process: | |||
python setup.py build_ext -j 4 | |||
python -m pip install -e . --no-build-isolation --no-use-pep517 | |||
|
|||
.. note:: | |||
If the ``python setup.py build_ext -j 4`` command fails on a linux distribution you might |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we be more specific than "you might be able to"? is there reason to believe this is a common failure mode?
Can you echo your ldconfig -vvvv variables before you build? |
Hey @rgommers! I added the logs to the description. |
One part of the output:
So as @rgommers said above, for some reason it is not looking for / not finding libpthread in the conda environment's sysroot. Googling a bit, it seems to come up regularly a while ago (ContinuumIO/anaconda-issues#7363, cocodataset/cocoapi#94), but that was end of 2017 with the migration to new compilers which could give problems. @cgarciae could you also show the output of You could maybe also try to explicitly deactive and reactive the environment, to be sure the env variables are properly set. One of the things mentioned in ContinuumIO/anaconda-issues#7363 is to check |
Another observation (from ContinuumIO/anaconda-issues#7363 (comment)) that I am seeing here as well: you have So I suppose that's indicating that not all compiler flags are properly set up for the conda env (not fully/correctly activated for some reason?) |
then you are probably building with your system compiler rather than the conda compilers.
I've found that in some cases that's not enough, and you really need a completely fresh terminal. |
Thanks a lot @jorisvandenbossche and @rgommers! Thanks to your pointers I found the culprit: vscode's terminal seems to do something that makes this happen. I manually activated the conda env from the gnome terminal then ran the previous code and it worked just fine. I'll be closing the issue since it must be something specific to my setup. |
My previous comment was wrong, I forgot to remove the patch to the |
Ah that makes sense, environment activation (which sets the sysroot, compilers, etc.) does shell-specific things. |
@cgarciae you might want to raise an issue at https://github.com/conda/conda to report about the |
Doing a quick look for their fish support, I actually see https://docs.conda.io/projects/conda/en/latest/user-guide/install/linux.html#using-with-fish-shell. Did you do |
@jorisvandenbossche yeah, # >>> conda initialize >>>
# !! Contents within this block are managed by 'conda init' !!
eval /home/cristian/miniforge3/bin/conda "shell.fish" "hook" $argv | source
# <<< conda initialize <<< and it works for most stuff but it seems to be failing for this specific use case. I'll post an issue there, thanks for all the help! |
Hey! This is my first PR, I am getting to know the code base and found this as I was doing the local setup.
I was installing the development environment and it was failing due to:
This also happens in the Docker container under some conditions (creating a new env).
Steps followed:
Adding a
extra_link_args.append("-L/usr/lib/x86_64-linux-gnu/")
fixed the issue on Ubuntu 20.04.Logs