Re: Advice on packaging SWIG generated python bindings and more

2013-06-07 Thread Srećko Jurić-Kavelj
Hi Anton, mentors,

On Mon, May 13, 2013 at 8:16 PM, Anton Gladky  wrote:
> 1) Please, add "--parallel" to dh to get a faster building.

Done.

> 2) Is it difficult to add some kind of auto-test? Better to add one more
> target into makefile (you patch it anyway), so it can be picked up by
> dh_autotest.

I've skimmed the tests and most of them need a robot (microcontroller)
connected to the computer (running libaria, over serial connection).
There are some tests of library functionality, but it seems I'd have
to patch them also, before creating test target. E.g. angleTest.cpp
reports the test result on stdout, and has the zero exit status
regardless of the test result.

> 3) Do you plan to add some more binaries (java-libaria or something
> like that)? After adding a binary to an existing package, it should go
> to new queue. So if you are planning to add something else, please do it
> now not to bother ftp-master to look at your package after each upload.

Python and Java bindings were made with SWIG 1.3. Python bindings
build with SWIG 2.0, but Java bindings do not. :(

> When you are ready, no need to upload a package to mentors. I can take
> it from VCS.

I've pushed the latest version on github
(https://github.com/jksrecko/libaria), together with some other
branches (squeeze-backport and libaria2-java). I might have a look at
tests next week, so there is no hurry to upload the package.

PS About the backported version, I had to use some backported packages
for building (debhelper, lintian, hardening-includes, dpkg-dev,
libdpkg-perl). As I understand, I'm to provide the resulting debs so
this is not a problem?

Best,
JKS


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/CAACrLC2o8vyyMPa=y0ajdp8kfur7beqeqn0-wq9pnmhw3v1...@mail.gmail.com



Re: Advice on packaging SWIG generated python bindings and more

2013-05-13 Thread Anton Gladky
Hi Srećko,

the package looks ok for me. Just a couple of notes:
1) Please, add "--parallel" to dh to get a faster building.
2) Is it difficult to add some kind of auto-test? Better to add one more
target into makefile (you patch it anyway), so it can be picked up by
dh_autotest.
3) Do you plan to add some more binaries (java-libaria or something
like that)? After adding a binary to an existing package, it should go
to new queue. So if you are planning to add something else, please do it
now not to bother ftp-master to look at your package after each upload.

When you are ready, no need to upload a package to mentors. I can take
it from VCS.

Cheers,

Anton

On 05/08/2013 11:32 AM, Srećko Jurić-Kavelj wrote:
> Dear mentors, Anton,
> 
> I've recently updated my package "libaria"
> 
>  * Package name: libaria
>Version : 2.7.5.2-3
>Upstream Maintainer : Reed Hedges 
>  * URL : http://robots.mobilerobots.com/wiki/ARIA
>  * License : GPL-2
>Section : libs
> 
>   It builds those binary packages:
> 
>   libaria-demo - C++ library for MobileRobots/ActivMedia robots (demo example)
>   libaria-dev - C++ library for MobileRobots/ActivMedia robots (devel)
>   libaria-dev-doc - C++ library for MobileRobots/ActivMedia robots (devel 
> docs)
>   libaria2 - C++ library for MobileRobots/ActivMedia robots
>   python-libaria - C++ library for MobileRobots/ActivMedia robots
> (Python bindings)
> 
>   To access further information about this package, please visit the following
>   URL:
> 
>   http://mentors.debian.net/package/libaria
> 
> 
>   Alternatively, one can download the package with dget using this command:
> 
> dget -x 
> http://mentors.debian.net/debian/pool/main/liba/libaria/libaria_2.7.5.2-3.dsc
> 
>   Changes since the last upload:
> 
>   * Removed statically linked library.
>   * Corrected Multi-Arch config for the doc package.
>   * Building python-libaria package (SWIG generated bindings)
>   * Building libaria-demo package
> 
> I've named the python package python-libaria, but the packaged module
> is named AriaPy. Debian Python Policy states that the package name
> should be the module name prefixed by python-, but on the other hand,
> package names should be lower case (Debian Policy). Compromise would
> be python-ariapy, but it seems redundant.
> 
> Also, I've added only the default python version to Build-Depends.
> Hope this is OK.
> 
> Furthermore, I've packaged an example binary using the library in a
> separate package (because of the Multi-Arch support). I've renamed it
> from demo to aria-demo.
> 
> And last, libaria-dev-doc packages CPP examples for using the library.
> I've also provided a Makefile there, that builds those examples in
> /tmp (default).
> 
> PS I've tried to build the package in squeeze environment (for
> backporting), and while it did succeed after adding backports
> repository (for debhelper, lintian, ...), python package wasn't
> correctly generated (uses dh_python2). Also, lintian complained about
> Hardening, which should be handled by the backported debhelper 9,
> right? Anyway, even if I do succeed in generating the package for
> squeeze, would it be of any good (since I've locally added backports
> to my squeeze environment).
> 
> Best regards,
> 
> Srećko Jurić-Kavelj, dipl.ing. (Ms.E.E)
> Research and Teaching Assistant at University of Zagreb
> (Faculty of Electrical Engineering and Computing, Department of
> Control and Computer Engineering)
> 
> Phone: +385 (0)1 6129 529
> Fax: +385 (0)1 6129 809
> E-mail: srecko.juric-kav...@fer.hr
> URL: http://www.fer.hr/srecko.juric-kavelj
> 
> Sanctus Hieronymus: "Parce mihi, Domine, quia dalmata sum!"
> 




signature.asc
Description: OpenPGP digital signature


Re: Advice on packaging SWIG generated python bindings and more

2013-05-10 Thread Jakub Wilk

* Srećko Jurić-Kavelj , 2013-05-08, 11:32:
I've named the python package python-libaria, but the packaged module 
is named AriaPy. Debian Python Policy states that the package name 
should be the module name prefixed by python-, but on the other hand, 
package names should be lower case (Debian Policy). Compromise would be 
python-ariapy,


python-ariapy is indeed the correct package name.


but it seems redundant.


Sure, but that's only because "Py" in the module name is redundant.

Also, I've added only the default python version to Build-Depends. Hope 
this is OK.


Building for all supported versions would be better, but now that 2.6 is 
no longer supported (and there hopefully won't be 2.8), it doesn't 
really matter.


PS I've tried to build the package in squeeze environment (for 
backporting), and while it did succeed after adding backports 
repository (for debhelper, lintian, ...), python package wasn't 
correctly generated (uses dh_python2).


Support for .pyinstall files was added in python 2.6.6-6.

--
Jakub Wilk


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20130510140709.gb4...@jwilk.net



Advice on packaging SWIG generated python bindings and more

2013-05-08 Thread Srećko Jurić-Kavelj
Dear mentors, Anton,

I've recently updated my package "libaria"

 * Package name: libaria
   Version : 2.7.5.2-3
   Upstream Maintainer : Reed Hedges 
 * URL : http://robots.mobilerobots.com/wiki/ARIA
 * License : GPL-2
   Section : libs

  It builds those binary packages:

  libaria-demo - C++ library for MobileRobots/ActivMedia robots (demo example)
  libaria-dev - C++ library for MobileRobots/ActivMedia robots (devel)
  libaria-dev-doc - C++ library for MobileRobots/ActivMedia robots (devel docs)
  libaria2 - C++ library for MobileRobots/ActivMedia robots
  python-libaria - C++ library for MobileRobots/ActivMedia robots
(Python bindings)

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

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


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

dget -x 
http://mentors.debian.net/debian/pool/main/liba/libaria/libaria_2.7.5.2-3.dsc

  Changes since the last upload:

  * Removed statically linked library.
  * Corrected Multi-Arch config for the doc package.
  * Building python-libaria package (SWIG generated bindings)
  * Building libaria-demo package

I've named the python package python-libaria, but the packaged module
is named AriaPy. Debian Python Policy states that the package name
should be the module name prefixed by python-, but on the other hand,
package names should be lower case (Debian Policy). Compromise would
be python-ariapy, but it seems redundant.

Also, I've added only the default python version to Build-Depends.
Hope this is OK.

Furthermore, I've packaged an example binary using the library in a
separate package (because of the Multi-Arch support). I've renamed it
from demo to aria-demo.

And last, libaria-dev-doc packages CPP examples for using the library.
I've also provided a Makefile there, that builds those examples in
/tmp (default).

PS I've tried to build the package in squeeze environment (for
backporting), and while it did succeed after adding backports
repository (for debhelper, lintian, ...), python package wasn't
correctly generated (uses dh_python2). Also, lintian complained about
Hardening, which should be handled by the backported debhelper 9,
right? Anyway, even if I do succeed in generating the package for
squeeze, would it be of any good (since I've locally added backports
to my squeeze environment).

Best regards,

Srećko Jurić-Kavelj, dipl.ing. (Ms.E.E)
Research and Teaching Assistant at University of Zagreb
(Faculty of Electrical Engineering and Computing, Department of
Control and Computer Engineering)

Phone: +385 (0)1 6129 529
Fax: +385 (0)1 6129 809
E-mail: srecko.juric-kav...@fer.hr
URL: http://www.fer.hr/srecko.juric-kavelj

Sanctus Hieronymus: "Parce mihi, Domine, quia dalmata sum!"


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/caacrlc0bqanfghc0gp+-e6az_wf_75cwwuugejd-frtbu_t...@mail.gmail.com