Bug#741258: libphonenumber packaging, libre2, ...
On 24/06/14 18:28, Fredrik Roubert wrote: > On Tue, Jun 24, 2014 at 6:05 PM, Daniel Pocock wrote: > >> I meant debian/changelog > > OK. What version number is supposed to be there? I thought you wrote > that debian/changelog only should be updated when the packaging itself > was updated, and the version number currently there is the version > number of when the packaging files last were updated. How is it > supposed to be done instead? > Please see this thread on the list: https://groups.google.com/forum/#!topic/libphonenumber-discuss/wm0bw6nMEwc What is there now is currently incorrect: 6.1~svn668-1 < 6.1 as 6.1 has already been released. You need to either use 6.1+svn668-1 or 6.2~svn668-1 or maybe 7.0~svn668-1 I will always update it for official package builds, but having a correct value in SVN makes it easier for people installing home-built versions of the package. -- To UNSUBSCRIBE, email to debian-wnpp-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/53a9b5fa.7020...@pocock.pro
Bug#741258: libphonenumber packaging, libre2, ...
On Tue, Jun 24, 2014 at 6:05 PM, Daniel Pocock wrote: > I meant debian/changelog OK. What version number is supposed to be there? I thought you wrote that debian/changelog only should be updated when the packaging itself was updated, and the version number currently there is the version number of when the packaging files last were updated. How is it supposed to be done instead? -- Fredrik Roubert roub...@google.com -- To UNSUBSCRIBE, email to debian-wnpp-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/caplbv_m+mfsuxrjfafox5mqoka2mazwpuu6xywt+afquc+g...@mail.gmail.com
Bug#741258: libphonenumber packaging, libre2, ...
On 24/06/14 14:19, Fredrik Roubert wrote: > On Tue, Jun 24, 2014 at 2:14 PM, Daniel Pocock wrote: > >> - updating the version number in debian/copyright in SVN (see the email >> thread about version numbers) - > > There is currently no version number at all in debian/copyright. What > is supposed to be there? > That is a typo - I meant debian/changelog https://github.com/libphonenumber/libphonenumber/blob/master/debian/changelog#L1 -- To UNSUBSCRIBE, email to debian-wnpp-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/53a9a1d0.3000...@pocock.pro
Bug#741258: libphonenumber packaging, libre2, ...
On Tue, Jun 24, 2014 at 2:14 PM, Daniel Pocock wrote: > - updating the version number in debian/copyright in SVN (see the email > thread about version numbers) - There is currently no version number at all in debian/copyright. What is supposed to be there? > - packaging the JavaScript - If anybody does this, just send the patch to me. -- Fredrik Roubert roub...@google.com -- To UNSUBSCRIBE, email to debian-wnpp-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/caplbv_ohuqbup0jjds_3obeg+wka_xyltr+akqedw3ijmah...@mail.gmail.com
Bug#741258: libphonenumber packaging, libre2, ...
On 24/06/14 13:04, Fredrik Roubert wrote: > On Sat, Jun 14, 2014 at 3:16 PM, Fredrik Roubert wrote: > >> As of r671, these changes have now been committed upstream. > Is there now anything else with which you need my help, in order to > proceed with this? > The package is in the NEW queue: https://ftp-master.debian.org/new.html It can take a few weeks for the FTP masters to approve something in the NEW queue In the meantime, these other things are not on the critical path but are a good idea: - getting libre2 into wheezy-backports - then we can release libphonenumber as a backport too - updating the version number in debian/copyright in SVN (see the email thread about version numbers) - this doesn't impact the official packages because I fix up the version number manually when I build a package to upload. But if you set the version correctly in SVN, it will be more convenient for users who want to build their own packages from SVN. - packaging the JavaScript - I saw the reply about that, but I haven't had time to do anything with that myself. I may come back to that or somebody else may contribute a patch. -- To UNSUBSCRIBE, email to debian-wnpp-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/53a96b9f.9030...@pocock.pro
Bug#741258: libphonenumber packaging, libre2, ...
Hi Daniel (2014.06.02_14:33:45_+0200) > Stefano, we are now at the point where another package needs to link > against re2 - could you please make an upload to unstable or add any > other comments about the situation? I notice the current libphonenumber package in Ubuntu is statically linking to libre2. Please try to avoid that if possible. And use Built-Using, if not possible. This seems relevant https://code.google.com/p/libphonenumber/source/browse/trunk/cpp/CMakeLists.txt#363 but as someone who doesn't really know CMake, it isn't immediately obvious to me why it isn't seeing the shared library. SR -- Stefano Rivera http://tumbleweed.org.za/ H: +27 21 461 1230 C: +27 72 419 8559 -- To UNSUBSCRIBE, email to debian-wnpp-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/20140603075202.gw3...@bach.rivera.co.za
Bug#741258: libphonenumber packaging, libre2, ...
Hi Daniel (2014.06.02_14:33:45_+0200) > Stefano, we are now at the point where another package needs to link > against re2 - could you please make an upload to unstable or add any > other comments about the situation? Done a bit of hacking on it, and I think it's pretty much good to go. Uploading to unstable when it's finished building. > I tried building libre2 manually on wheezy, one of the test cases failed: That's https://code.google.com/p/re2/issues/detail?id=102 which I'd forgotten all about, and have just re-worked for upstream submission. SR -- Stefano Rivera http://tumbleweed.org.za/ H: +27 21 461 1230 C: +27 72 419 8559 -- To UNSUBSCRIBE, email to debian-wnpp-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/20140602220445.gv3...@bach.rivera.co.za
Bug#741258: libphonenumber packaging, libre2, ...
On 02/06/14 12:25, Fredrik Roubert wrote: > On 27/05/14 15:59, tony mancill wrote: > >> * it has a build-dep on libre2-dev, which is currently only in >> experimental > > That's explained here: > > http://alioth.debian.org/scm/loggerhead/collab-maint/re2/trunk/view/head:/debian/README.Debian Stefano, we are now at the point where another package needs to link against re2 - could you please make an upload to unstable or add any other comments about the situation? I tried building libre2 manually on wheezy, one of the test cases failed: obj/test/parse_test FAIL; Output: re2/testing/parse_test.cc:214: Check failed: (string(tests[i].parse)) == (s)Regexp: \s parse: cc{0x9-0xd 0x20} s: cc{0x9-0xa 0xc-0xd 0x20} flag=1676 > > On Thu, May 29, 2014 at 8:43 PM, Daniel Pocock wrote: > >> Fredrik indicated upstream would welcome improvement to the packaging >> files. > > Yes, just send code reviews to me. Or, if they're about the > CMakeLists.txt file, better to Philippe Liard > , who actually knows CMake ... I've spent some time going over this. Here is my diff - you could just take my whole debian/* and dump it on top of yours (notice some files renamed/moved): https://github.com/dpocock/libphonenumber/tree/debian Rather than keeping this in debian/patches, you probably want to apply this directly in SVN, I think this is the fix for the issue Tony noticed with boost system: https://github.com/dpocock/libphonenumber/blob/debian/debian/patches/0001-add-boost-system.patch I also made an upload to mentors so it is easy for people to review visually: https://mentors.debian.net/package/libphonenumber Here are some things that need fixing in SVN: - the content that was in debian/changelog should be in a top-level ChangeLog file called changelog.txt - the debian/changelog file should only be changed when somebody does an upload to Debian, it only needs to mention things that change in the package (e.g. "New upstream version", or "initial release of JavaScript sub-package") - distribution of a source tarball - we can autogenerate source tarballs from the tags in the github mirror: https://github.com/libphonenumber/libphonenumber/releases or could somebody put source tarballs (without any binary JARs inside) in the Google Code download page? http://code.google.com/p/libphonenumber/downloads/list - remove binary JARs from SVN so they don't end up in the autogenerated source tree (see the lintian alerts on the mentors link to see the filenames) - the typos One thing stands out for me - the package numbering, e.g. libphonenumber5, libphonenumber6, ... The SONAME in the file appears correct: $ objdump -p ./cpp/build/libphonenumber.so | grep SONAME SONAME libphonenumber.so.6 If you change the ABI number regularly, then it will be hard to push updates to stable distributions (e.g. Debian stable, Ubuntu LTS, EPEL). Ideally, if updates are just data changes (to recognize new area codes and numbering plan changes) then they would be in some separate package that can be distributed through $(stable)-updates - just like anti-virus patterns, timezones, etc: https://lists.debian.org/debian-volatile-announce/2012/msg0.html The binary ABI and the developer API would stay the same for a given Linux version. I don't want to deter you from adding new features or make it more tedious to support, but would it be possible to distribute the pattern data separately in such a manner? The other problem with the package name is that each time there is a new package name anywhere in the control file (e.g. a libphonenumber7 package) then the next upload to Debian will need to go in the FTP NEW queue for manual approval, that adds a delay of 1-4 weeks. If the package names haven't changed, then uploads to Debian (and propagation to Ubuntu) happen automatically within a few hours. Usually if this happens no more than once per year it is OK. If the version number is really significant, maybe the Java packages should also be named libphonenumber${VERSION}-java? Does anybody care either way? This source file: cpp/src/phonenumbers/test_metadata.cc changes during the build. If it is an auto-generated file, maybe it should not be in SVN at all as it will always be recreated when needed? There are various other files like this (*.cc) that are not in SVN. One thing I haven't look at yet is the JavaScript code - we should make a libjs-phonenumber.deb package containing that code too. I've done a first upload to the NEW queue today, there seems to be quite a backlog there so I thought it would be good to get it in the queue while we continue refining this. -- To UNSUBSCRIBE, email to debian-wnpp-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/538c6f29.3090...@pocock.com.au