(New change proposal) |
(adding tracker bug) |
||
(8 intermediate revisions by 2 users not shown) | |||
Line 1: | Line 1: | ||
= Separate package for dtrace | = Separate package for dtrace from systemtap-sdt-devel <!-- The name of your change proposal --> = | ||
== Summary == | == Summary == | ||
Split <code>/usr/bin/dtrace | Split <code>/usr/bin/dtrace</code> from <code>systemtap-sdt-devel</code> ({{package|systemtap}}) into a separate package to optimize many buildroots by removing unnecessary Python dependencies. | ||
== Owner == | == Owner == | ||
Line 11: | Line 13: | ||
== Current status == | == Current status == | ||
[[Category: | [[Category:ChangeAcceptedF41]] | ||
<!-- When your change proposal page is completed and ready for review and announcement --> | <!-- When your change proposal page is completed and ready for review and announcement --> | ||
<!-- remove Category:ChangePageIncomplete and change it to Category:ChangeReadyForWrangler --> | <!-- remove Category:ChangePageIncomplete and change it to Category:ChangeReadyForWrangler --> | ||
Line 29: | Line 31: | ||
ON_QA -> change is fully code complete | ON_QA -> change is fully code complete | ||
--> | --> | ||
* [Announced] | * [https://lists.fedoraproject.org/archives/list/devel-announce@lists.fedoraproject.org/thread/OW4KAQUBZKPZRLUHLZWA6BIQWO4KAP46/ Announced] | ||
* [ | * [https://discussion.fedoraproject.org/t/f41-change-proposal-separate-package-for-dtrace-from-systemtap-sdt-devel-self-contained/119451 Discussion thread] | ||
* FESCo issue: | * FESCo issue: [https://pagure.io/fesco/issue/3227 #3227] | ||
* Tracker bug: | * Tracker bug: [https://bugzilla.redhat.com/show_bug.cgi?id=2296275 #2296275] | ||
* Release notes tracker: <will be assigned by the Wrangler> | * Release notes tracker: <will be assigned by the Wrangler> | ||
== Detailed Description == | == Detailed Description == | ||
The package <code>systemtap-sdt-devel</code> currently contains header files and the script <code>/usr/bin/dtrace</code>, which is written in Python and uses pycparser. This results in unnecessary Python and pycparser installations for many packages that do not need the script (and many times Python at all), as they only require the header files. | |||
Moreover, some important packages (like perl-devel) require systemtap-sdt-devel which means hundreds of packages have Python installed in their buildroots because of the single script they don't need. | Moreover, some important packages (like perl-devel) require systemtap-sdt-devel which means hundreds of Perl packages have Python installed in their buildroots because of the single script they don't need. | ||
The idea was tested on all packages build-requiring perl-devel but don't build-requiring python-devel directly - 520 in total. And from that number: | The idea was tested on all packages build-requiring perl-devel but don't build-requiring python-devel directly - 520 in total. And from that number: | ||
* 7 failed to build for unrelated reasons | |||
* 3 packages have python3-devel in buildroot (different reasons than systempat-sdt-devel) | |||
81 | * 81 packages have python3-libs but not python3-devel (different reasons than systemtap-sdt-devel) | ||
and finally, the rest - 436 packages - builds fine without the Python script in systemtap-sdt-devel and therefore without Python at all. | and finally, the rest - 436 packages - builds fine without the Python script in systemtap-sdt-devel and therefore without Python at all. | ||
Further testing on packages directly requiring systemtap-sdt-devel identified the following needing the dtrace script: | |||
* glib2 | |||
* sssd | |||
* qemu | |||
* python2.7 | |||
* postgresql15 | |||
* postgresql16 | |||
* perl | |||
* php | |||
* mariadb10.11 | |||
* libvirt | |||
Those will depend on a new package to which we move the script. | |||
== Feedback == | == Feedback == | ||
The | The proposal has been positively received on the [https://lists.fedorahosted.org/archives/list/devel@lists.fedoraproject.org/thread/4HKXN77BMFHRMXC7BU2HXECIKOI7B6CS/#4HKXN77BMFHRMXC7BU2HXECIKOI7B6CS Fedora devel list]. | ||
== Benefit to Fedora == | == Benefit to Fedora == | ||
By splitting the <code>/usr/bin/dtrace</code> script into a separate package, we reduce the buildroot size and improve build times for hundreds of packages that do not require Python. | |||
== Scope == | == Scope == | ||
* Proposal owners: | * Proposal owners: | ||
# Move the script to a new package <code>systemtap-sdt-dtrace</code> and update <code>systemtap-sdt-devel</code> to require this new package for backward compatibility. | |||
# Update affected packages to depend on <code>systemtap-sdt-dtrace</code>. | |||
# Remove the backward-compatible requirement from <code>systemtap-sdt-devel</code>. | |||
* Other developers: Review and merge proposed changes and report bugs. | * Other developers: Review and merge proposed changes, and report any bugs. | ||
* Release engineering: N/A (not needed for this Change) | * Release engineering: N/A (not needed for this Change) | ||
Line 80: | Line 95: | ||
== How To Test == | == How To Test == | ||
Package maintainers can | Package maintainers can proactively prepare their packages after the first step from the plan above is implemented. Failed builds identifying packages requiring changes are available in [https://copr.fedorainfracloud.org/coprs/lbalhar/sdt-devel-no-dtrace/ COPR]. | ||
Maintainers can also build and test their packages with the version of systemtap-sdt-devel from which the script has been removed. | Maintainers can also build and test their packages with the version of systemtap-sdt-devel from which the script has been removed. |
Latest revision as of 12:04, 8 July 2024
Separate package for dtrace from systemtap-sdt-devel
Summary
Split /usr/bin/dtrace
from systemtap-sdt-devel
(
) into a separate package to optimize many buildroots by removing unnecessary Python dependencies.
systemtap
Owner
- Name: Lumír Balhar
- Email: lbalhar@redhat.com
Current status
- Targeted release: Fedora Linux 41
- Last updated: 2024-07-08
- Announced
- Discussion thread
- FESCo issue: #3227
- Tracker bug: #2296275
- Release notes tracker: <will be assigned by the Wrangler>
Detailed Description
The package systemtap-sdt-devel
currently contains header files and the script /usr/bin/dtrace
, which is written in Python and uses pycparser. This results in unnecessary Python and pycparser installations for many packages that do not need the script (and many times Python at all), as they only require the header files.
Moreover, some important packages (like perl-devel) require systemtap-sdt-devel which means hundreds of Perl packages have Python installed in their buildroots because of the single script they don't need.
The idea was tested on all packages build-requiring perl-devel but don't build-requiring python-devel directly - 520 in total. And from that number:
- 7 failed to build for unrelated reasons
- 3 packages have python3-devel in buildroot (different reasons than systempat-sdt-devel)
- 81 packages have python3-libs but not python3-devel (different reasons than systemtap-sdt-devel)
and finally, the rest - 436 packages - builds fine without the Python script in systemtap-sdt-devel and therefore without Python at all.
Further testing on packages directly requiring systemtap-sdt-devel identified the following needing the dtrace script:
- glib2
- sssd
- qemu
- python2.7
- postgresql15
- postgresql16
- perl
- php
- mariadb10.11
- libvirt
Those will depend on a new package to which we move the script.
Feedback
The proposal has been positively received on the Fedora devel list.
Benefit to Fedora
By splitting the /usr/bin/dtrace
script into a separate package, we reduce the buildroot size and improve build times for hundreds of packages that do not require Python.
Scope
- Proposal owners:
- Move the script to a new package
systemtap-sdt-dtrace
and updatesystemtap-sdt-devel
to require this new package for backward compatibility. - Update affected packages to depend on
systemtap-sdt-dtrace
. - Remove the backward-compatible requirement from
systemtap-sdt-devel
.
- Other developers: Review and merge proposed changes, and report any bugs.
- Release engineering: N/A (not needed for this Change)
- Policies and guidelines: N/A (not needed for this Change)
- Trademark approval: N/A (not needed for this Change)
- Alignment with the Fedora Strategy: N/A (not needed for this Change)
Upgrade/compatibility impact
How To Test
Package maintainers can proactively prepare their packages after the first step from the plan above is implemented. Failed builds identifying packages requiring changes are available in COPR.
Maintainers can also build and test their packages with the version of systemtap-sdt-devel from which the script has been removed.
User Experience
Regular distro users shouldn't notice any change.
Dependencies
Contingency Plan
- Contingency mechanism: Change owner will revert the change in
.systemtap
- Contingency deadline: N/A (not a System Wide Change)
- Blocks release? N/A (not a System Wide Change)
Documentation
N/A (not a System Wide Change)