[Bug 439941] New: Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

https://bugzilla.redhat.com/show_bug.cgi?id=439941

Summary: Review Request: javasqlite
           Product: Fedora
           Version: rawhide
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: low
          Priority: low
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail],[e-mail]

Spec URL: http://cdn.verbum.org/javasqlite.spec
SRPM URL: http://cdn.verbum.org/javasqlite-20080315-1.fc9.src.rpm

%description
A Java wrapper including a basic JDBC driver for the 
SQLite 2/3 database engine. It is designed using JNI 
to interface to the SQLite API. That API is wrapped by 
methods in the SQLite.Database class. Since June, 2002, 
it includes a small JDBC driver, which allows the most 
basic things to be carried out using the java.sql package.
Since September, 2004, SQLite3 (minimum 3.0.7) is supported
which is rather experimental.

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]    |[e-mail]
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review?

------- Additional Comments From [e-mail]  2008-04-01 11:19 EST -------
Excellent, I was about to submit this but you beat me to it.  Here's the first
pass of the review.

Many of the following have rationales at
http://fedoraproject.org/wiki/PackagingDrafts/Java, and my locale package at
http://cachalot.mine.nu/8/SRPMS/javasqlite-20080315-1.fc8.src.rpm should have
fixes for most of them.

- Group: Libraries: please change to Development/Libraries or System/Libraries

- BuildRequires: sqlite-devel missing

- Javadocs not built nor packaged

- Requires JNI lib -> jar can't be installed in /usr/share/java, install both to
%{_libdir}/%{name} instead (see eg. jnipath patch in my package)

- Documentation missing, %doc ChangeLog license.terms

- Test suite not run during build

- Should probably use specific -source argument to javac (eg. -source 5.0)
(mostly for gcj)

- Dependency on jre (versioned) missing

- I think the versioned jar filenames are pretty much pointless (see recent
discussion on fedora-devel and fedora-packaging)

- GCJ stuff not needed?

- $RPM_OPT_FLAGS not honored (see my cflags patch)

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

------- Additional Comments From [e-mail]  2008-04-01 11:36 EST -------
Your spec looks a lot more complete; let's just replace this review with yours?

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

------- Additional Comments From [e-mail]  2008-04-01 11:44 EST -------
(Assuming we do that)

* Would be good to at least have a comment about the status of upstreaming the
relevant patches; ideally links to a bug tracker but it doesn't look like
upstream has one =/

That's the only issue I saw going through your spec in a first pass.

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

------- Additional Comments From [e-mail]  2008-04-01 11:54 EST -------
(In reply to comment #2)
> Your spec looks a lot more complete; let's just replace this review with yours?

Works for me.  Would you still like to maintain it or do we switch roles and you
become the reviewer?  Either way is fine with me, you make the call.

(In reply to comment #3)
> Would be good to at least have a comment about the status of upstreaming the
>
> relevant patches; ideally links to a bug tracker but it doesn't look like
> upstream has one =/

Yes, I already have a history of submitting patches to javasqlite upstream, and
will send the cflags and libtool patches there right now (I thought I already
had, but seems I remembered wrong).  The jnipath patch is Fedora specific, no
need to submit it.

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

------- Additional Comments From [e-mail]  2008-04-01 17:22 EST -------
I get this error trying to build in mock:

config.status: creating Makefile
+ make -j2 'JAVAC_FLAGS=-source 5.0' LIBTOOL=/usr/bin/libtool
/usr/bin/libtool --mode=link gcc  -I/usr/include \
            -DHAVE_SQLITE2=0 -DHAVE_SQLITE3=1 \
            -o native/mkconst native/mkconst.c -lsqlite3
make: *** 
No rule to make target `SQLite/DBDump.class', needed by `sqlite.jar'
.  Stop.
make: 
*** Waiting for unfinished jobs....

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]         |[e-mail]

------- Additional Comments From [e-mail]  2008-04-01 17:24 EST -------
For ownership, if you have contacts with upstream it'd probably be better for
you to be the primary maintainer, though I do tend to like a
collaborative/multi-maintainer style approach.

I'll take the review.

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

------- Additional Comments From [e-mail]  2008-04-01 18:28 EST -------
Created an attachment (id=299981)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=299981&action=view)
fix parallel make

This patch makes the Makefile.in work in parallel.

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

------- Additional Comments From [e-mail]  2008-04-01 18:29 EST -------
Also, I needed to change the "-source 5.0" to "-source 5"; this is with
java version "1.6.0"
IcedTea Runtime Environment (build 1.6.0-b06)
IcedTea 64-Bit Server VM (build 1.6.0-b06, mixed mode)

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
               Flag|fedora-review?              |

------- Additional Comments From [e-mail]  2008-04-02 13:55 EST -------
Thanks for those fixes.

http://scop.fedorapeople.org/packages/javasqlite.spec
http://scop.fedorapeople.org/packages/javasqlite-20080401-1.fc9.src.rpm

* Wed Apr  2 2008 Ville Skyttä <ville.skytta at iki.fi> - 20080401-1
- Update to 20080401.
- Patch to install *.so as an unversioned module.
- Patch to fix parallel make (#439941, Colin Walters).
- Build with "-source 5" instead of "-source 5.0" (#439941, Colin Walters).
- Use %%{_jvmdir} instead of %%{_prefix}/lib/jvm.

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-review+

------- Additional Comments From [e-mail]  2008-04-02 15:20 EST -------
Builds fine in mock now, spec looks good according to the guidelines.  Marking
as reviewed.

I'd still like a comment above the patches stating their current upstream status
(and comment the non-upstreamable loadLibrary patch specifically), but that's
not a blocker.

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

------- Additional Comments From [e-mail]  2008-04-02 16:12 EST -------
Thanks.  Will add patch statuses before the first build.

Here's the request for branches I'm interested in.  Please feel free to request
co-maintainership for branches you're interested in in pkgdb once this is in (I
don't know your FAS username and the search doesn't seem to work so I can't do
it right now), or request additional ones in case you're interested in some that
are not listed here.

New Package CVS Request
=======================
Package Name: javasqlite
Short Description: SQLite Java Wrapper/JDBC Driver
Owners: scop
Branches: F-8 EL-5
InitialCC: 
Cvsextras Commits: yes

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

------- Additional Comments From [e-mail]  2008-04-03 16:25 EST -------
cvs done.

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |NEXTRELEASE

------- Additional Comments From [e-mail]  2008-04-03 17:16 EST -------
Rawhide build done, F-8 in progress.
http://koji.fedoraproject.org/koji/buildinfo?buildID=44865

Will look into building for EPEL-5 next week.

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

------- Additional Comments From [e-mail]  2008-04-04 17:29 EST -------
javasqlite-20080401-2.fc8 has been submitted as an update for Fedora 8

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

------- Additional Comments From [e-mail]  2008-04-09 01:22 EST -------
javasqlite-20080401-2.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.

[Bug 439941] Review Request: javasqlite

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasqlite

https://bugzilla.redhat.com/show_bug.cgi?id=439941

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CLOSED                      |CLOSED
         Resolution|NEXTRELEASE                 |CURRENTRELEASE
   Fixed In Version|                            |20080401-2.fc8