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

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

Martin Gieseking <martin.gieseking@xxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |martin.gieseking@xxxxxx

--- Comment #3 from Martin Gieseking <martin.gieseking@xxxxxx> 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.
_______________________________________________
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]