[Bug 618451] Review Request: gdb-heap - Extensions to gdb for debugging dynamic memory allocation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=618451

--- Comment #6 from Chen Lei <supercyper1@xxxxxxxxx> 2010-07-27 12:20:54 EDT ---
(In reply to comment #5)
> (In reply to comment #4)

> > I've changed this to a dependency on:
>   Requires: glibc%{_isa} = %{glibc_version}
> though this really ought to read:
>   Requires: glibc-debuginfo%{_isa} = %{glibc_version}
> as that's what it really needs.
> Without the debuginfo package this code not only is useless, but won't load or
> run.
> I don't know if it's possible to rig things up so that this autoloaded whenever
> gdb starts up (CCing tromey for gdb advice), that way we could issue a message.
>   (Though it strikes me that this ought to be handled at the rpm metadata
> level, not as a runtime failure, and that we ought to allow packages that
> require debuginfo to require that debuginfo).

Actually,  Requires: glibc%{_isa} is useless for this package, gdb-heap depends
on python/gdb, python and gdb will pull in glibc as a dependency, I think
adding a README to %doc will be more helpful.

Also, I suggest you to install head modules to %{_datadir}/gdb-heap, arch
independent data should go into /usr/share in FHS. Many other packages(e.g.
ibus) already install their private python modules to %{_datadir}

See
http://www.pathname.com/fhs/pub/fhs-2.3.html#USRSHAREARCHITECTUREINDEPENDENTDATA

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]