[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#787578: marked as done (RFS: soci/3.2.3-1 [ITP] -- C++ Database Access Library)

Your message dated Mon, 15 Jun 2015 12:57:07 +0200
with message-id <1434365827.21711.12.camel@debian.org>
and subject line Re: Bug#787578: RFS: soci/3.2.3-1 [ITP] -- C++ Database Access Library
has caused the Debian Bug report #787578,
regarding RFS: soci/3.2.3-1 [ITP] -- C++ Database Access Library
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
787578: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=787578
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: sponsorship-requests
Severity: wishlist

  Dear mentors,

  I am looking for a sponsor for my package "soci"

 * Package name    : soci
   Version         : 3.2.3-1
 * URL             : http://sourceforge.net/projects/soci/
 * License         : Boost 1.0
   Section         : libs

  It builds those binary packages:

 libsoci-core3.2 - C++ Database Access Library (core)
 libsoci-dbg - C++ Database Access Library (debug symbols)
 libsoci-dev - C++ Database Access Library (devel)
 libsoci-doc - C++ Database Access Library (development docs)
 libsoci-firebird3.2 - C++ Database Access Library (Firebird backend)
 libsoci-mysql3.2 - C++ Database Access Library (MySQL backend)
 libsoci-odbc3.2 - C++ Database Access Library (ODBC backend)
 libsoci-postgresql3.2 - C++ Database Access Library (PostgreSQL backend)
 libsoci-sqlite3-3.2 - C++ Database Access Library (SQLite3 backend)

  To access further information about this package, please visit the following
  URL:

  http://mentors.debian.net/package/soci


  Alternatively, one can download the package with dget using this command:

    dget -x http://mentors.debian.net/debian/pool/main/s/soci/soci_3.2.3-1.dsc

  More information about soci can be obtained from
  http://sourceforge.net/projects/soci/

  Changes since the last upload:

  * Reintroduce package to archive (Closes: #752696)
  * New upstream release
  * Remove patches for issues fixed upstream
  * Update to debhelper v9 compatibility
  * Specify source format as 3.0 (quilt)
  * Update packaging to work with new (cmake-based) build scripts
  * Update to support multiarch
  * Update copyright to dep5 format
  * Add VCS fields in d/control
  * Rename packages to match SONAMEs
  * Update to standards version 3.9.6


 Note: This package previously existed in the archive, but was removed several
    years ago due to maintainer request and a mostly-dead upstream.  Upstream
    development has resumed, and my coworkers and I use this library often, 
    so I would like to reintroduce it to the archive and maintain it.

  Regards,
   Bill Blough

--- End Message ---
--- Begin Message --- 
Am Donnerstag, den 04.06.2015, 10:39 -0400 schrieb Bill Blough:
> On Wed, Jun 03, 2015 at 02:41:08PM +0200, Tobias Frost wrote:
> > Quoting Bill Blough <devel@blough.us>:
> > 
> > I suggest that you base your work on my repository. (link to the repository
> > is in the ITP). <snip>
> 
> You raise some good points.  I did some of what you suggest, but didn't go as
> far with it as you did.  I'll spend some time redoing it with your repo as a 
> base.
> 
> > Do you see a posiblity to host the repository at alioth.debian.org?
> 
> Yes, I would be open to that.  In fact, I should probably move my other
> packages there as well.  I'll work on getting that set up.
> 
> > So, thats enough for the short review... I did only check "on paper", I did
> > not to try to build your package..
> 
> Thanks for the feedback.  I appreciate you taking the time to look at it.  As I
> mentioned above, I think I will be switching to using your packaging, so all of
> these items should be taken care of once that change is complete.  However,
> I've made a note of your comments so I can keep them in mind for future 
> packages.
> 
> 
> Best regards,
> Bill

Hi Bill,

I'm just uploading your package, thanks for your contribution!
(On any subsequent sponsoring requests, feel free to ping me
explicitly) 

As it will have to go through NEW, please remove the package from
mentors manually.

Some homework, though:
-> d/rules: you can delete the clean override
-> d/rules: the test will be failing badly on the buildds, so disable
running the tests and check if we could have dep8-style test instead.

-> symbols files should be added (its doable on c++, at least it is
worth a try)

(ran check-all-the-things)
-> there are several spelling errors, maybe send upstream a patch.
-> there are other messages, like usage of unsafe strcpy, memcpy and
others in the log. Maybe check them an encourage upstream to use "safe"
variants where appropiate as defensive programming approach
(I attach the logfile for your convenience)

--
tobi

Attachment: checkallthethings_soci.log.xz
Description: application/xz

Attachment: signature.asc
Description: This is a digitally signed message part


--- End Message ---
Reply to: