Bug 2134832

I'll start a dialog with the upstream developers.  I'm not sure what the best way to fix this would be.

(In reply to Steven A. Falco from comment #1)
> I'll start a dialog with the upstream developers.  I'm not sure what the
> best way to fix this would be.

Steven,

Looking at `pcbnew/CMakeLists.txt` I would suggest to add `-DPYTHON_DEST=${python3_sitearch}` or even `-DPYTHON_DEST=%{buildroot}%{python3_sitearch}` to cmake invocation (I am not sure would cmake add the `%{buildroot}` on its own to `PYTHON_DEST` in `install section`. However, `rpmlint` would complain if you use `%{buildroot}` in `build section`)

Also, you could replace `%{buildroot}%{_prefix}/lib/python%{python3_version}/site-packages/` with %{buildroot}%{python3_sitearch} in further references to `_pcbnew.so` 

Sergey

Sergey,

Thanks very much for the suggestion.  I see that PYTHON_DEST is set from PYTHON_SITE_PACKAGE_PATH.  PYTHON_SITE_PACKAGE_PATH comes from the top level CMakeLists.txt:

execute_process( COMMAND ${PYTHON_EXECUTABLE} -c "import distutils.sysconfig;print(\"%s\"%distutils.sysconfig.get_python_lib(plat_specific=0, standard_lib=0, prefix=''))"

But distutils.sysconfig is deprecated, so it may be that the top level CMakeLists.txt should be changed to something like:

execute_process( COMMAND ${PYTHON_EXECUTABLE} -c "from sysconfig import get_path; print(get_path(\"stdlib\"))"

That would set the PYTHON_SITE_PACKAGE_PATH to /usr/lib64/python3.10, which might fix the problem.

But I'm not at all a python expert, and that's why I want to get upstream to look at it.  I think they are currently traveling, so it will take a few days to get an answer from them.

FEDORA-2022-4f27b4267e has been submitted as an update to Fedora 36. https://bodhi.fedoraproject.org/updates/FEDORA-2022-4f27b4267e

FEDORA-2022-f140db8af1 has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2022-f140db8af1

FEDORA-2022-6051239c6d has been submitted as an update to Fedora 37. https://bodhi.fedoraproject.org/updates/FEDORA-2022-6051239c6d

I created a patch for this and made the builds.  Please test.  I'll leave the bug open for now.

(In reply to Steven A. Falco from comment #7)
> I created a patch for this and made the builds.  Please test.  I'll leave
> the bug open for now.

Yes, this solves the problem. However, this works just fine as well (tested):

     -DKICAD_VERSION_EXTRA=%{release} \
     -DKICAD_SHOW_GTK_MESSAGES=ON \
-    -DCMAKE_BUILD_TYPE=RelWithDebInfo
+    -DCMAKE_BUILD_TYPE=RelWithDebInfo \
+    -DPYTHON_SITE_PACKAGE_PATH=%{python3_sitearch}
 %cmake_build
 

Regards,
Sergey

Thanks.  We'll see what upstream wants to do.  If they don't like my patch I can always revert it and make the change you suggested.

FEDORA-2022-6051239c6d has been pushed to the Fedora 37 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-6051239c6d`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-6051239c6d

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

FEDORA-2022-4f27b4267e has been pushed to the Fedora 36 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-4f27b4267e`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-4f27b4267e

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

FEDORA-2022-f140db8af1 has been pushed to the Fedora 35 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-f140db8af1`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-f140db8af1

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Given the fact that distutils.sysconfig is deprecated and will be removed from python 3.12, it makes sense to drop my approach and go with the one Sergey came up with.

Builds are in progress.

Thanks Sergey!

FEDORA-2022-fc8c504abe has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2022-fc8c504abe

FEDORA-2022-f231ce034c has been submitted as an update to Fedora 36. https://bodhi.fedoraproject.org/updates/FEDORA-2022-f231ce034c

FEDORA-2022-54e7ae3712 has been submitted as an update to Fedora 37. https://bodhi.fedoraproject.org/updates/FEDORA-2022-54e7ae3712

FEDORA-2022-54e7ae3712 has been pushed to the Fedora 37 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-54e7ae3712`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-54e7ae3712

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

FEDORA-2022-f231ce034c has been pushed to the Fedora 36 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-f231ce034c`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-f231ce034c

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

FEDORA-2022-fc8c504abe has been pushed to the Fedora 35 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-fc8c504abe`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-fc8c504abe

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

FEDORA-2022-f231ce034c has been pushed to the Fedora 36 stable repository.
If problem still persists, please make note of it in this bug report.

FEDORA-2022-fc8c504abe has been pushed to the Fedora 35 stable repository.
If problem still persists, please make note of it in this bug report.

FEDORA-2022-5d8d131a29 has been pushed to the Fedora 37 stable repository.
If problem still persists, please make note of it in this bug report.