Bug 438587 - Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine
Summary: Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: David Woodhouse
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-03-22 14:26 UTC by Thomas Sailer
Modified: 2008-12-11 08:00 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-11-19 14:52:58 UTC
Type: ---
Embargoed:
dwmw2: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Thomas Sailer 2008-03-22 14:26:33 UTC
Spec URL: http://sailer.fedorapeople.org/libsqlite3x.spec
SRPM URL: http://sailer.fedorapeople.org/libsqlite3x-20071018-2.fc8.src.rpm
Description: libsqlite3x provides thin C++ wrappers for sqlite3.

Comment 1 Jason Tibbitts 2008-05-11 04:17:46 UTC
This failed to build for me in F9/rawhide:

/bin/sh ./libtool --tag=CXX   --mode=compile g++ -DHAVE_CONFIG_H -I.     -O2 -g
-pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector
--param=ssp-buffer-size=4 -m64 -mtune=generic -O2 -Wall -ansi -MT sq3.lo -MD -MP
-MF .deps/sq3.Tpo -c -o sq3.lo sq3.cpp
 g++ -DHAVE_CONFIG_H -I. -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
-fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -O2 -Wall -ansi
-MT sq3.lo -MD -MP -MF .deps/sq3.Tpo -c sq3.cpp  -fPIC -DPIC -o .libs/sq3.o
sq3.cpp: In member function 'int sq3::statement::prepare(const char*, int)':
sq3.cpp:70: error: 'strlen' was not declared in this scope
sq3.cpp: In member function 'int sq3::statement::bind(int, const char*, int)':
sq3.cpp:190: error: 'strlen' was not declared in this scope
sq3.cpp: In member function 'int sq3::statement::bind(const char*, const char*,
int)':
sq3.cpp:226: error: 'strlen' was not declared in this scope


Comment 2 Thomas Sailer 2008-05-16 22:21:22 UTC
Thanks. Apparently, the C++ headers have gotten stricter, so I've added the
necessary includes.

Spec URL: http://sailer.fedorapeople.org/libsqlite3x.spec
SRPM URL: http://sailer.fedorapeople.org/libsqlite3x-20071018-3.fc9.src.rpm


Comment 3 Jason Tibbitts 2008-07-03 04:28:34 UTC
I'm kind of curious as to why you need to drop in an autotools system.  Is the
package simply unbuildable without it?  Has that work been sent to upstream?


Comment 4 Jason Tibbitts 2008-08-21 19:29:24 UTC
Any response to comment #3?

Comment 5 Thomas Sailer 2008-08-21 21:43:33 UTC
Yes, I sent it upstream; but the author will unlikely integrate this, as he is also the author toc.

I tried half a day to get toc working, without success. Even if I got it to work, it would unlikely have worked and picked the correct compiler flags etc. for any architecture I don't own (i.e. everything but x86). So it seemed far less work to just drop in autotools scripts. And autotool is working and maintained for pretty much all architectures fedora supports, so it seems to me that with autotools build scripts, I will have a better chance of getting the packages to compile and run on other architectures as well.

Tom

Comment 6 David Woodhouse 2008-11-02 12:04:44 UTC
"Here you can find "sqlite3x" and "sq3", two different C++ wrapper APIs
for working with sqlite3 databases. These two independent APIs are different in nature but are distributed together because they are maintained together."

We should probably put them into separate binary subpackages.

I agree with ditching pointless stupid build systems when they're too complex to get them to work. But normally I use that as a reason for ditching autocrap and just writing proper Makefiles. I would advocate that in this case, rather than replacing one crap overengineered build system with another. But not a review blocker.

Would be good to have a licence file, but not mandatory since upstream doesn't have it.

Approved -- but please consider splitting into separate packages for sqlite3x vs. sq3 before doing the first build.

Comment 7 David Woodhouse 2008-11-02 12:43:43 UTC
Oops, missed a requirement -- you must add Requires: pkgconfig in the -devel subpackage(s) because they provide .pc files.

Comment 8 Thomas Sailer 2008-11-04 09:43:30 UTC
New Package CVS Request
=======================
Package Name: libsqlite3x
Short Description: C++ Wrapper for the SQLite3 database engine
Owners: sailer
Branches: F-9
InitialCC:

Comment 9 Thomas Sailer 2008-11-04 13:57:56 UTC
I thought about splitting up sqlite3x and sq3 into different packages, but then decided it's not worth it because both libraries are very small. But if you think differently, fine with me, here is the split version:

Spec URL: http://sailer.fedorapeople.org/libsqlite3x.spec
SRPM URL: http://sailer.fedorapeople.org/libsqlite3x-20071018-4.fc10.src.rpm

Also fixed the pkgconfig dependency.

Comment 10 Kevin Fenzi 2008-11-05 22:27:38 UTC
cvs done.

Comment 11 Fedora Update System 2008-11-06 13:11:14 UTC
libsqlite3x-20071018-4.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/libsqlite3x-20071018-4.fc9

Comment 12 Fedora Update System 2008-11-06 13:13:28 UTC
libsqlite3x-20071018-4.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/libsqlite3x-20071018-4.fc10

Comment 13 Fedora Update System 2008-11-07 02:53:46 UTC
libsqlite3x-20071018-4.fc9 has been pushed to the Fedora 9 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update libsqlite3x'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2008-9435

Comment 14 Fedora Update System 2008-11-19 14:52:54 UTC
libsqlite3x-20071018-4.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 15 Fedora Update System 2008-12-11 08:00:26 UTC
libsqlite3x-20071018-4.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.