[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1064109: marked as done (slurm-wlm: install PAM modules into /usr)

Your message dated Sun, 12 May 2024 16:31:46 +0000
with message-id <E1s6C70-0003Ym-Sl@fasolo.debian.org>
and subject line Bug#1064109: fixed in slurm-wlm 23.11.6-1
has caused the Debian Bug report #1064109,
regarding slurm-wlm: install PAM modules into /usr
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
1064109: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1064109
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Source: slurm-wlm
Version: 23.11.3-2
Severity: normal
Tags: patch
User: helmutg@debian.org
Usertags: dep17m2

We want to finalize the /usr-merge via DEP17 by moving all files to
/usr. slurm-wlm installs files into /lib; these should be moved into the
respective canonical locations in /usr/.

Please find a patch attached. It has been build-tested.

Note 1: this change includes moving the .so into a multiarch path which
is recommended on Debian nowadays.

Note 2: this should not be backported to bookworm. If you intend to
backport, please use dh_movetousr instead.

Note 3: this change also drops the libpam dev packages as they appear to
be unnecessary. Instead the .la/.a files are listed as not-installed.

If your package will change for the t64 transition or otherwise
rename/split/move its binaries (packages) during trixie, please
then upload to experimental and get in touch with the UsrMerge
driver, please see the wiki [1].

Michael

[1] https://wiki.debian.org/UsrMerge

diff --git a/debian/changelog b/debian/changelog
index f8262c05..9206ce5d 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,13 @@
+slurm-wlm (23.11.3-2.1) UNRELEASED; urgency=medium
+
+  * Non-maintainer upload.
+  * Drop libpam dev packages.
+    PAM modules are shared libraries that are loaded by the PAM stack. They
+    do not have an public API which would warrant a -dev package.
+  * Install PAM modules into multiarch path in /usr. (Closes: #-1)
+
+ -- Michael Biebl <biebl@debian.org>  Sat, 17 Feb 2024 12:49:38 +0100
+
 slurm-wlm (23.11.3-2) unstable; urgency=medium
 
   * Team upload
diff --git a/debian/control b/debian/control
index 62482dce..f6de02f6 100644
--- a/debian/control
+++ b/debian/control
@@ -654,21 +654,6 @@ Description: PAM module to authenticate using the Slurm resource manager
  restricted to user root and users who have been allocated resources on
  that node.
 
-Package: libpam-slurm-dev
-Architecture: any
-Pre-Depends: ${misc:Pre-Depends}
-Depends:
- ${shlibs:Depends},
- ${misc:Depends}
-Description: Slurm PAM module development files
- The Slurm Workload Manager is an open-source cluster resource management and
- job scheduling system that strives to be simple, scalable, portable,
- fault-tolerant, and interconnect agnostic.
- Pluggable Authentication Module (PAM) for restricting access to compute
- nodes where Slurm performs resource management. Access to the node is
- restricted to user root and users who have been allocated resources on
- This package contains development files for the Slurm pam module
-
 Package: libpam-slurm-adopt
 Architecture: any
 Pre-Depends: ${misc:Pre-Depends}
@@ -685,23 +670,6 @@ Description: PAM module to authenticate users running a Slurm job and track thei
  step of the job so that processes spawned are tracked and Slurm can
  perform a complete cleanup when the job is completed.
 
-Package: libpam-slurm-adopt-dev
-Architecture: any
-Pre-Depends: ${misc:Pre-Depends}
-Depends:
- ${shlibs:Depends},
- ${misc:Depends}
-Description: Slurm adopt PAM module development files
- The Slurm Workload Manager is an open-source cluster resource management and
- job scheduling system that strives to be simple, scalable, portable,
- fault-tolerant, and interconnect agnostic.
- Pluggable Authentication Module (PAM) for restricting access to compute
- nodes where Slurm performs resource management to users who have a
- running job. The user's connection is "adopted" into the "external"
- step of the job so that processes spawned are tracked and Slurm can
- perform a complete cleanup when the job is completed.
- This package contains development files for the Slurm adopt pam module
-
 Package: slurm-wlm-emulator
 Architecture: any
 Depends: ${misc:Depends}, ${shlibs:Depends},
diff --git a/debian/libpam-slurm-adopt-dev.install b/debian/libpam-slurm-adopt-dev.install
deleted file mode 100644
index 25bd6cc4..00000000
--- a/debian/libpam-slurm-adopt-dev.install
+++ /dev/null
@@ -1,2 +0,0 @@
-lib/security/pam_slurm_adopt.a
-lib/security/pam_slurm_adopt.la
diff --git a/debian/libpam-slurm-adopt.install b/debian/libpam-slurm-adopt.install
index ee78e99c..c3f95b7b 100644
--- a/debian/libpam-slurm-adopt.install
+++ b/debian/libpam-slurm-adopt.install
@@ -1,2 +1,2 @@
-lib/security/pam_slurm_adopt.so
+usr/lib/*/security/pam_slurm_adopt.so
 debian/pam-configs/slurm-adopt usr/share/pam-configs/
diff --git a/debian/libpam-slurm-dev.install b/debian/libpam-slurm-dev.install
deleted file mode 100644
index 524f2623..00000000
--- a/debian/libpam-slurm-dev.install
+++ /dev/null
@@ -1,2 +0,0 @@
-lib/security/pam_slurm.a
-lib/security/pam_slurm.la
diff --git a/debian/libpam-slurm.install b/debian/libpam-slurm.install
index 60be2be0..4b51d83c 100644
--- a/debian/libpam-slurm.install
+++ b/debian/libpam-slurm.install
@@ -1,2 +1,2 @@
-lib/security/pam_slurm.so
+usr/lib/*/security/pam_slurm.so
 debian/pam-configs/slurm usr/share/pam-configs/
diff --git a/debian/not-installed b/debian/not-installed
index a43db53c..5d535cf6 100644
--- a/debian/not-installed
+++ b/debian/not-installed
@@ -1,2 +1,4 @@
 debian/tmp/usr/lib/*/slurm-wlm/src/sattach/sattach.wrapper.c
 debian/tmp/usr/lib/*/slurm-wlm/src/srun/srun.wrapper.c
+debian/tmp/usr/lib/*/security/*.a
+debian/tmp/usr/lib/*/security/*.la
diff --git a/debian/rules b/debian/rules
index bdc741ac..a1035f55 100755
--- a/debian/rules
+++ b/debian/rules
@@ -32,7 +32,7 @@ override_dh_auto_clean:
 # hardening-no-fortify-functions
 # Notice that -g in CFLAGS is still provided by dpkg-buildflags
 override_dh_auto_configure:
-	dh_auto_configure -- --sysconfdir=/etc/slurm --with-munge --enable-pam --without-rpath --disable-debug --enable-multiple-slurmd --with-pmix=/usr/lib/$(DEB_HOST_MULTIARCH)/pmix2 $(ENABLEDEPRECATED) SUCMD=/bin/su SLEEP_CMD=/bin/sleep
+	dh_auto_configure -- --sysconfdir=/etc/slurm --with-munge --enable-pam --without-rpath --disable-debug --enable-multiple-slurmd --with-pmix=/usr/lib/$(DEB_HOST_MULTIARCH)/pmix2 --with-pam_dir=/usr/lib/$(DEB_HOST_MULTIARCH)/security $(ENABLEDEPRECATED) SUCMD=/bin/su SLEEP_CMD=/bin/sleep
 	dh_auto_configure --builddirectory build-emulator -- -sysconfdir=/etc/slurm --with-munge --enable-pam --without-rpath --disable-debug --enable-front-end --enable-multiple-slurmd --with-pmix=/usr/lib/$(DEB_HOST_MULTIARCH)/pmix2 $(ENABLEDEPRECATED) SUCMD=/bin/su SLEEP_CMD=/bin/sleep
 override_dh_auto_build:
 	dh_auto_build
@@ -164,8 +164,7 @@ override_dh_strip:
 	dh_strip -plibslurm-dev --no-automatic-dbgsym
 	dh_strip -pslurmrestd --no-automatic-dbgsym
 	strip --verbose --strip-unneeded --remove-section=.comment \
-		--remove-section=.note debian/slurm-*-plugin*-dev/usr/lib/*/slurm-wlm/*.a \
-		debian/libpam-slurm*-dev/lib/security/*.a
+		--remove-section=.note debian/slurm-*-plugin*-dev/usr/lib/*/slurm-wlm/*.a
 
 override_dh_auto_test:

--- End Message ---
--- Begin Message --- 
Source: slurm-wlm
Source-Version: 23.11.6-1
Done: Gennaro Oliva <oliva@debian.org>

We believe that the bug you reported is fixed in the latest version of
slurm-wlm, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 1064109@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Gennaro Oliva <oliva@debian.org> (supplier of updated slurm-wlm package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Thu, 09 May 2024 23:27:28 +0200
Source: slurm-wlm
Architecture: source
Version: 23.11.6-1
Distribution: unstable
Urgency: medium
Maintainer: Debian HPC Team <debian-hpc@lists.debian.org>
Changed-By: Gennaro Oliva <oliva@debian.org>
Closes: 1056122 1059711 1064109 1070898
Changes:
 slurm-wlm (23.11.6-1) unstable; urgency=medium
 .
   [ Gennaro Oliva ]
   * New upstream release
   * Update libslurm symbols file (Closes: #1070898)
   * Send SIGUSR2 to daemon in postrotate
   * Set delaycompress for logrotates (Closes: #1056122)
   * Add libpmix2 dependency for basic plugin (Closes: #1059711)
 .
   [ Benjamin Drung ]
   * Restore /etc/slurm/plugstack.conf in slurm-wlm-basic-plugins.
   Make the build more robust by always generating
   `slurm-wlm-basic-plugins.install` from scratch. Restore
   `/etc/slurm/plugstack.conf` by putting it into a `.install.in` file that
   is copied into the `.install` file in the beginning.
 .
   [ Michael Biebl ]
   * Drop libpam dev packages
   * Install PAM modules into multiarch path in /usr (Closes: #1064109)
Checksums-Sha1:
 610375e72d9a379010d30ebab5b5f74ce2002fc9 5580 slurm-wlm_23.11.6-1.dsc
 e64c49490389bc3c97d95672eed9a3acfc058448 10110647 slurm-wlm_23.11.6.orig.tar.gz
 2ef76a666dd8e015a5fbff30a9836d57dca5a1cf 136712 slurm-wlm_23.11.6-1.debian.tar.xz
 567a122ddba9244fd386a9a438be68bd7c8ec25f 34331 slurm-wlm_23.11.6-1_amd64.buildinfo
Checksums-Sha256:
 658f44bb348ef113b4ea09fe30a59bf469e42dd1cd605b887d24af99ae25cf73 5580 slurm-wlm_23.11.6-1.dsc
 7c19d9655493f72b8ce4583b91183ee04fed369b908e2d07a32d0b8c77d71a60 10110647 slurm-wlm_23.11.6.orig.tar.gz
 cc3ea98793b899e7ac00a52d4e05de9722b88cd031a79ecfa2f524b60f76f1c7 136712 slurm-wlm_23.11.6-1.debian.tar.xz
 b884b5217a506e358ba2da46119f0004f37d7abfb524af271f3ecb09715349a8 34331 slurm-wlm_23.11.6-1_amd64.buildinfo
Files:
 6662ca8f1bf3d8b10002245fbd0fb99b 5580 admin optional slurm-wlm_23.11.6-1.dsc
 eebb1047ebe95fa334d0edd0d99fc506 10110647 admin optional slurm-wlm_23.11.6.orig.tar.gz
 39732215d688a977e054610d2db9880a 136712 admin optional slurm-wlm_23.11.6-1.debian.tar.xz
 52c690df501da55c823167ea7bc44859 34331 admin optional slurm-wlm_23.11.6-1_amd64.buildinfo

-----BEGIN PGP SIGNATURE-----

iQJFBAEBCgAvFiEE6zNF9WRBuLgad5h2ffpBrZYZhdcFAmZAxCERHG9saXZhQGRl
Ymlhbi5vcmcACgkQffpBrZYZhdd1XA//cjAEq8nSk/5JQA7UuOGRNORXP6aeE7wD
SOCtn/yTiI3bzMmzoO6AsMnKpPEEyd/iO9N1e6ifJPepb/co6GwVsFxd0X475Wd4
i2xsyNUm1FX2wGmDvsrMmRtwaz/shZQ9ca1/+o+eS35qoBfUOA+KPAXJW+vA/IYG
dfhHX+I2TeaFiyUpI5v2kKd0vhab01zLUw2H2CKYikv6G1CtJZTxDGLSjY4JOj+S
5vhNLigjo/r3QvX+mrq8w1LqOu/p+lsyvRRnoA0JxuLmawSM8b7kWyXc7DoZ/KCc
ryneCqkwtR+OpL3pluJqGOTDZivF9o5IeyEAxdN0D9AAb/5waxO3cU6J2j7Kn0Su
bSQdrCfPFUbeLLPTy/z12hWNZXJoN3bVL21ZBHtpg/KnmDpGsxGZGkRGrd6rzenC
BCrDFLV8KRWLm/Mz8eaLFbGjWFwSJihs5wJ6yJ9O9UFDL6w4H0TwKxOoyp0KvQuW
ygvQeTQyAXuRRt1HCmsib1xBQtAOwbdFZ6yUdAFyh4T5ZZXQNId8msgT7UpGN38w
OMbPVau5D9aXjxCrw7WSBHYBYm4yvx7TAGbzM1uNUw7HBmS2f97vHgZBYtFOXEri
of8WVU58yUH7KCY1rYTEj/Uvf3/hvOuzOFdg6Cxz+xW/IGsesnfa5X2HYaOKHCKH
9O0fIkrdTjA=
=Z/X8
-----END PGP SIGNATURE-----

Attachment: pgp_G_eIqW6VO.pgp
Description: PGP signature


--- End Message ---
Reply to: