[Bug 653682] Review Request: jemalloc - General-purpose scalable concurrent malloc(3) implementation

[bugzilla at redhat.com]

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=653682

Martin Gieseking <martin.gieseking at uos.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |martin.gieseking at uos.de

--- Comment #3 from Martin Gieseking <martin.gieseking at uos.de> 2010-11-18 03:59:15 EST ---
Here are some additional notes:

- the Group of the devel package should be Development/Libraries

- I suggest to use the default %description for the devel package:
  The %{name}-devel package contains libraries and header files for
  developing applications that use %{name}.

- the man3 manual page belongs to the devel package as it contains the API
  documentation of the library

- Drop %{_bindir}/pprof from the package since it's part of google-perftools.

- replace %{_libdir}/*so with %{_libdir}/*.so 
  (or even better with %{_libdir}/libjemalloc.so)

- The same files must not be packaged multiple times. So, drop all %doc files
  from the devel package.


(In reply to comment #2)
> jemalloc.x86_64: W: manual-page-warning /usr/share/man/man3/jemalloc.3.gz 65:
> warning: `UR' not defined
> jemalloc.x86_64: W: manual-page-warning /usr/share/man/man3/jemalloc.3.gz 67:
> warning: `UE' not defined

The two groff macros UR and UE are used to create hyperlinks. However, the man
backend currently doesn't evaluate them, i.e. the target URL doesn't show up in
the manpage. But that isn't a problem at all.

-- 
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.