Re: [Pkg-shadow-devel] Help wanted: test new shadow source package (login, passwd, uidmap, etc.)

2014-05-02 Thread Christian PERRIER
Quoting Serge Hallyn (serge.hal...@ubuntu.com):
 Quoting Christian PERRIER (bubu...@debian.org):
  Quoting Christian PERRIER (bubu...@debian.org):
   Hello fellow developers,
   
   I would like to request your help in testing the new version of the
   shadow package (that provides login, passwd and such other important
   or base packages).
  
  I haven't got much feedbackwhich is indeed what I was more or less
  expecting. ;-)
  
  So, well, let's jump into the mud (I love to do that when
  running.not sure I love to do that in my FLOSS activities) and
  I'll soon upload shadow to unstable Be prepared.
 
 Hi,
 
 so first glitch I found is that /etc/subuid was not created for me.
 login.postinst only creates that on new installs.  In Ubuntu it
 does so anytime it does not exist - I assume you made that change
 on purpose?  usermod -v refuses to run if the file does not exist,
 so users will need to be told to create those files themselves.

What makes you think this?

In login.postinst, we have:

# Create subuid/subgid if missing
if [ ! -e /etc/subuid ]; then
touch /etc/subuid
chown root:root /etc/subuid
chmod 644 /etc/subuid
fi


(strangely indented, admitedlybut unless I'm missing something
obvious, it is unconditionnally run)

That code probably somes unchanged from the patches that have been
proposed, indeed.

And, well, on my system, /etc/subuid and /etc/subgid were indeed
created when I manually installed the new login package.



signature.asc
Description: Digital signature


Re: [Pkg-shadow-devel] Help wanted: test new shadow source package (login, passwd, uidmap, etc.)

2014-05-02 Thread Serge Hallyn
Quoting Christian PERRIER (bubu...@debian.org):
 Quoting Serge Hallyn (serge.hal...@ubuntu.com):
  Quoting Christian PERRIER (bubu...@debian.org):
   Quoting Christian PERRIER (bubu...@debian.org):
Hello fellow developers,

I would like to request your help in testing the new version of the
shadow package (that provides login, passwd and such other important
or base packages).
   
   I haven't got much feedbackwhich is indeed what I was more or less
   expecting. ;-)
   
   So, well, let's jump into the mud (I love to do that when
   running.not sure I love to do that in my FLOSS activities) and
   I'll soon upload shadow to unstable Be prepared.
  
  Hi,
  
  so first glitch I found is that /etc/subuid was not created for me.
  login.postinst only creates that on new installs.  In Ubuntu it
  does so anytime it does not exist - I assume you made that change
  on purpose?  usermod -v refuses to run if the file does not exist,
  so users will need to be told to create those files themselves.
 
 What makes you think this?
 
 In login.postinst, we have:
 
 # Create subuid/subgid if missing
 if [ ! -e /etc/subuid ]; then
 touch /etc/subuid
 chown root:root /etc/subuid
 chmod 644 /etc/subuid
 fi
 
 
 (strangely indented, admitedlybut unless I'm missing something

Oh, yo'ure right.  The indenting threw me off.

And the reason they weren't created was that I was blindly expecting
the uidmap package to depend on both passwd and login = 1:4.2-1,
so while i had upgraded passwd to experimental's version i hadn't
upgraded login's.

Sorry, my bad.  So all seemed to be working fine!

 obvious, it is unconditionnally run)
 
 That code probably somes unchanged from the patches that have been
 proposed, indeed.
 
 And, well, on my system, /etc/subuid and /etc/subgid were indeed
 created when I manually installed the new login package.
 



 ___
 Pkg-shadow-devel mailing list
 pkg-shadow-de...@lists.alioth.debian.org
 http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-shadow-devel


-- 
To UNSUBSCRIBE, email to debian-devel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/20140502133733.GB1760@ubuntumail



Re: [Pkg-shadow-devel] Help wanted: test new shadow source package (login, passwd, uidmap, etc.)

2014-05-02 Thread Serge Hallyn
Quoting Steve Langasek (vor...@debian.org):
 On Fri, May 02, 2014 at 04:38:15AM +, Serge Hallyn wrote:
  Quoting Christian PERRIER (bubu...@debian.org):
   Quoting Christian PERRIER (bubu...@debian.org):
Hello fellow developers,

I would like to request your help in testing the new version of the
shadow package (that provides login, passwd and such other important
or base packages).
 
   I haven't got much feedbackwhich is indeed what I was more or less
   expecting. ;-)
 
   So, well, let's jump into the mud (I love to do that when
   running.not sure I love to do that in my FLOSS activities) and
   I'll soon upload shadow to unstable Be prepared.
 
  so first glitch I found is that /etc/subuid was not created for me.
  login.postinst only creates that on new installs.  In Ubuntu it
  does so anytime it does not exist - I assume you made that change
  on purpose?  usermod -v refuses to run if the file does not exist,
  so users will need to be told to create those files themselves.
 
 The right answer is probably to create the file on new installs and on
 upgrades from versions earlier than the first version introducing this.

Yeah I was thinking that last night - i guess this is pretty easy to
do, and some people will not want to have those files.


-- 
To UNSUBSCRIBE, email to debian-devel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/20140502133823.GC1760@ubuntumail