Bug 461363 - Review Request: ptlib - is a portable tools library
Summary: Review Request: ptlib - is a portable tools 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: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 460888
TreeView+ depends on / blocked
 
Reported: 2008-09-06 16:26 UTC by Peter Robinson
Modified: 2008-09-11 12:48 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-09-11 12:48:53 UTC
Type: ---
Embargoed:
mclasen: fedora-review+
huzaifas: fedora-cvs+


Attachments (Terms of Use)

Description Peter Robinson 2008-09-06 16:26:23 UTC
SPEC: http://pbrobinson.fedorapeople.org/ptlib.spec
SRPM: http://pbrobinson.fedorapeople.org/ptlib-2.3.1-1.fc9.src.rpm

PTLib (Portable Tools Library) is a moderately large class library that 
has it's genesis many years ago as PWLib (portable Windows Library), a 
method to product applications to run on both Microsoft Windows and Unix 
systems. It has also been ported to other systems such as Mac OSX, VxWorks 
and other embedded systems.

It is supplied mainly to support the OPAL project, but that shouldn't stop
you from using it in whatever project you have in mind if you so desire.

Comment 1 Matthias Clasen 2008-09-10 06:21:36 UTC
Doesn't quite build in mock:

RPM build errors:
    Installed (but unpackaged) file(s) found:
   /usr/lib/ptlib/devices/videoinput/v4l_pwplugin.so

Comment 2 Matthias Clasen 2008-09-10 06:41:59 UTC
After fixing that, rpmlint has some complaints:

[mclasen@localhost ~]$ rpmlint /var/lib/mock/fedora-rawhide-i386/result/*.rpm
ptlib.i386: E: non-standard-executable-perm /usr/lib/ptlib/devices/videoinput/v4l2_pwplugin.so 0555
ptlib.i386: E: non-standard-executable-perm /usr/lib/libpt_linux_x86_r.so.2.3-beta1 0555
ptlib.i386: E: non-standard-executable-perm /usr/lib/ptlib/devices/sound/alsa_pwplugin.so 0555
ptlib.i386: E: non-standard-executable-perm /usr/lib/ptlib/devices/videoinput/v4l_pwplugin.so 0555
ptlib-devel.i386: W: no-documentation
ptlib-devel.i386: E: zero-length /usr/include/ptlib/devplugin.h
4 packages and 0 specfiles checked; 5 errors, 1 warnings.

the warning can be ignored, the rest should be fixed.

Comment 3 Peter Robinson 2008-09-10 09:05:50 UTC
There's a new srpm up here and spec in same place as before
http://pbrobinson.fedorapeople.org/ptlib-2.3.1-2.fc9.src.rpm

The build error was a mistake from playing and the rest reports as resolved in rpmlint.

Comment 4 Matthias Clasen 2008-09-11 03:58:44 UTC
Builds fine now, and rpmlint is silent


package name: ok
spec file name: ok
packaging guidelines: ok
license: ok
license field: ok
license file: ok
spec file language: ok
spec file legibility: ok
upstream sources: ok
buildable: ok
excludearch: n/a
buildrequires: ok
locale handling: n/a
shared libs: ok
relocatable: n/a
directory ownership: ok
duplicate files: ok
permissions: ok
%clean: ok
macro use: consistent
content: permissible
large docs: n/a
%doc content: ok
header files: ok
static libs: ok
pkgconfig files: ok
shared libs: ok
-devel deps: ok
libtool archives: ok
gui apps: n/a
file ownership: ok
%install: ok
utf8 filenames: ok

Approved

Comment 5 Peter Robinson 2008-09-11 08:18:12 UTC
New Package CVS Request
=======================
Package Name: ptlib
Short Description: Portable Tools Library
Owners: pbrobinson veillard
Branches: F-8 F-9
InitialCC: pbrobinson
Cvsextras Commits: yes

Comment 6 Huzaifa S. Sidhpurwala 2008-09-11 08:34:13 UTC
cvs done


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