[Pkg-shadow-devel] Bug#669698: Bug#669698: login: Please support dynamic motd (initscripts change)

[Roger Leigh]

On Sat, Apr 21, 2012 at 02:07:31PM +0200, Christian PERRIER wrote:
> Quoting Roger Leigh (rleigh at debian.org):
> > Package: login
> > Version: 1:4.1.5-1
> > Severity: normal
> > Tags: patch
> > 
> > Separation of static and dynamic motd components
> 
> THanks for your patch, Roger, I committed it.

Please could you add the additional change below.  This just ensures
that in the unlikely even that scripts are present in
/etc/update-motd.d, they aren't run and displayed twice.

If it would be possible to update this this week, that would be
much appreciated--I'm going to make an additional sysvinit
upload to experimental shortly, and hopefully to unstable by the
end of the week, so it would be nice to have this in place before
it goes into unstable.


Thanks again,
Roger


Index: debian/login.pam
===================================================================
--- debian/login.pam	(revision 3740)
+++ debian/login.pam	(working copy)
@@ -84,7 +84,7 @@
 # (Replaces the `MOTD_FILE' option in login.defs)
 # This includes a dynamically generated part from /run/motd.dynamic
 # and a static (admin-editable) part from /etc/motd.
-session    optional   pam_motd.so  motd=/run/motd.dynamic
+session    optional   pam_motd.so  motd=/run/motd.dynamic noupdate
 session    optional   pam_motd.so
 
 # Prints the status of the user's mailbox upon succesful login


-- 
  .''`.  Roger Leigh
 : :' :  Debian GNU/Linux             http://people.debian.org/~rleigh/
 `. `'   Printing on GNU/Linux?       http://gutenprint.sourceforge.net/
   `-    GPG Public Key: 0x25BFB848   Please GPG sign your mail.