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

Re: [DebianGIS] RFS: obdgpslogger [ITP #580176]


I recommend joining the Debian GIS team (in cc:) and requesting
sponsorship there.
They/We maintain many GIS and GPS related packages.

I will have a look to your package later.


On Sun, Aug 22, 2010 at 9:51 PM, Gary Briggs <chunky@icculus.org> wrote:
> On Tue, Aug 17, 2010 at 11:50:57PM +0200, Niels Thykier wrote:
>> > * Package name    : obdgpslogger
>> >   Version         : 0.15-1
>> >   Upstream Author : Gary Briggs <chunky@icculus.org> [myself]
>> > * URL             : http://icculus.org/obdgpslogger/
>> > * License         : GPLv2+
>> >   Section         : science
>> >
>> > It builds these binary packages:
>> > obdgpslogger - Suite of tools to log OBDII and GPS data
>> First off, thanks for your interest in packaging this package for
>> Debian. I have done a little review of your package, though IANADD so I
>> cannot upload your package even if you address all my comments. Also a
>> DD may have additional comments on top of mine.
> I really appreciate your taking the time to look at it.
>> As far as I can tell the package embeds a copy of sqlite3, which is
>> already packaged in Debian (as libsqlite3-dev). Optimally you would not
>> ship this in your tarball at all.
> I've added an option to my build to use the system-wide sqlite3 library
> instead of the included one, and modified the rules file to set that
> option in the debian build.
>> There are several files in the package not under GPL v2; you can find
>> some (possibly all) of them by running:
>>   licensecheck -r  * | grep  -v v2\ or\ later
> I've added the files in question to the copyright file. I explicitly
> disavow copyright on a couple of files, and I added the GPL header to
> the top of the others.
> The only remaining file is a simple shell script that's provided
> only as an example in doc/ in the source package. Do I need to
> explicitly mention it anywhere? It's not distributed in the final
> package. We're in the realm of this GPL FAQ here:
> http://www.gnu.org/licenses/gpl-faq.html#WhatIfWorkIsShort
> I'll upload a new package once I know what to do about that.
>> Their copyright holders and licenses should be listed in
>> debian/copyright. Also "Michael Carpenter" is listed as a copyright
>> holder of a handful of files, but he is not mentioned in d/copyright.
> I've added Michael to the copyright holders list.
>> On a related note, the copyright file refers to the versionless GPL from
>> common-licenses, which implies Debian is relicensing it under GPL-3 or
>> later (lintian finds this with issue --pedantic).
> Fixed.
>> Your package is also missing a watch file (lintian reports this with
>> - -I). Even though you are your own upstream and therefore undeniable
>> aware of new upstream releases it is good practise for future packages -
>> and it would also be nice to have in case you at some point in the
>> future retire either your upstream or your downstream "hat".
> Done.
>> I did not have time to run test it - unfortunately I do not own hardware
>> needed either, so at best I would run the simulator.
> Understandable. The only part that communicates with the hardware is
> obdgpslogger itself. All the other binaries are utilities associated
> with the logfiles created [or the sim].
> To try obdgpslogger attached to the sim, you can do it in one step
> ["obdsim -o"], or multiple steps:
> 1) Run "obdsim" without parameters
> 2) Read the "SimPort name"; on *nix this is a pty.
> 3) Run "obdgpslogger -t -s <simport name>"
> The "-t" flag is "spam stdout"; you'll know if stuff's working because
> you see some numbers appear. The default logfile name is "obdgpslogger.db"
> Once you see it working, you can use obd2csv to create a csv file
> from the logfile just created:
> obd2csv -d obdgpslogger.db -o obdgpslogger.csv
> Thank-you so much for taking the time to look at this,
> Gary (-;
> --
> To UNSUBSCRIBE, email to debian-mentors-REQUEST@lists.debian.org
> with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
> Archive: http://lists.debian.org/20100822195149.GR21615@gamehenge.icculus.org

Reply to: