Add an explicit --with-python= configure option#37
Add an explicit --with-python= configure option#37monnerat wants to merge 1 commit intomate-desktop:masterfrom monnerat:python-version
Conversation
This allows specifying a Python interpreter without using the obscure trick of externally defining PYTHON in environment. The configure report now also displays the effective Python version. Ref: #34 (comment)
You have to specify the Python interpreter, not the version. Example:
|
|
It could be done as Actually, I didn't know that |
This has no effect, because they are not handled by our configure file.
Yes, and that's the way raveit65's rpm file for Fedora 29 does. As a consequence, this |
|
Ok, I'm fine with just showing Python version in the configure's summary. |
Successful build with new instructions
|
I just got a successful build and install with but that's probably something wrong on my end and nothing to do with the package. I did NOT get the "site-packages" wrong directory install problem I often see with python 2.7 packages locally built on Debian |
|
@lukefromdc: same old Debian-specific problem (#31), you need to keep the patch I've mentioned there... |
|
OK, applying that patch gave a successful build and install, though it looks like caja-admin(which I used to test a build against python 3.7) might not like python3.7 (though I am no expert on python): |
The problem is definitely not in python-caja: you should report it to the caja-admin project and continue this discussion there.
Thus my plan is to cancel the current PR and create a new one showing in the configure report:
The proposed |
|
I see caja-admin in four different repos on github, not sure if |
This project is not in the Fedora package database , thus I can't help you with some official reference we have. Since you seem to use debian, maybe have a look of their reference if some. If it's a custom package, then you have to chose yourself :-) |
|
I don't actually use it anymore, I was just trying to test this PR |
|
@monnerat: ok, I'm fine with the plan. Yes, caja-admin is still Python 2. I see the same urlparse error in my Debian Testing VM. |
@monsta: I have found an alternate possible definition for
Can you please test on Debian ? If it works, I propose to integrate it in my next PR. Thanks in advance. |
|
Using the older Debian patch and a python 2.7 build, caja-admin worked. |
|
@lukefromdc: thanks for your testing and reporting. I'll then not change the |
|
When things come up that are Debian specific I try to get to them |
I note it for eventual future test requests :-) |
|
@lukefromdc: you can find a Python 3 version of caja-admin at https://github.com/monnerat/caja-admin/tree/python3 |
|
PR #39 replaces this one, thus closing. |
|
Yeah, that change didn't work in my Debian as well... |
Thanks for the test and info. As established last night with lukefromdc, I've dropped this idea for PR #39. |
This allows specifying a Python interpreter without using the obscure trick
of externally defining PYTHON in environment.
The configure report now also displays the effective Python version.
Ref: #34 (comment)