Red Hat Bugzilla – Bug 438932
Review Request: libotf - Library for handling OpenType Font
Last modified: 2008-05-25 20:24:16 EDT
Spec URL: http://nbecker.dyndns.org:8080/RPM/libotf.spec
SRPM URL: http://nbecker.dyndns.org:8080/RPM/libotf-0.9.7-1.fc8.src.rpm
Description: The library "libotf" provides the following facilites.
Read Open Type Layout Tables from OTF file. Currently these tables are
supported; head, name, cmap, GDEF, GSUB, and GPOS. Convert a Unicode
character sequence to a glyph code sequence by using the above tables.
The combination of libotf and the FreeType library (Ver.2) realizes
CTL (complex text layout) by OpenType fonts. This library is currently
used by the m17n library. It seems that the probject Free Type Layout
provides the similar (or better) facility as this library, but
currently they have not yet released their library. So, we have
developed this one.
Useful for emacs-22.214.171.124
Mock build OK
1)No need of INSTALL as %doc
2)add Requires: pkgconfig to -devel package
3)Use following to preserve timestamps while installing
make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"
4) rpmlint gave me
libotf.i386: W: devel-file-in-non-devel-package /usr/bin/libotf-config
A development file (usually source code) is located in a non-devel
package. If you want to include source code in your package, be sure to
create a development package.
==> Good if you move file libotf-config to -devel package
I believe all the above have been fixed.
+ package builds in mock.
koji build => http://koji.fedoraproject.org/koji/taskinfo?taskID=535524
+ rpmlint is silent for SRPM and RPM.
+ source files match upstream.
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is small so no need of -doc subpackage.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code.
+ no static libraries.
+ libotf.pc files are present.
+ -devel subpackage exists.
+ no .la files.
+ no translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ ldconfig scriptlets are used.
+ package libotf-0.9.7-2.fc9->
Requires: libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1)
libc.so.6(GLIBC_2.1.3) libc.so.6(GLIBC_2.3.4) libc.so.6(GLIBC_2.4)
libfreetype.so.6 libotf.so.0 rtld(GNU_HASH)
+ package libotf-devel-0.9.7-2.fc9 ->
Requires: /bin/sh libotf.so.0
+ Not a GUI app.
1) change license to LGPLv2+
2) add examples directory to %doc of -devel package.
3) Source URL should be
Please submit updated SRPM here.
Minor: installing example also installs subdir .deps. I recommend ignoring
this because removing it will break the Makefile.
simple way then is to add to -devel
ping for updated SRPM
You mean only install example/*.c and not Makefile? I think the original
(including Makefile and .deps) is better - at least it has then a working
I have not seen such case where .deps also got installed. Just search for .deps
under /usr/share/doc and afaics there is no .deps there.
so if you want .deps also then I will ask you to drop
OK, .deps removed
sorry I thought I posted review here but it seems I need to do that yet. Will
review this tomorrow.
(In reply to comment #11)
> sorry I thought I posted review here but it seems I need to do that yet.
> review this tomorrow.
sorry. I got so busy schedule at home and work also. will review now.
koji build http://koji.fedoraproject.org/koji/taskinfo?taskID=595436
Package looks Ok now.
New Package CVS Request
Package Name: libotf
Short Description: Library for handling OpenType Font
Branches: F-8 F-9
Cvsextras Commits: yes
Neal, has this been built? If so, you can close this bug.