Bug 713126 - Review Request: harfbuzz - Text shaping library
Summary: Review Request: harfbuzz - Text shaping library
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Matthias Clasen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-06-14 11:42 UTC by Kalev Lember
Modified: 2011-06-17 04:30 UTC (History)
7 users (show)

Fixed In Version: harfbuzz-0.6.0-2.fc16
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-06-16 18:07:27 UTC
Type: ---
mclasen: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Kalev Lember 2011-06-14 11:42:41 UTC
Spec URL: http://kalev.fedorapeople.org/harfbuzz.spec
SRPM URL: http://kalev.fedorapeople.org/harfbuzz-0.6.0-1.fc15.src.rpm
Description:
HarfBuzz is an implementation of the OpenType Layout engine.

Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3130477

Comment 1 Matthias Clasen 2011-06-16 03:32:55 UTC
Builds fine in mock.

rpmlint says:
rpmlint /var/lib/mock/fedora-rawhide-x86_64/root/builddir/build/RPMS/harfbuzz-*
harfbuzz.x86_64: W: no-manual-page-for-binary hb-view
harfbuzz-devel.x86_64: W: no-documentation
3 packages and 0 specfiles checked; 0 errors, 2 warnings.

which is fine.
Formal review to follow.

Comment 2 Matthias Clasen 2011-06-16 03:58:20 UTC
I think hb-view is probably better off in the -devel package ?
Isn't that mostly a debug/demo thing ? I don't think it is interesting for most people who just install the library because pango is (going to) depend on it.

Comment 3 Matthias Clasen 2011-06-16 04:14:54 UTC
Package name: ok
Spec file name: ok
Packaging guidelines: ok
License: ok
License field: ok
License file: ok
Spec file language: ok
Spec file readable: yes
Upstream sources: ok
Buildable: ok
ExcludeArch: ok
BuildRequires: ok
locale handling: ok
ldconfig: ok
system libraries: ok
relocatable: ok
directory ownership: ok
duplicate files: ok
permissions: ok (I just learned that %defattr is no longer required)
macro use: ok
permissable content: ok
large docs: ok
%doc content: ok
headers: ok
static libs: ok
shared libs: ok
devel deps: ok
libtool archives: ok
gui apps: ok
file ownership: ok
utf8 filenames: ok

Looks fine; approved

Comment 4 Kalev Lember 2011-06-16 09:50:36 UTC
(In reply to comment #2)
> I think hb-view is probably better off in the -devel package ?
> Isn't that mostly a debug/demo thing ? I don't think it is interesting for most
> people who just install the library because pango is (going to) depend on it.

Good idea, I'll move it to -devel when importing the new package.

Thanks for the review!

Comment 5 Kalev Lember 2011-06-16 09:52:18 UTC
New Package SCM Request
=======================
Package Name: harfbuzz
Short Description: Text shaping library
Owners: kalev mclasen
Branches:
InitialCC:

Comment 6 Gwyn Ciesla 2011-06-16 12:19:03 UTC
Git done (by process-git-requests).

Comment 7 Kalev Lember 2011-06-16 18:07:27 UTC
Package imported and built; closing the ticket.

Comment 8 Parag Nemade 2011-06-17 04:30:41 UTC
No package depends on this package, so I think its ok to branch for f15 also.
Can this package be built for F15?

Benefit I see, people who generally don't use rawhide but current stable release can also work on harfbuzz.


Note You need to log in before you can comment on or make changes to this bug.