diff mbox series

libpam: Fix build with DISTRO_FEATURES usrmerge

Message ID 20211012111726.493810-1-zboszor@pr.hu
State New
Headers show
Series libpam: Fix build with DISTRO_FEATURES usrmerge | expand

Commit Message

Böszörményi Zoltán Oct. 12, 2021, 11:17 a.m. UTC
From: Zoltán Böszörményi <zboszor@gmail.com>

Signed-off-by: Zoltán Böszörményi <zboszor@gmail.com>
---
 meta/recipes-extended/pam/libpam_1.5.2.bb | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Böszörményi Zoltán Oct. 14, 2021, 6:54 a.m. UTC | #1
On 2021. 10. 13. 21:38, Dan McGregor wrote:
> Thanks for this. I hit it pretty immediately.
> 
> On Tue, 12 Oct 2021 at 05:17, Zoltan Boszormenyi via
> lists.openembedded.org <zboszor=pr.hu@lists.openembedded.org> wrote:
>>
>> From: Zoltán Böszörményi <zboszor@gmail.com>
>>
>> Signed-off-by: Zoltán Böszörményi <zboszor@gmail.com>
>> ---
>>   meta/recipes-extended/pam/libpam_1.5.2.bb | 6 ++++--
>>   1 file changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/recipes-extended/pam/libpam_1.5.2.bb b/meta/recipes-extended/pam/libpam_1.5.2.bb
>> index a3c4a50cb3..ee3a84a3b6 100644
>> --- a/meta/recipes-extended/pam/libpam_1.5.2.bb
>> +++ b/meta/recipes-extended/pam/libpam_1.5.2.bb
>> @@ -147,8 +147,10 @@ do_install() {
>>          if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true','false',d)}; then
>>                  echo "session optional pam_systemd.so" >> ${D}${sysconfdir}/pam.d/common-session
>>          fi
>> -        install -d ${D}/${libdir}/
>> -       mv ${D}/${base_libdir}/pkgconfig ${D}/${libdir}/
>> +       if ${@bb.utils.contains('DISTRO_FEATURES','usrmerge','false','true',d)}; then
> 
> In the patch I made that I kept local for now my condition was
> 
> if [ "${base_libdir}" != "${libdir}" ]
> 
> It could be that a user sets libdir and base_libdir to be equal
> without enabling usrmerge, but both checks solve the immediate
> problem.

Right, this is the correct solution.

> 
> 
>> +               install -d ${D}/${libdir}/
>> +               mv ${D}/${base_libdir}/pkgconfig ${D}/${libdir}/
>> +       fi
>>   }
>>
>>   do_install_ptest() {
>> --
>> 2.31.1
>>
>>
>>
>>
>>
>>
>> 
>>
Alexandre Belloni Oct. 14, 2021, 11:39 a.m. UTC | #2
On 14/10/2021 08:54:35+0200, Zoltan Boszormenyi via lists.openembedded.org wrote:
> On 2021. 10. 13. 21:38, Dan McGregor wrote:
> > Thanks for this. I hit it pretty immediately.
> > 
> > On Tue, 12 Oct 2021 at 05:17, Zoltan Boszormenyi via
> > lists.openembedded.org <zboszor=pr.hu@lists.openembedded.org> wrote:
> > > 
> > > From: Zoltán Böszörményi <zboszor@gmail.com>
> > > 
> > > Signed-off-by: Zoltán Böszörményi <zboszor@gmail.com>
> > > ---
> > >   meta/recipes-extended/pam/libpam_1.5.2.bb | 6 ++++--
> > >   1 file changed, 4 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/meta/recipes-extended/pam/libpam_1.5.2.bb b/meta/recipes-extended/pam/libpam_1.5.2.bb
> > > index a3c4a50cb3..ee3a84a3b6 100644
> > > --- a/meta/recipes-extended/pam/libpam_1.5.2.bb
> > > +++ b/meta/recipes-extended/pam/libpam_1.5.2.bb
> > > @@ -147,8 +147,10 @@ do_install() {
> > >          if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true','false',d)}; then
> > >                  echo "session optional pam_systemd.so" >> ${D}${sysconfdir}/pam.d/common-session
> > >          fi
> > > -        install -d ${D}/${libdir}/
> > > -       mv ${D}/${base_libdir}/pkgconfig ${D}/${libdir}/
> > > +       if ${@bb.utils.contains('DISTRO_FEATURES','usrmerge','false','true',d)}; then
> > 
> > In the patch I made that I kept local for now my condition was
> > 
> > if [ "${base_libdir}" != "${libdir}" ]
> > 
> > It could be that a user sets libdir and base_libdir to be equal
> > without enabling usrmerge, but both checks solve the immediate
> > problem.
> 
> Right, this is the correct solution.
> 

The patch was just merged in master so you'd have to rebase on that

> > 
> > 
> > > +               install -d ${D}/${libdir}/
> > > +               mv ${D}/${base_libdir}/pkgconfig ${D}/${libdir}/
> > > +       fi
> > >   }
> > > 
> > >   do_install_ptest() {
> > > --
> > > 2.31.1
> > > 
> > > 
> > > 
> > > 
> > > 
> > > 
> > > 
> > > 
> 

> 
> 
>
diff mbox series

Patch

diff --git a/meta/recipes-extended/pam/libpam_1.5.2.bb b/meta/recipes-extended/pam/libpam_1.5.2.bb
index a3c4a50cb3..ee3a84a3b6 100644
--- a/meta/recipes-extended/pam/libpam_1.5.2.bb
+++ b/meta/recipes-extended/pam/libpam_1.5.2.bb
@@ -147,8 +147,10 @@  do_install() {
 	if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true','false',d)}; then
 		echo "session optional pam_systemd.so" >> ${D}${sysconfdir}/pam.d/common-session
 	fi
-        install -d ${D}/${libdir}/
-	mv ${D}/${base_libdir}/pkgconfig ${D}/${libdir}/
+	if ${@bb.utils.contains('DISTRO_FEATURES','usrmerge','false','true',d)}; then
+		install -d ${D}/${libdir}/
+		mv ${D}/${base_libdir}/pkgconfig ${D}/${libdir}/
+	fi
 }
 
 do_install_ptest() {