<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 11, 2015 at 7:07 PM, Joe MacDonald <span dir="ltr"><<a href="mailto:Joe_MacDonald@mentor.com" target="_blank">Joe_MacDonald@mentor.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">[Re: [yocto] [meta-selinux][PATCH v1] libpam: use wildcard for version and cleanup] On 15.08.11 (Tue 16:39) Shrikant Bobade wrote:<br>
<br>
> Hi Philip,<br>
><br>
><br>
> On Tue, Aug 11, 2015 at 10:39 AM, Philip Tricca <<a href="mailto:flihp@twobit.us">flihp@twobit.us</a>> wrote:<br>
><br>
>  Â  Â Hey Shrikant,<br>
><br>
>  Â  Â On 07/30/2015 02:31 AM, Shrikant Bobade wrote:<br>
>  Â  Â > This patch provides green build for core-image-selinux<br>
>  Â  Â > (meta-selinux:master & poky:master) against libpam upgrade from 1.1.6 to<br>
>  Â  Â > 1.2.1,<br>
>  Â  Â > image boots fine,but I am unable to login at target. I have prepared<br>
>  Â  Â > build for qemuarm, does anyone else facing similar issue? please advice.<br>
>  Â  Â ><br>
>  Â  Â > Observed the login issue appears even with disabled selinux support<br>
>  Â  Â > (selinux=0).<br>
><br>
>  Â  Â I just tested this again after Joe merged the commits from the backlog.<br>
>  Â  Â I'm not longer able to reproduce the failed login. Are you still having<br>
>  Â  Â login problems?<br>
><br>
><br>
> I also got similar results:<br>
> With the check on latest bits: the login issue is not reproducible on<br>
> core-image-selinux(with poky-selinux distro)<br>
> I can now login properly.<br>
<br>
</span>I'm glad to hear that, guys, because I wasn't able to reproduce the<br>
login problem on my setup and was thinking I needed to spend time in the<br>
next couple of days hunting down what's polluting my environment that I<br>
was getting different results than you.  :-)  This is a nice treat.<br>
<br>
-J.<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br></div><div>Thanks Joe & Philip, <br><br></div><div>I just compared my latest setup (login working) with older one(login issue) <br></div><div>observed this patch at poky served the login fix: <a href="http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/meta/recipes-extended/pam/libpam?id=c75cefe8a382a63f625123c156137782db118f64">http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/meta/recipes-extended/pam/libpam?id=c75cefe8a382a63f625123c156137782db118f64</a> <br></div><div><br></div><div>Thanks!<br></div><div>Shrikant<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
><br>
><br>
> Build Configuration:<br>
> BB_VERSION  Â  Â  Â  = "1.27.1"<br>
> BUILD_SYS  Â  Â  Â  Â = "x86_64-linux"<br>
> NATIVELSBSTRING  Â = "Ubuntu-12.04"<br>
> TARGET_SYS  Â  Â  Â  = "arm-poky-linux-gnueabi"<br>
> MACHINE  Â  Â  Â  Â  Â = "qemuarm"<br>
> DISTRO  Â  Â  Â  Â  Â  = "poky-selinux"<br>
> DISTRO_VERSION  Â  = "1.8+snapshot-20150811"<br>
> TUNE_FEATURES  Â  Â = "arm armv5 thumb dsp"<br>
> TARGET_FPU  Â  Â  Â  = "soft"<br>
> meta<br>
> meta-yocto<br>
> meta-yocto-bsp  Â  = "master:a16e0b4014173af46ef80d643bb71055219b0dab"<br>
> meta-selinux  Â  Â  = "master:684ee9401f33db7c9d5b183988d89c688c9dd0be"<br>
><br>
> Thanks!<br>
> Shrikant<br>
><br>
><br>
><br>
><br>
>  Â  Â Philip<br>
><br>
>  Â  Â > On Thu, Jul 30, 2015 at 2:55 PM, Shrikant Bobade<br>
>  Â  Â > <<a href="mailto:bobadeshrikant@gmail.com">bobadeshrikant@gmail.com</a> <mailto:<a href="mailto:bobadeshrikant@gmail.com">bobadeshrikant@gmail.com</a>>> wrote:<br>
>  Â  Â ><br>
>  Â  Â >  Â  Â From: Shrikant Bobade <<a href="mailto:shrikant_bobade@mentor.com">shrikant_bobade@mentor.com</a><br>
>  Â  Â >  Â  Â <mailto:<a href="mailto:shrikant_bobade@mentor.com">shrikant_bobade@mentor.com</a>>><br>
>  Â  Â ><br>
>  Â  Â >  Â  Â use wildcard for version: adopting libpam upgrade from 1.1.6 to<br>
>  Â  Â 1.2.1,<br>
>  Â  Â >  Â  Â cleanup older recipe and remove patch<br>
>  Â  Â sepermit-add-DESTDIR-prefix.patch<br>
>  Â  Â >  Â  Â since the changes already available with latest source.<br>
>  Â  Â ><br>
>  Â  Â >  Â  Â Signed-off-by: Shrikant Bobade <<a href="mailto:shrikant_bobade@mentor.com">shrikant_bobade@mentor.com</a><br>
>  Â  Â >  Â  Â <mailto:<a href="mailto:shrikant_bobade@mentor.com">shrikant_bobade@mentor.com</a>>><br>
>  Â  Â >  Â  Â ---<br>
>  Â  Â >  Â  Â  .../pam/libpam/sepermit-add-DESTDIR-prefix.patch  Â |  Â 31<br>
>  Â  Â >  Â  Â --------------------<br>
>  Â  Â >  Â  Â  recipes-extended/pam/libpam_%.bbappend  Â  Â  Â  Â  Â  Â |  Â  3 ++<br>
>  Â  Â >  Â  Â  recipes-extended/pam/libpam_1.1.6.bbappend  Â  Â  Â  Â |  Â 10 -------<br>
>  Â  Â >  Â  Â  3 files changed, 3 insertions(+), 41 deletions(-)<br>
>  Â  Â >  Â  Â  delete mode 100644<br>
>  Â  Â >  Â  Â recipes-extended/pam/libpam/sepermit-add-DESTDIR-prefix.patch<br>
>  Â  Â >  Â  Â  create mode 100644 recipes-extended/pam/libpam_%.bbappend<br>
>  Â  Â >  Â  Â  delete mode 100644 recipes-extended/pam/libpam_1.1.6.bbappend<br>
>  Â  Â ><br>
>  Â  Â >  Â  Â diff --git<br>
>  Â  Â >  Â  Â a/recipes-extended/pam/libpam/sepermit-add-DESTDIR-prefix.patch<br>
>  Â  Â >  Â  Â b/recipes-extended/pam/libpam/sepermit-add-DESTDIR-prefix.patch<br>
>  Â  Â >  Â  Â deleted file mode 100644<br>
>  Â  Â >  Â  Â index d48d386..0000000<br>
>  Â  Â >  Â  Â --- a/recipes-extended/pam/libpam/sepermit-add-DESTDIR-prefix.patch<br>
>  Â  Â >  Â  Â +++ /dev/null<br>
>  Â  Â >  Â  Â @@ -1,31 +0,0 @@<br>
>  Â  Â >  Â  Â -Subject: [PATCH] libpam: add missing DESTDIR prefix<br>
>  Â  Â >  Â  Â -<br>
>  Â  Â >  Â  Â -The DESTDIR prefix is missing, this will cause build failures for<br>
>  Â  Â >  Â  Â -mkdir /var/run/sepermit on the host.<br>
>  Â  Â >  Â  Â -<br>
>  Â  Â >  Â  Â -| mkdir -p /var/run/sepermit<br>
>  Â  Â >  Â  Â -| mkdir: cannot create directory `/var/run/sepermit': Permission<br>
>  Â  Â denied<br>
>  Â  Â >  Â  Â -<br>
>  Â  Â >  Â  Â -Upstream-Status: Pending<br>
>  Â  Â >  Â  Â -<br>
>  Â  Â >  Â  Â -Signed-off-by: Xin Ouyang <<a href="mailto:Xin.Ouyang@windriver.com">Xin.Ouyang@windriver.com</a><br>
>  Â  Â >  Â  Â <mailto:<a href="mailto:Xin.Ouyang@windriver.com">Xin.Ouyang@windriver.com</a>>><br>
>  Â  Â >  Â  Â ----<br>
>  Â  Â >  Â  Â - modules/pam_sepermit/Makefile.am |  Â  2 +-<br>
>  Â  Â >  Â  Â - 1 files changed, 1 insertions(+), 1 deletions(-)<br>
>  Â  Â >  Â  Â -<br>
>  Â  Â >  Â  Â -diff --git a/modules/pam_sepermit/Makefile.am<br>
>  Â  Â >  Â  Â b/modules/pam_sepermit/Makefile.am<br>
>  Â  Â >  Â  Â -index cfc5594..bc82275 100644<br>
>  Â  Â >  Â  Â ---- a/modules/pam_sepermit/Makefile.am<br>
>  Â  Â >  Â  Â -+++ b/modules/pam_sepermit/Makefile.am<br>
>  Â  Â >  Â  Â -@@ -35,7 +35,7 @@ if HAVE_LIBSELINUX<br>
>  Â  Â >  Â  Â -  Â securelib_LTLIBRARIES = <a href="http://pam_sepermit.la" rel="noreferrer" target="_blank">pam_sepermit.la</a> <<a href="http://pam_sepermit.la" rel="noreferrer" target="_blank">http://pam_sepermit.la</a>><br>
>  Â  Â >  Â  Â -<br>
>  Â  Â >  Â  Â - install-data-local:<br>
>  Â  Â >  Â  Â --  Â  Â  mkdir -p $(sepermitlockdir)<br>
>  Â  Â >  Â  Â -+  Â  Â  mkdir -p $(DESTDIR)$(sepermitlockdir)<br>
>  Â  Â >  Â  Â - endif<br>
>  Â  Â >  Â  Â - if ENABLE_REGENERATE_MAN<br>
>  Â  Â >  Â  Â - noinst_DATA = README pam_sepermit.8 sepermit.conf.5<br>
>  Â  Â >  Â  Â ---<br>
>  Â  Â >  Â  Â -1.7.5.4<br>
>  Â  Â >  Â  Â -<br>
>  Â  Â >  Â  Â diff --git a/recipes-extended/pam/libpam_%.bbappend<br>
>  Â  Â >  Â  Â b/recipes-extended/pam/libpam_%.bbappend<br>
>  Â  Â >  Â  Â new file mode 100644<br>
>  Â  Â >  Â  Â index 0000000..adcf938<br>
>  Â  Â >  Â  Â --- /dev/null<br>
>  Â  Â >  Â  Â +++ b/recipes-extended/pam/libpam_%.bbappend<br>
>  Â  Â >  Â  Â @@ -0,0 +1,3 @@<br>
>  Â  Â >  Â  Â +inherit enable-selinux<br>
>  Â  Â >  Â  Â +<br>
>  Â  Â >  Â  Â +RDEPENDS_${PN}-runtime += "${@target_selinux(d,<br>
>  Â  Â 'pam-plugin-selinux')}"<br>
>  Â  Â >  Â  Â diff --git a/recipes-extended/pam/libpam_1.1.6.bbappend<br>
>  Â  Â >  Â  Â b/recipes-extended/pam/libpam_1.1.6.bbappend<br>
>  Â  Â >  Â  Â deleted file mode 100644<br>
>  Â  Â >  Â  Â index 71acecc..0000000<br>
>  Â  Â >  Â  Â --- a/recipes-extended/pam/libpam_1.1.6.bbappend<br>
>  Â  Â >  Â  Â +++ /dev/null<br>
>  Â  Â >  Â  Â @@ -1,10 +0,0 @@<br>
>  Â  Â >  Â  Â -#FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"<br>
>  Â  Â >  Â  Â -FILESPATH_append := ":${@base_set_filespath(['${THISDIR}/${PN}'],<br>
>  Â  Â d)}"<br>
>  Â  Â >  Â  Â -<br>
>  Â  Â >  Â  Â -SRC_URI += "file://sepermit-add-DESTDIR-prefix.patch"<br>
>  Â  Â >  Â  Â -<br>
>  Â  Â >  Â  Â -PR .= ".4"<br>
>  Â  Â >  Â  Â -<br>
>  Â  Â >  Â  Â -inherit enable-selinux<br>
>  Â  Â >  Â  Â -<br>
>  Â  Â >  Â  Â -RDEPENDS_${PN}-runtime += "${@target_selinux(d,<br>
>  Â  Â 'pam-plugin-selinux')}"<br>
>  Â  Â >  Â  Â --<br>
>  Â  Â >  Â  Â 1.7.9.5<br>
>  Â  Â ><br>
>  Â  Â ><br>
>  Â  Â ><br>
>  Â  Â ><br>
><br>
><br>
><br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
-Joe MacDonald.<br>
:wq<br>
</font></span></blockquote></div><br></div></div>