1. debian/changelog should include (LP: #1664256)

Usually new packages have only one changelog entry (because whatever
happened before getting into Ubuntu isn't as important) like this:

* Initial release (LP: #1664256)

2. debian/control: The package is uninstallable because it depends on 
ubuntukylin-default-settings (>= 1.3.18)
but the latest version in Ubuntu is only 1.3.17.

3. debian/control: Drop ukwm from description because there won't be a
UKWM in Ubuntu, right?

4. I get this lintian error:
E: ukui-session-manager: doc-base-file-syntax-error ukui-session-manager:4

https://lintian.debian.org/tags/doc-base-file-syntax-error.html
http://www.chiark.greenend.org.uk/doc/doc-base/doc-base.html/ch-interface.html

Please remove line 4 from ukui-session-manager.doc-base

5. I will probably comment out the peony Recommends for now. Then once
peony is in Ubuntu, we can add that back and anyone upgrading ukui-
session-manager will automatically get the recommended package
installed.

6. I don't consider this a blocker for zesty, but I wonder whether it's
possible to do this without needing to fork mate-session-manager. Is the
purpose of this package only to enable login to a MATE session with the
environment variables set to UKUI? If so, maybe mate-session-manager
could just be patched to do that.

Please fix #1-4 and probably #5 too.

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

Title:
  [FFe] ukui-session-manager

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntukylin/+bug/1664256/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to