[FFE] [needs packaging ] unity8-lxc

[Christopher Townsend]

The unity8-lxc package provides a way to run the Unity8 Desktop Preview in a Linux Container (LXC) so that users can use the latest and greatest Unity8 Desktop Preview on their systems without the need for developers to SRU Unity8/Mir and related packages on released versions of Ubuntu. For example, Trusty users can install this package and then run the setup script and have a fully functioning Unity8 session based on the latest Unity8 Desktop Preview ISO image.

URL: https://launchpad.net/unity8-preview-lxc
License: GPL-3

[Christopher Townsend (townsend)]

Packages for the project can be found in PPA https://code.launchpad.net/~unity8-desktop-session-team/+archive/ubuntu/unity8-preview-lxc

[Martin Pitt (pitti)]

Subscribing release team for reviewing FFE.

[Martin Pitt (pitti)]

I reviewed the package in the NEW queue:

- Obsolete source package format. Please move debian/source/format to "3.0 (quilt)".
- Maintainer: should be Ubuntu Developers <email address hidden> (supposedly you don't plan to upload this to Debian).
- debian/control: could do with a Homepage: field, if available.
- debian/control: Missing ${python3:Depends}
- debian/copyright: "Upstream-Name: Stéphane Graber"; no, that's the project name, not the maintainer name

Otherwise it looks ok. I reject the package for the above reasons. The release team needs to approve the feature freeze exception anyway in the meantime. I'm happy to review/sponsor an updated package.

Thanks!

[Martin Pitt (pitti)]

> - debian/control: Missing ${python3:Depends}

Sorry, that's bogus, it's not a python3 library and doesn't use pybuild. Ignore that part.

[Stéphane Graber (stgraber)]

Chris, based on the above:
 - debian/source/format should be "3.0 (native)"
 - debian/control: Maintainer should be set to "Ubuntu Developers <email address hidden> "
 - debian/control: Homepage field in the source section should be set to https://launchpad.net/unity8-preview-lxc
 - debian/copyright: Upstream-Name should be set to "Unity8 preview LXC"

[Stéphane Graber (stgraber)]

Oh and since the package is native, the version number shouldn't include a -XubuntuY part.

[Robert Ancell (robert-ancell)]

- Obsolete source package format. Please move debian/source/format to "3.0 (quilt)".

I changed this to 1.0 so it could build with bzr-buildpackage - can you do 3.0 packages using bzr yet?

[Scott Kitterman (kitterman)]

On Wednesday, October 08, 2014 19:35:10 you wrote:
> - Obsolete source package format. Please move debian/source/format to
> "3.0 (quilt)".
>
> I changed this to 1.0 so it could build with bzr-buildpackage - can you
> do 3.0 packages using bzr yet?

I don't think that's a reject reason on it's own in any case.

[Christopher Townsend (townsend)]

After discussing this with Martin, we decided to leave it 1.0 since this is going to be maintained on the CI train.

I've made the other requested changes (haven't pushed yet) except for Stéphane's comment about the -XubuntuY version suffix. I asked him if it's necessary to remove this since this package is going to be maintained in CI train. If anyone else has the correct recommendation about this, please let me know.

[Martin Pitt (pitti)]

Robert Ancell [2014-10-08 19:35 -0000]:
> I changed this to 1.0 so it could build with bzr-buildpackage - can you
> do 3.0 packages using bzr yet?

Sure, that has never been a problem of bzr-buildpackage. Using 1.0 is
a workaround for some CI train limitation AFAIK.

[Martin Pitt (pitti)]

Christopher Townsend [2014-10-08 19:54 -0000]:
> I've made the other requested changes (haven't pushed yet) except for
> Stéphane's comment about the -XubuntuY version suffix. I asked him if
> it's necessary to remove this since this package is going to be
> maintained in CI train.

CI train or not really shouldn't matter. Iff the package has an
orig.tar, it should not be native, and thus have -XubuntY.
(FTR, many/all other CI train packages are the same)

[Christopher Townsend (townsend)]

Ok, I've committed and pushed the requested changes to trunk. I've left in the -XubuntuY suffix since this is really a non-native package.

Thanks!

[Martin Pitt (pitti)]

Reuploaded to utopic's NEW queue. This still requires a feature freeze exception approval from the RT.

[Scott Kitterman (kitterman)]

Since it's a new package not to be installed by default, there's no regression
risk here. Approved contingent on an archive admin agreeing to do the New
review and an appropriate developer agreeing to sign up for package bugs and
monitor them.

[Martin Pitt (pitti)]

NEWed

[Launchpad Janitor (janitor)]

This bug was fixed in the package unity8-lxc - 1.0.0-0ubuntu1

---------------
unity8-lxc (1.0.0-0ubuntu1) utopic; urgency=medium

  * Initial Ubuntu release. (LP: #1372930)
 -- Chris Townsend <email address hidden> Tue, 23 Sep 2014 09:27:37 -0400