[MIR] qtquickcontrols-opensource-src

[Łukasz Zemczak]

[Availability]

Currently in universe.

[Rationale]

This package became one of the crucial dependencies of ubuntu-ui-toolkit - a main package. The original source of the package comes from Qt upstreams and is actively maintained.

[Dependencies]

All package dependencies are in main and do not cause any mismatches.

[Standards compliance]

Package follows the same standards as all base Qt packages that we have in the archives.

[Maintenance]

Upstream is development is ongoing, while the packages themselves are actively maintained by Debian and our Qt-enablement team in Ubuntu.

[Michael Terry (mterry)]

- Needs team bug subscriber
- There are tests that aren't being run in the tests/ directory. Looks like "tests" isn't in the toplevel SUBDIRS? You may need to run them under "xvfb-run -a" too. Enabling during build, or even better as a dep8 test, should be done.
- qml-module-qtquick-layouts should use "Pre-Depends: ${misc:Pre-Depends}"

[Iain Lane (laney)]

1. ~touch-packages is now subscribed.
2. Done at build-time in ubuntu2. dep8 pending.
3. Not done. What do you think this is going to add? I think it's a valid stance to want to have control over pre dependencies & therefore to omit this. Also the only use of this field I know of is for multiarch-support which isn't required any more.

Please can you get this promoted ASAP if possible? It's blocking desktop image builds.

[Michael Terry (mterry)]

1 & 2: Awesome, thanks.

3: By your "control over pre depends" logic, wouldn't you also be opposed to misc:Depends? It's true that the only thing missing right now is the multiarch-support Pre-Depend. It's not a blocker, but I'm surprised by the opposition.

I'll mark fix-committed once ubuntu2 shows up.

[Timo Jyrinki (timo-jyrinki)]

I don't see the ubuntu2 in queues or in laney's code branches. Did the upload somehow fail?

[Iain Lane (laney)]

Yes, there was an outage. Sorry - I went away straight after uploading it and didn't find out until this morning. It's there now; hopefully will build.

Pre-Depends are kind of sensitive. See some of the wording about them in policy (https://www.debian.org/doc/debian-policy/ch-binary.html#s-dependencies) Since multiarch-support isn't relevant any more, I think we shouldn't require packages to add this.

Thanks for checking! :)

[Michael Terry (mterry)]

Thanks, Laney! Approved.

As for Pre-Depends, I get your caution after seeing how intense the Policy manual is about Pre-Depends. But still... You either trust debhelper or you don't. And that way lies madness.