https://bugzilla.redhat.com/show_bug.cgi?id=1781251
Jerry James <loganjerry(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |iarnell(a)gmail.com,
| |jplesnik(a)redhat.com,
| |perl-devel(a)lists.fedoraproj
| |ect.org
Component|flamegraph |perl-Devel-NYTProf
Version|31 |rawhide
Assignee|loganjerry(a)gmail.com |jplesnik(a)redhat.com
Summary|flamegraph uses |Unbundle flamegraph from
|/usr/bin/flamegraph.pl |perl-Devel-NYTProf
|which conflicts with older |
|package perl-Devel-NYTProf, |
|and should be renamed |
Doc Type|--- |If docs needed, set a value
--- Comment #1 from Jerry James <loganjerry(a)gmail.com> ---
Actually, they are not different. The flamegraph.pl in perl-Devel-NYTProf is a
bundled copy of Brendan Gregg's flamegraph, which is now shipped in Fedora in
the flamegraph package. The correct fix to this issue is to unbundle
flamegraph from perl-Devel-NYTProf. It looks like it should be easy. Simply
remove %{buildroot}%{_bindir}/flamegraph.pl in %install, and add "Requires:
flamegraph" to the spec file.
The version in perl-Devel-NYTProf is a little older than the one in the
flamegraph package. I have looked at the differences. The version in the
flamegraph package has several bug fixes (mostly regarding Unicode characters
and SVG images) that are not fixed in the version in perl-Devel-NYTProf. There
are a couple of backwards incompatible changes, but grep did not find any uses
of them in perl-Devel-NYTProf, so I believe that unbundling is safe.
--
You are receiving this mail because:
You are on the CC list for the bug.