Hi Balint,
I've uploaded 0.4.0-2 with the suggested fixes.
reply inlined below:
On 09/11/2023 16:23, Bálint Réczey wrote:
Hi Colin,
Colin King (gmail) <colin.i.k...@gmail.com> ezt írta (időpont: 2023.
nov. 7., K, 15:18):
Hi Balint,
Thanks for responding with the review. I was waiting for the upstream
project to release a 0.4 with some minor fixes before re-uploading to
mentors.
I've addressed the issues you found as below:
Please see my observations below.
On 22/10/2023 22:38, Bálint Réczey wrote:
Hi Colin,
I've checked the second upload at [1].
As you can see in the Lintian warnings there is a .git directory which
is not ideal for a source package.
I suggest using the most widely used git-buildpackage based workflow
where the gbp command takes care of exporting the source package
without the .git dir from the packaging repository.
I'd be happy to set up a packaging repo for you at
https://salsa.debian.org/debian/libtypec and add you as a maintainer
as described in [2]
I still hold up my offer about setting up a git repo for packaging on
Salsa. That comes with the benefit of automated fixes from Debian
Janitor and I could also comment on changes right where they happened.
Thank you for your kind offer; I definitely think this is a good idea,
please can you set this up for me. Much appreciated!
Other observations regarding the packaging:
* There is debian/install and also there are binary package specific
*.install files which is slightly confusing.
I suggest dropping debian/install.
Fixed
* In the debian/*.install files you need to specify only the target
dir, not the target file.
Fixed
In libtypec-dev
/usr/share/pkgconfig/${DEB_HOST_MULTIARCH}/libtypec.pc/libtypec.pc
gets shipped, which is not desired.
Fixed
I think my comment here was misleading, sorry for that.
Shipping *.pc is desired, shipping it in the .../libtypec.pc/ dir as a
result of specifying .../libtypec.pc as the target dir in the .install
file was not desired. It was even patched to have the right content.
Please ship the .pc file in the -dev package.
Fixed
* libtypec.h seems to be the same on all architectures. Does it have
to be shipped in a multiarch include location?
Fixed. Now in /usr/include and in the multiarch include location
* Binary packages in debian/control are not marked as Multi-Arch: same
* Please target experimental. The package needs to pass NEW and to
migrate to testing it will need a new source-only upload anyway.
Fixed.
Please review the 0.4 release upload and let me know if this can be
sponsored further to the changes I made.
* Both libtypec-dev.install and libtypec1.install lists
usr/lib/${DEB_HOST_MULTIARCH} and as a result both packages ship the
*.so symlink and *.so.0.4.0.
Please ship *.so.0.4.0 in the library package and the *.so symlink in
the -dev package only.
Fixed.
* As you switched back to use upstream's 0.4.0 SO version the .symbols
file became wrong not matching the shipped SO version. Please fix
that and also switch to the libtypec0 package name since it needs to
match upstream's major SO version
Fixed.
.
* I'd recommend asking upstream to switch to semantic SO versioning
instead of using the project's version and switching to major version
1 when the API stabilized.
Good idea. Will do when API changes and stabilizes.
Colin
Cheers,
Balint
Kind regards,
Colin
Cheers,
Balint
[1] https://mentors.debian.net/package/libtypec/
[2]
https://wiki.debian.org/Salsa/Doc#Collaborative_Maintenance:_.22Debian.22_group
On Thu, 3 Aug 2023 17:00:58 +0100 "Colin King (gmail)"
<colin.i.k...@gmail.com> wrote:
Hi,
I've uploaded a fixed package that addresses these issues.
Colin
On 18/07/2023 08:50, Adam Borowski wrote:
On Mon, Jul 17, 2023 at 03:29:13PM +0100, Colin King (gmail) wrote:
* Package name : libtypec
Version : 0.3-1
* URL : https://github.com/Rajaram-Regupathy/libtypec
libtypec1 - generic interface for efficient USB-C port management
libtypec-dev - Development files for an interface for USB-C port management
libtypec (0.3-1) unstable; urgency=low
.
* Initial release (Closes: #1023477)
* Add patch 0001-fix-libtypec-so-version.patch to fix .so name version
Hi!
Before doing manual review, let's start with lintian:
E: libtypec1: pkg-config-multi-arch-wrong-dir full text contains architecture
specific dir x86_64-linux-gnu [usr/share/pkgconfig/libtypec.pc]
W: libtypec-dev: empty-binary-package
W: libtypec1: lacks-unversioned-link-to-shared-library example:
usr/lib/x86_64-linux-gnu/libtypec.so
[usr/lib/x86_64-linux-gnu/libtypec.so.0.3.0]
W: libtypec1: link-to-shared-library-in-wrong-package
usr/lib/x86_64-linux-gnu/libtypec.so.0.3.0
[usr/lib/x86_64-linux-gnu/libtypec.so]
Meow!
