Re: [OE-core] [PATCH 03/33] dpkg-native: hardcode SYSCONFDIR to /etc in update-alternatives

2016-12-06 Thread Burton, Ross
On 6 December 2016 at 16:37, Andreas Oberritter wrote: > I was asking because the result after stripping would essentially equal > ${sysconfdir_native}: > > STAGING_ETCDIR_NATIVE = "${STAGING_DIR_NATIVE}${sysconfdir_native}" > > It wouldn't work if someone set it to

Re: [OE-core] [PATCH 03/33] dpkg-native: hardcode SYSCONFDIR to /etc in update-alternatives

2016-12-06 Thread Andreas Oberritter
On 06.12.2016 17:12, Burton, Ross wrote: > > On 6 December 2016 at 16:08, Andreas Oberritter > wrote: > > I guess ${sysconfdir_native} should work, unless someone overrides > STAGING_ETCDIR_NATIVE. Would this be ok? > > > If the

Re: [OE-core] [PATCH 03/33] dpkg-native: hardcode SYSCONFDIR to /etc in update-alternatives

2016-12-06 Thread Burton, Ross
On 6 December 2016 at 16:08, Andreas Oberritter wrote: > I guess ${sysconfdir_native} should work, unless someone overrides > STAGING_ETCDIR_NATIVE. Would this be ok? > If the recipe inherits native then the class reassigns: sysconfdir = "${STAGING_ETCDIR_NATIVE}" So

Re: [OE-core] [PATCH 03/33] dpkg-native: hardcode SYSCONFDIR to /etc in update-alternatives

2016-12-06 Thread Andreas Oberritter
On 06.12.2016 16:39, Burton, Ross wrote: > > On 6 December 2016 at 11:49, Andreas Oberritter > wrote: > > +sed -i -e 's|SYSCONFDIR|"/etc"|' ${S}/utils/update-alternatives.c > > > Hardcoding /etc is bad, can't you just remove the

Re: [OE-core] [PATCH 03/33] dpkg-native: hardcode SYSCONFDIR to /etc in update-alternatives

2016-12-06 Thread Burton, Ross
On 6 December 2016 at 11:49, Andreas Oberritter wrote: > +sed -i -e 's|SYSCONFDIR|"/etc"|' ${S}/utils/update-alternatives.c > Hardcoding /etc is bad, can't you just remove the native sysroot prefix from $sysconfdir? Ross --