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/bugzilla/show_bug.cgi?id=230394
Summary: Review Request: mysql++ - C++ wrapper for the MySQL C API Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: Fedora@FamilleCollet.com QAContact: fedora-package-review@redhat.com
Spec URL: http://remi.collet.free.fr/rpms/extras/mysql++.spec SRPM URL: http://remi.collet.free.fr/rpms/extras/mysql++-2.2.1-1.fc7.src.rpm Mock Log: http://remi.collet.free.fr/rpms/extras/mysql++-build.log Description: MySQL++ makes working with MySQL server queries as easy as working with STL containers. This package contains only the libraries needed to run MySQL++-based programs. If you are building your own MySQL++-based programs, you also need to install the -devel package.
--- rpmlint output W: mysql++ no-soname /usr/lib/libmysqlpp.so => don't know how to fix this, think it's an upstream issue W: mysql++-debuginfo spurious-executable-perm /usr/src/debug/mysql++-2.2.1/lib/common.h => don't know what to do with this
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: mysql++ - C++ wrapper for the MySQL C API
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230394
------- Additional Comments From bugs.michael@gmx.net 2007-03-01 12:52 EST -------
W: mysql++ no-soname /usr/lib/libmysqlpp.so
You can patch the linker options to set a soname. Even better would be to talk to upstream and ask whether there is any reason why they don't use a versioned soname.
W: mysql++-debuginfo spurious-executable-perm
chmod -x inside the extracted source archive, e.g. in %prep
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: mysql++ - C++ wrapper for the MySQL C API
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230394
------- Additional Comments From Fedora@FamilleCollet.com 2007-03-03 03:39 EST -------
You can patch the linker options to set a soname. Even better would be to talk to upstream and ask whether there is any reason why they don't use a versioned soname.
I've send a message to the "plusplus" ML.
They use "bakefile" to generate the Makefile.in and i found very few informations about this. It seems tag exist for this but i don't find a way to make it working. And bakefile not available in RPM (i've build one for test, but it need a lot of work, latest 0.2.2 doesn't work for mysql++ (need 0.2.0), i don't want to be the maintener for this tools).
So t hink we'll have to keep "no-soname" for a while...
chmod -x inside the extracted source archive, e.g. in %prep
So simple... ;)
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: mysql++ - C++ wrapper for the MySQL C API
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230394
------- Additional Comments From Fedora@FamilleCollet.com 2007-03-19 15:05 EST ------- Work in progress with upstream (Warren Yooung) to have .soname in next release...
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: mysql++ - C++ wrapper for the MySQL C API
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230394
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |medium
------- Additional Comments From Fedora@FamilleCollet.com 2007-04-16 04:48 EST ------- Work with upstream done.
This new version includes a lot of changes - soname support - new documentation layout - upstream specfile near the same than this one
Spec URL: http://remi.collet.free.fr/rpms/extras/mysql++.spec SRPM URL: http://remi.collet.free.fr/rpms/extras/mysql++-2.2.2-1.fc7.src.rpm Mock Log: http://remi.collet.free.fr/rpms/extras/mysql++-build.log
Mock log is devel/x86_64.
Ready for review.
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: mysql++ - C++ wrapper for the MySQL C API
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230394
------- Additional Comments From Fedora@FamilleCollet.com 2007-04-17 13:21 EST ------- Spec URL: http://remi.collet.free.fr/rpms/extras/mysql++.spec SRPM URL: http://remi.collet.free.fr/rpms/extras/mysql++-2.2.3-1.fc7.src.rpm Mock Log: http://remi.collet.free.fr/rpms/extras/mysql++-build.log
%changelog * Tue Apr 17 2007 Remi Collet rpms@FamilleCollet.com 2.2.3-1 - update to 2.2.3, - del doc patch - change BuildRoot - add Requires mysql-devel for mysql++-devel
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: mysql++ - C++ wrapper for the MySQL C API
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230394
lxtnow@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |lxtnow@gmail.com
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: mysql++ - C++ wrapper for the MySQL C API
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230394
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Priority|normal |medium
lxtnow@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review+
------- Additional Comments From lxtnow@gmail.com 2007-04-18 17:48 EST ------- OK - Mock Build on FC-6 and FC-Devel (tested on i386 and i86_64 arch) OK - Package meets naming and packaging guidelines OK - Spec file matches base package name. OK - Spec has consistant macro and subpackages usage. OK - Meets Packaging Guidelines. OK - License is LGPL OK - License field in spec matches OK - License file included in package OK - Spec file is in American English OK - Spec files is legible. OK - Sources SHOULD match upstream md5sum: 2856acf395940aa02a27d2db1bfc0801 mysql++-2.2.3.tar.gz OK - Package has correct buildroot. OK - BuildRequires isn't redundant. OK - Sub-packages is proper. OK - %build and %install stages is correct and work. OK - Package has %defattr and permissions on files is good. OK - Package has a correct %clean section. OK - Package is code or permissible content. OK - Packages and Sub-packages %doc files don't affect runtime. OK - Package and Sub-packages has no duplicate files in %files section. OK - Package and Sub-packages don't own any directories other packages own. OK - Changelog section is correct. OK - rpmlint output are silent on RPMs (including sub-package) and SRPM files. Soname clearly fixed.
OK - Should function as described. OK - Should package latest version
---------- APPROVED ----------
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: mysql++ - C++ wrapper for the MySQL C API
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230394
Fedora@FamilleCollet.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From Fedora@FamilleCollet.com 2007-04-19 00:50 EST ------- New Package CVS Request ======================= Package Name: mysql++ Short Description: C++ wrapper for the MySQL C API Owners: Fedora@FamilleCollet.com Branches: devel FC-6 FC-5 InitialCC:
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: mysql++ - C++ wrapper for the MySQL C API
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230394
wtogami@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
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: mysql++ - C++ wrapper for the MySQL C API
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230394
Fedora@FamilleCollet.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
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=230394
Remi Collet fedora@famillecollet.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
--- Comment #8 from Remi Collet fedora@famillecollet.com 2010-04-03 11:20:53 EDT --- Package Change Request ====================== Package Name: mysql++ New Branches: EL-4 EL-5 Owners: remi
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=230394
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #9 from Kevin Fenzi kevin@tummy.com 2010-04-03 13:33:48 EDT --- cvs done.
package-review@lists.fedoraproject.org