On 11/22/2013 02:49 AM, Saul Wold wrote:
On 11/21/2013 01:14 AM, qiang.c...@windriver.com wrote:
From: Qiang Chen <qiang.c...@windriver.com>
The /etc/default/locale missing message appears when running su <user>
root@qemu0:/var/log# su root
root@qemu0:/var/log# tail auth.log
Oct 8 07:47:54 qemu0 login[983]: pam_unix(login:session): session opened for
user root by LOGIN(uid=0)
Oct 8 07:47:55 qemu0 login[995]: ROOT LOGIN on '/dev/console'
Oct 8 07:48:21 qemu0 su[999]: Successful su for root by root
Oct 8 07:48:21 qemu0 su[999]: + /dev/console root:root
Oct 8 07:48:21 qemu0 su[999]: pam_env(su:session): Unable to open env file:
/etc/default/locale: No such file or directory
Oct 8 07:48:21 qemu0 su[999]: pam_unix(su:session): session opened for user
root by root(uid=0)
This commit remove reference from pam.d/su to /etc/default/locale
env file to avoid the error messages.
Signed-off-by: Qiang Chen <qiang.c...@windriver.com>
---
meta/recipes-extended/shadow/files/pam.d/su | 3 ---
meta/recipes-extended/shadow/shadow.inc | 2 +-
2 files changed, 1 insertion(+), 4 deletions(-)
diff --git a/meta/recipes-extended/shadow/files/pam.d/su
b/meta/recipes-extended/shadow/files/pam.d/su
index 8e35137..8d590a3 100644
--- a/meta/recipes-extended/shadow/files/pam.d/su
+++ b/meta/recipes-extended/shadow/files/pam.d/su
@@ -34,9 +34,6 @@ auth sufficient pam_rootok.so
#
# parsing /etc/environment needs "readenv=1"
session required pam_env.so readenv=1
-# locale variables are also kept into /etc/default/locale in etch
-# reading this file *in addition to /etc/environment* does not hurt
-session required pam_env.so readenv=1 envfile=/etc/default/locale
Should we be providing an empty locale file instead, what happens if
someone wants to use the locale mechanism?
Hi Sau,
Thanks for your comments !
I didn't see much sense providing an empty/faked locale file there, as we
didn't provide a update-locale command tool either.
Meanwhile, RHEL, fedora didn't require an env file to /etc/default/locale
too. If someone really cares the env for locale related, they just need
specify their envfile instead our faked env file.
# Defines the MAIL environment variable
# However, userdel also needs MAIL_DIR and MAIL_FILE variables
diff --git a/meta/recipes-extended/shadow/shadow.inc
b/meta/recipes-extended/shadow/shadow.inc
index 5ab978a..572c3bc 100644
--- a/meta/recipes-extended/shadow/shadow.inc
+++ b/meta/recipes-extended/shadow/shadow.inc
@@ -44,7 +44,7 @@ SRC_URI_append_class-nativesdk = " \
SRC_URI[md5sum] = "b8608d8294ac88974f27b20f991c0e79"
SRC_URI[sha256sum] =
"633f5bb4ea0c88c55f3642c97f9d25cbef74f82e0b4cf8d54e7ad6f9f9caa778"
-PR = "r13"
+PR = "r14"
No more PR bumps please.
OK. I will send V2 patch dropping PR bumps and remove login reference to
/etc/default/locale too.
Thanks !
Qiang
Sau!
# Additional Policy files for PAM
PAM_SRC_URI = "file://pam.d/chfn \
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
