[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-11-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from David Woodhouse <[EMAIL PROTECTED]>  2008-11-02 07:43:43 
EDT ---
Oops, missed a requirement -- you must add Requires: pkgconfig in the -devel
subpackage(s) because they provide .pc files.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-11-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


David Woodhouse <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
   Flag||fedora-review+




--- Comment #6 from David Woodhouse <[EMAIL PROTECTED]>  2008-11-02 07:04:44 
EDT ---
"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.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-11-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


David Woodhouse <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-11-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Thomas Sailer <[EMAIL PROTECTED]>  2008-11-04 08:57:56 EDT 
---
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.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-11-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Thomas Sailer <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag||fedora-cvs?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-11-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Thomas Sailer <[EMAIL PROTECTED]>  2008-11-04 04:43:30 EDT 
---
New Package CVS Request
===
Package Name: libsqlite3x
Short Description: C++ Wrapper for the SQLite3 database engine
Owners: sailer
Branches: F-9
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-11-05 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag|fedora-cvs? |fedora-cvs+




--- Comment #10 from Kevin Fenzi <[EMAIL PROTECTED]>  2008-11-05 17:27:38 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-11-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #11 from Fedora Update System <[EMAIL PROTECTED]>  2008-11-06 
08:11:14 EDT ---
libsqlite3x-20071018-4.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/libsqlite3x-20071018-4.fc9

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-11-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #12 from Fedora Update System <[EMAIL PROTECTED]>  2008-11-06 
08:13:28 EDT ---
libsqlite3x-20071018-4.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/libsqlite3x-20071018-4.fc10

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-11-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #13 from Fedora Update System <[EMAIL PROTECTED]>  2008-11-06 
21:53:46 EDT ---
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

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-11-19 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #14 from Fedora Update System <[EMAIL PROTECTED]>  2008-11-19 
09:52:54 EDT ---
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.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-11-19 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-12-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #15 from Fedora Update System <[EMAIL PROTECTED]>  2008-12-11 
03:00:26 EDT ---
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.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-08-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Jason Tibbitts <[EMAIL PROTECTED]>  2008-08-21 15:29:24 EDT 
---
Any response to comment #3?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-08-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Thomas Sailer <[EMAIL PROTECTED]>  2008-08-21 17:43:33 EDT 
---
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

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-05-10 Thread 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: libsqlite3x - C++ Wrapper for the SQLite3 database 
engine


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-11 00:17 EST ---
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


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-05-16 Thread 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: libsqlite3x - C++ Wrapper for the SQLite3 database 
engine


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-16 18:21 EST ---
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


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438587] Review Request: libsqlite3x - C++ Wrapper for the SQLite3 database engine

2008-07-02 Thread 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: libsqlite3x - C++ Wrapper for the SQLite3 database 
engine


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-03 00:28 EST ---
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?


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review