Bug 672221 - Review Request: sqlite3-dbf - Converter of XBase / FoxPro tables to SQLite
Summary: Review Request: sqlite3-dbf - Converter of XBase / FoxPro tables to SQLite
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Michel Alexandre Salim
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2011-01-24 13:02 UTC by Pavel Alexeev
Modified: 2012-02-27 03:27 UTC (History)
3 users (show)

Fixed In Version: sqlite3-dbf-2011.01.24-1.el6
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2012-02-21 01:26:32 UTC
Type: ---
michel: fedora-review+
gwync: fedora-cvs+

Attachments (Terms of Use)

Description Pavel Alexeev 2011-01-24 13:02:33 UTC
Spec URL: http://hubbitus.net.ru/rpm/Fedora13/sqlite3-dbf/sqlite3-dbf.spec
SRPM URL: http://hubbitus.net.ru/rpm/Fedora13/sqlite3-dbf/sqlite3-dbf-2011.01.24-1.fc13.src.rpm
SQLiteDBF converts XBase databases, particularly FoxPro tables with  memo files,
into a SQL dump. It has no dependencies other than standard Unix libraries.

SQLiteDBF is designed to be incredibly fast and as efficient as possible.

This use code base of the PgDBF project (http://pgdbf.sourceforge.net/).

P.S. Spec file formatted by tabs with 5 space width ( http://fedoraproject.org/wiki/PavelAlexeev/tabsize ). Please, do not start review if it is a problem for you.

Comment 1 Michel Alexandre Salim 2012-02-04 16:23:25 UTC
Taking the review

Comment 2 Michel Alexandre Salim 2012-02-04 16:42:40 UTC
Almost everything looks good (typo on the comment for BuildRoot: still, not steel). APPROVED. Note that I've tested installing the built package but not actually doing a conversion -- I have no XBase/FoxPro sample tables at hand.

updated URLs:

spec: http://hubbitus.info/rpm/Fedora13/sqlite3-dbf/sqlite3-dbf.spec
SRPM: http://hubbitus.info/rpm/Fedora13/sqlite3-dbf/sqlite3-dbf-2011.01.24-1.fc13.src.rpm

#+TODO: TODO(t) WAIT(w@/!) FAIL(f@) | DONE(d) N/A(n)

* TODO Review [90%]
  - [X] Names [2/2]
    - [X] Package name
    - [X] Spec name
  - [X] Package version [2/2]
    - [X] Version number
    - [X] Release tag
  - [X] Meets [[http://fedoraproject.org/wiki/Packaging/Guidelines][guidelines]]
  - [X] Source files match upstream
    ✗ sha1sum sqlite3-dbf_2011.01.24.tar.gz ../SOURCES/sqlite3-dbf_2011.01.24.tar.gz 
    3081deeff2572579a6ccab765c3c8288f1dbba97  sqlite3-dbf_2011.01.24.tar.gz
    3081deeff2572579a6ccab765c3c8288f1dbba97  ../SOURCES/sqlite3-dbf_2011.01.24.tar.gz
  - [X] [[http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries][No bundled libraries]]
  - [X] License [4/4]
    - [X] License is Fedora-approved
    - [X] No licensing conflict
    - [X] License field accurate
    - [X] License included iff packaged by upstream
  - [X] rpmlint [2/2]
    - [X] on src.rpm
      1 packages and 0 specfiles checked; 0 errors, 0 warnings.
    - [X] on x86_64.rpm
      sqlite3-dbf.x86_64: W: no-manual-page-for-binary sqlite3-dbf
      2 packages and 0 specfiles checked; 0 errors, 1 warnings.
  - [-] Language & locale [1/2]
    - [ ] Spec in US English
      in comment for BuildRoot, s|Steel|Still|
    - [X] Spec legible
  - [X] Build [3/3]
    - [X] Koji results
    - [X] BRs complete
    - [X] Directory ownership
  - [X] Spec inspection [7/7]
    - [X] No duplicate files
    - [X] File permissions
    - [X] Filenames must be UTF-8
    - [X] [RHEL]  %clean section
    - [X] [RHEL 5] %buildroot cleaned on %install
    - [X] Macro usage consistent
    - [X] Documentation [1/1]
      - [X] %doc files are non-essential

Comment 3 Pavel Alexeev 2012-02-05 14:24:46 UTC
Thank you!

I have fixed type.

Thank you very much for the review!

New Package SCM Request
Package Name: sqlite3-dbf
Short Description: Converter of XBase / FoxPro tables to SQLite
Owners: hubbitus
Branches: F-16 F-16 F-17 EL-5 EL-6

Comment 4 Michel Alexandre Salim 2012-02-05 22:40:21 UTC
The branches have been renamed, they should be "f15 f16 el5 el6". I think f17 won't be branched for a few more days.

Comment 5 Gwyn Ciesla 2012-02-06 13:17:03 UTC
Git done (by process-git-requests).

Comment 6 Fedora Update System 2012-02-11 09:40:35 UTC
sqlite3-dbf-2011.01.24-1.el5 has been submitted as an update for Fedora EPEL 5.

Comment 7 Fedora Update System 2012-02-11 09:51:55 UTC
sqlite3-dbf-2011.01.24-1.el6 has been submitted as an update for Fedora EPEL 6.

Comment 8 Fedora Update System 2012-02-11 10:01:04 UTC
sqlite3-dbf-2011.01.24-1.fc16 has been submitted as an update for Fedora 16.

Comment 9 Fedora Update System 2012-02-11 18:56:22 UTC
sqlite3-dbf-2011.01.24-1.el6 has been pushed to the Fedora EPEL 6 testing repository.

Comment 10 Fedora Update System 2012-02-21 01:26:32 UTC
sqlite3-dbf-2011.01.24-1.fc16 has been pushed to the Fedora 16 stable repository.

Comment 11 Fedora Update System 2012-02-27 03:27:40 UTC
sqlite3-dbf-2011.01.24-1.el5 has been pushed to the Fedora EPEL 5 stable repository.

Comment 12 Fedora Update System 2012-02-27 03:27:50 UTC
sqlite3-dbf-2011.01.24-1.el6 has been pushed to the Fedora EPEL 6 stable repository.

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