Bug 467391 (mingw32-gdbm) - Review Request: mingw32-gdbm - MinGW port of GNU database routines
Summary: Review Request: mingw32-gdbm - MinGW port of GNU database routines
Keywords:
Status: CLOSED NEXTRELEASE
Alias: mingw32-gdbm
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Tim Lauridsen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 454410
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-10-17 09:35 UTC by Richard W.M. Jones
Modified: 2009-01-21 21:35 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-01-21 21:35:04 UTC
Type: ---
Embargoed:
tim.lauridsen: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Richard W.M. Jones 2008-10-17 09:35:48 UTC
Spec URL: http://hg.et.redhat.com/misc/fedora-mingw--devel/?cmd=manifest;manifest=82a0662b5376fe1a90612ed85628670c902a43d3;path=/gdbm/
SRPM URL: http://www.annexia.org/tmp/mingw/fedora-9/src/SRPMS/mingw32-gdbm-1.8.0-1.fc9.src.rpm
Description: 
MinGW port of GNU database routines

Approved MinGW packaging guidelines are here:
http://fedoraproject.org/wiki/Packaging/MinGW

I sent the MinGW-related patch upstream.  However it looks to
me like upstream for this project is dead.

A few things require this package, like OCaml & Python.

Comment 2 Tim Lauridsen 2009-01-16 09:48:35 UTC
I will take this one

Comment 3 Tim Lauridsen 2009-01-16 09:54:34 UTC
$ rpmlint Download/mingw32-gdbm-1.8.0-1.fc10.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint rpmbuild/RPMS/noarch/mingw32-gdbm-1.8.0-1.fc10.noarch.rpm 
mingw32-gdbm.noarch: W: no-documentation
mingw32-gdbm.noarch: W: devel-file-in-non-devel-package /usr/i686-pc-mingw32/sys-root/mingw/include/gdbm.h
mingw32-gdbm.noarch: E: script-without-shebang /usr/i686-pc-mingw32/sys-root/mingw/lib/libgdbm.la
mingw32-gdbm.noarch: W: devel-file-in-non-devel-package /usr/i686-pc-mingw32/sys-root/mingw/lib/libgdbm.dll.a
mingw32-gdbm.noarch: W: spurious-executable-perm /usr/i686-pc-mingw32/sys-root/mingw/lib/libgdbm.dll.a
mingw32-gdbm.noarch: E: arch-independent-package-contains-binary-or-object /usr/i686-pc-mingw32/sys-root/mingw/lib/libgdbm.dll.a
mingw32-gdbm.noarch: W: non-standard-dir-in-usr i686-pc-mingw32
1 packages and 0 specfiles checked; 2 errors, 5 warnings.

Looks like false positives, because of the cross compiler nature of the package.

Comment 4 Tim Lauridsen 2009-01-16 10:02:55 UTC
MUST:
* package must match the upstream source 
cd543862287f55ad66e62cc9d82cc906  gdbm-1.8.0.tar.gz (upstream)
cd543862287f55ad66e62cc9d82cc906  gdbm-1.8.0.tar.gz (srpm)

* package is named appropriately
    mingw32-*
* it is legal for Fedora to distribute this
* license field matches the actual license.
* license is open source-compatible.
* specfile name matches %{name}
* summary and description fine
* correct buildroot
* %{?dist} is used
* no license text included in package  
* package meets FHS (http://www.pathname.com/fhs/)
* changelog format fine 
* Packager, Vendor ,Distribution tag not used
* License used and not Copyright 
* Summary tag does not end in a period
* specfile is legible
* package successfully compiles and builds on at least x86
* make sure lines are <= 80 characters
* specfile written in American English
* no -doc sub-package necessary
* /sbin/ldconfig used in packages containing libraries.
    no native libs
* GUI app, has .desktop file and uses desktop-file-install to install it.
    not a gui app
* header files goes into -devel sub-package.
    no native header files
* *.so goes into -devel sub-package.
    no *.so files
* macros used appropriately and consistently
* no %makeinstall
* install section must begin with rm -rf $RPM_BUILD_ROOT or %{buildroot}
* The spec file handles locales properly (%find_lang )
    no locales
* split Requires(pre,post) into two separate lines
* package not relocatable
* package contains code
* package owns all directories and files
* no %files duplicates
* %defattrs present 
* %clean present
* %doc files do not affect runtime
* packages is following the MinGW guidelines (http://fedoraproject.org/wiki/Packaging/MinGW)

APPROVED

Comment 5 Richard W.M. Jones 2009-01-16 10:16:34 UTC
New Package CVS Request
=======================
Package Name: mingw32-gdbm
Short Description: MinGW port of GNU database routines
Owners: rjones berrange lfarkas
Branches: EL-5 F-10
InitialCC:

Comment 6 Kevin Fenzi 2009-01-17 03:27:00 UTC
cvs done.

Comment 7 Fedora Update System 2009-01-17 11:52:03 UTC
mingw32-gdbm-1.8.0-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/mingw32-gdbm-1.8.0-1.fc10

Comment 8 Fedora Update System 2009-01-21 21:35:01 UTC
mingw32-gdbm-1.8.0-1.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.


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