Bug 1004111 - Review Request: libXaw3dXft - An extended version of Xaw3d with support for UTF8
Summary: Review Request: libXaw3dXft - An extended version of Xaw3d with support for UTF8
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Gwyn Ciesla
QA Contact: Fedora Extras Quality Assurance
URL: http://sourceforge.net/projects/sf-xp...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-09-04 01:18 UTC by Paulo Roma Cavalcanti
Modified: 2013-09-21 13:00 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-09-21 13:00:00 UTC
Type: ---
Embargoed:
gwync: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Paulo Roma Cavalcanti 2013-09-04 01:18:27 UTC
Spec URL: http://roma.fedorapeople.org/specs/libXaw3dXft.spec
SRPM URL: http://roma.fedorapeople.org/srpms/libXaw3dXft-1.6.2b-1.fc18.src.rpm
Description: 

Xaw3dxft is an extended version of Xaw3d, developed as part of xpaint
with support for UTF8 input and UTF8 encoding of text, and rendering
text with the Freetype library and Truetype fonts.

It should be mostly compatible with the original Xaw3d library, except
for font management: everything using the old X11 core font routines
should be replaced by their freetype equivalents.

Fedora Account System Username: roma

Comment 1 Paulo Roma Cavalcanti 2013-09-04 01:23:39 UTC
Xaw3dxft used to be part of xpaint. However, since xpaint 2.9.9,
it supplied as a separate package, which is a new build requirement for updating xpaint to its latest release.

Comment 2 Paulo Roma Cavalcanti 2013-09-04 12:23:59 UTC
Koji build for F18:

http://koji.fedoraproject.org/koji/taskinfo?taskID=5893360

Koji build for F20:

http://koji.fedoraproject.org/koji/taskinfo?taskID=5893377

Rpmlint output:

rpmlint libXaw3dXft-1.6.2b-1.fc18.x86_64.rpm
libXaw3dXft.x86_64: W: spelling-error %description -l en_US xpaint -> paint, x paint
libXaw3dXft.x86_64: W: spelling-error %description -l en_US freetype -> free type, free-type, freestyle
libXaw3dXft.x86_64: W: shared-lib-calls-exit /usr/lib64/libXaw3dxft.so.8.0.1 exit.5
1 packages and 0 specfiles checked; 0 errors, 3 warnings.


rpmlint libXaw3dXft-devel-1.6.2b-1.fc18.x86_64.rpm 
libXaw3dXft-devel.x86_64: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Comment 3 Gwyn Ciesla 2013-09-04 13:11:03 UTC
- rpmlint checks return:

See above.  It would be best to fix the exit if possible, but it may not be.

- package meets naming guidelines
- package meets packaging guidelines
- license ( GPLv3+ ) OK, text in %doc, matches source MIT also present, but compiled together, GPLv3+ is correct.
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file
- devel package ok
- no .la files
- post/postun ldconfig ok
- devel requires base package n-v-r 

Pretty straightforward.  APPROVED.

Comment 4 Paulo Roma Cavalcanti 2013-09-07 17:03:21 UTC
Thanks, Jon, for finishing this review.

New Package SCM Request
=======================
Package Name: libXaw3dXft
Short Description: An extended version of Xaw3d with support for UTF8
Owners: roma
Branches: f18 f19 f20 el5 el6
InitialCC: roma

Comment 5 Kevin Fenzi 2013-09-07 17:40:36 UTC
Git done (by process-git-requests).


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