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 roub...@google.com 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, ...
On Tue, Jun 24, 2014 at 2:14 PM, Daniel Pocock dan...@pocock.pro 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 14:19, Fredrik Roubert wrote: On Tue, Jun 24, 2014 at 2:14 PM, Daniel Pocock dan...@pocock.pro 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 6:05 PM, Daniel Pocock dan...@pocock.pro 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 18:28, Fredrik Roubert wrote: On Tue, Jun 24, 2014 at 6:05 PM, Daniel Pocock dan...@pocock.pro 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, ...
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, ...
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 dan...@pocock.com.au 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
philip.li...@gmail.com, 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
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
