[Merge] ~mirespace/ubuntu/+source/openssh:merge-lp2040406-noble into ubuntu/+source/openssh:debian/sid

Nick Rosbrook mp+459366 at code.launchpad.net
Fri Jan 26 15:51:00 UTC 2024


Review: Needs Fixing

I didn't take this as a unilateral change at all, and I agree the MP is an appropriate place to discuss this! I apologize if my review came across the wrong way.

However, the change still doesn't look quite right to me. Looking at your commits, I see changes to d/rules in 1faaf8b6aa and 46ad7bf628. And, the overall debian/rules diff between your branch and pkg/debian/sid shows that we did not keep Debian's rescue-ssh.target change:

nr at six:/t/t/openssh$ git diff pkg/debian/sid mirespace/merge-lp2040406-noble -- debian/rules
diff --git a/debian/rules b/debian/rules
index c921ece2..6b3b6ea5 100755
--- a/debian/rules
+++ b/debian/rules
@@ -189,9 +189,9 @@ override_dh_installinit:
        dh_installinit -R --name ssh
 
 override_dh_installsystemd:
-       dh_installsystemd -popenssh-server ssh.service
-       dh_installsystemd -popenssh-server --no-enable ssh.socket
-       dh_installsystemd -popenssh-server --no-start rescue-ssh.target
+       dh_installsystemd -popenssh-server --no-start ssh.socket
+       dh_installsystemd -popenssh-server --no-enable --no-start ssh.service
+       dh_installsystemd -popenssh-server rescue-ssh.target
 
 debian/openssh-server.sshd.pam: debian/openssh-server.sshd.pam.in
 ifeq ($(DEB_HOST_ARCH_OS),linux)
-- 
https://code.launchpad.net/~mirespace/ubuntu/+source/openssh/+git/openssh/+merge/459366
Your team Ubuntu Sponsors is requested to review the proposed merge of ~mirespace/ubuntu/+source/openssh:merge-lp2040406-noble into ubuntu/+source/openssh:debian/sid.




More information about the Ubuntu-sponsors mailing list