search

colcon / colcon-core

Command line tool to build sets of software packages
http://colcon.readthedocs.io
Apache License 2.0
103 stars 46 forks source link

Add workaround for Fedora's rpm_prefix distribution patch #510

Closed cottsay closed 2 years ago

cottsay commented 2 years ago

Similar to what Debian did when Ubuntu Jammy shipped, Fedora has patched CPython to override the installation directory for Python packages, presumably to avoid collision with the system-packaged Python modules.

Fedora's approach is slightly different, but can be patched around in a similar way.

Here's Fedora's patch: https://src.fedoraproject.org/rpms/python3.10/blob/fdfd6c1d945a381cbc85aa6d60b2369ac3ce7f50/f/00251-change-user-install-location.patch

Similar to #504

codecov[bot] commented 2 years ago

Codecov Report

Merging #510 (5ea0c5f) into master (8b59ff6) will decrease coverage by 0.04%. The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #510      +/-   ##
==========================================
- Coverage   81.70%   81.66%   -0.05%     
==========================================
  Files          60       60              
  Lines        3564     3566       +2     
  Branches      684      685       +1     
==========================================
  Hits         2912     2912              
- Misses        601      602       +1     
- Partials       51       52       +1
Impacted Files Coverage Δ
colcon_core/python_install_path.py 60.00% <0.00%> (-15.00%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 8b59ff6...5ea0c5f. Read the comment docs.