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
libainstall doesn't create $(BINDIR) directory #61070
Comments
make libainstall fails when $(BINDIR) does not exist. |
Thanks for the fix. The patches fixes it. Since this is a bug fix, it should be backported all the way till 2.7. |
Digging a little deeper, make libainstall needs $(BINDIR) because of this *And* that python$(VERSION)-config file refers to the python exe which is available only after bininstall or it's dependees are called. In affect, if make libainstall is for using lib/* contents this fix is okay, but leaves a resultant invalid bin/python-config I am adding Martin and Georg who have dabbled with BUILD previously for their views. |
I was using this in the case where I just want to link against libpython.a and for me it is a limited case where I don't really need the functionality of python-config; so for me this is certainly the best approach. But I concede this use case is probably a very little general interest. If the goal is to have a working python-config, then I guess libainstall should depend on bininstall. I'm not too fussed which approach is taken, I just think that (make && make libainstall) should work. I'm happy to fashion a patch for whichever approach is thought to be best. |
Thanks Benno. I think that making libainstall depend on bininstall is |
In a similar manner the bininstall target relies on $(LIBPC), but does not create that. This makes me consider if the libainstall target should be installing pkg-config sciprt at all (and whether bininstall should be installing the .pc files). It is hard for me to determine what the exact intended goals of each of these targets is, so I can't really come up with the right fix. Naively for both targets ensuring that the directory exists solves the symptom, but to be it looks like there is probably a greater underlying thing to determine here. |
Looks like the status for this one should really be "needs decision". It is sounds like the provided patch is not sufficient regardless, so 'commit review' is no longer the appropriate state. I'm changing it back to patch needed as the closest available state, but we also need a decision. |
This is still a problem behaviour as of May 2021. ./configure --prefix=/home/senthil/foobar failed with /usr/bin/install: cannot create regular file '/home/senthil/foobar/bin/python3.11-config': No such file or directory |
bpo-25696 was related, and it fixed bininstall issue. |
Looks like the BINDIR dependency was added here - 49fd7fa#diff-1f0a8db227d22005511b0d90f5339b97db345917b863954b3b3ccb9ec308767cR833 but we didn't add the directory creation dependency then. A simple fix of add BINDIR as dependency seems _OK_ to me. At least it wont break the libainstall standalone target. |
Fixed in 3.11 here - 80d250d |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: