← Back to team overview

ubuntu-x-swat team mailing list archive

[Bug 810217] Re: [MIR] wayland protocol package

 

* debian/copyright:
- wayland/scanner.c: GPL (v2 or later) 
   -> this one is indeed listed but there is no COPYING or LICENSE file of GPLv2 upstream, which should be mandatory to release the upstream package (ok, they never released a tarball yet, but it's not too late to fix that!)
- wayland/wayland-hash.c: MIT/X11
   -> this is listed as well in debian/copyright, but the Licence: should be set to "MIT"
- for the remaining pieces, I don't know what to put for empty Licence: I would just either skip the section or put "unkown" or "public domain"

* debian/rules:
more a question than anything else, I'm surprised about this:
# Install in debian/tmp to retain control through dh_install:
override_dh_auto_install:
        dh_auto_install --destdir=debian/tmp
is it really needed? Is it the fact to change the builddir which affects the destdir?

* debian/wayland.lintian-overrides:
I guess this override should be on libwayland0 to have some effect (it doesn't right now)

* debian/libwayland-dev.install
# Tool to build various other packages:
usr/bin/wayland-scanner

wayland-scanner.mk seems to indicate (and confirmed by RAOF) that the scanner is used to generate the header files from the xml protocol.
Consequently, I'm wondering why we need to ship it in the -dev package, as we don't ship the xml file for the protocole but directly the generated header?


All the rest of the packaging is really nice, usage of --fail-missing, ensuring some control over the instability of the ABI, autogenerated symbol files and such :)


Please reopen once the copyright is fixed (and ping me on IRC) and those quesstion answered! Thanks a lot : )

** Changed in: wayland (Ubuntu)
       Status: New => Incomplete

-- 
You received this bug notification because you are a member of Ubuntu-X,
which is subscribed to wayland in Ubuntu.
https://bugs.launchpad.net/bugs/810217

Title:
  [MIR] wayland protocol package

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/wayland/+bug/810217/+subscriptions


References