Re: [Angstrom-devel] [meta-angstrom][PATCH 2/2] fs-perms-angstrom.txt: backport patches from openembedded-core
Ping. Max 2016-07-18 21:45 GMT+02:00 Max Krummenacher : > Since the split from openembedded-core there has been a number of patches > to fs-perms.txt. Backport them here. > > Signed-off-by: Max Krummenacher > --- > files/fs-perms-angstrom.txt | 16 +--- > 1 file changed, 9 insertions(+), 7 deletions(-) > > diff --git a/files/fs-perms-angstrom.txt b/files/fs-perms-angstrom.txt > index 58066b3..098a739 100644 > --- a/files/fs-perms-angstrom.txt > +++ b/files/fs-perms-angstrom.txt > @@ -47,17 +47,19 @@ ${oldincludedir}0755rootroottrue0644 > rootroot > # Links > ${localstatedir}/run link/run > > +/home 0755rootrootfalse - - - > +${prefix}/src 0755rootrootfalse - - - > +${localstatedir}/local 0755rootrootfalse - - - > + > # Special permissions from base-files > # Set 1777 > /tmp 01777 rootrootfalse - - - > > -# Set 2775 > -/home 02755 rootrootfalse - - - > -${prefix}/src 02755 rootrootfalse - - - > -${localstatedir}/local 02755 rootrootfalse - - - > +# Set 0700 > +${ROOT_HOME} 0700rootrootfalse - - - > > -# Set 3755 > +# Set 755-lsb > /srv 0755rootrootfalse - - - > > -# Set 4775 > -/var/mail 02755 rootrootfalse - - - > +# Set 2775-lsb > +/var/mail 02775 rootmailfalse - - - > -- > 2.6.6 > ___ Angstrom-distro-devel mailing list Angstrom-distro-devel@linuxtogo.org http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/angstrom-distro-devel
Re: [Angstrom-devel] [meta-angstrom][PATCH 1/2] angstrom-core-tweaks.inc: actually use fs-perms-angstrom.txt
Ping. Max 2016-07-29 15:47 GMT+02:00 Max Krummenacher : > Hi Andreas > > Am Freitag, den 29.07.2016, 13:09 +0200 schrieb Andreas Müller: >> On Mon, Jul 18, 2016 at 9:45 PM, Max Krummenacher < >> max.oss...@gmail.com> wrote: >> > FILESYSTEM_PERMS_TABLES must be set with the relative path from one >> > of the >> > BBPATH paths. >> > >> > Without this patch the setting of FILESYSTEM_PERMS_TABLES disables >> > the use >> > of openembedded-core/meta/files/fs-perms.txt but, as the file in >> > meta-angstrom/files/fs-perms-angstrom.txt is not found, that file >> > is not >> > used for the fixup-perms task. >> > >> > With fs-perms-angstrom.txt active files deployed from >> > copydebugsources no >> > longer trigger these warnings: >> > > WARNING: helloworld-1.0-r0 do_package_qa: QA Issue: helloworld: >> > > .../src/main.c >> > > is owned by uid 1000, which is the same as the user running >> > > bitbake. This may >> > > be due to host contamination [host-user-contaminated] >> > >> > Signed-off-by: Max Krummenacher >> > --- >> > conf/distro/include/angstrom-core-tweaks.inc | 2 +- >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > diff --git a/conf/distro/include/angstrom-core-tweaks.inc >> > b/conf/distro/include/angstrom-core-tweaks.inc >> > index 33aaaef..1a3af9d 100644 >> > --- a/conf/distro/include/angstrom-core-tweaks.inc >> > +++ b/conf/distro/include/angstrom-core-tweaks.inc >> > @@ -62,7 +62,7 @@ PACKAGE_CLASSES = "package_ipk" >> > # Image.bbclass stuff: >> > >> > # Override default fs-perms.txt that messes with our /var setup: >> > -FILESYSTEM_PERMS_TABLES = "fs-perms-angstrom.txt" >> > +FILESYSTEM_PERMS_TABLES = "files/fs-perms-angstrom.txt" >> > >> > # >> > # udev, devfsd, busybox-mdev (from busybox) or none >> > -- >> > 2.6.6 >> > >> Do we need fs-perms / for special /var setup - see also base-files? >> Note that /var/lock configuration is broken causing very ugly effects >> [1]. >> >> [1] https://github.com/schnitzeltony/meta-angstrom/commit/937fb6a41f5 >> 5132982aba1308aed4fe3f9b82be0 >> >> Andreas > > My goal is to have the host-user-contaminated warnings go away. They > occur because the source files are hard linked in do_package and pseudo > then set them to root:root. > > In poky this is fixed with the fs-perms mechanism and the configuration > file from meta/files/fs-perms.txt. > > I then found that Koen's original patch to provide an angstrom fs-perms > configuration file did in fact disable the use of meta/files/fs > -perms.txt but did not make available what is in meta-angstrom/files/fs > -perms-angstrom.txt. > > And as there has been some fixes in oe-core since splitting of the fs > -perms.txt file I tried to backport those which looked not to interfere > with Koen's intentions with the commit > 6d518574c11bca1a56ec9b83afd18196a82fbfb0 'angstrom: use custom fs > -perms.txt that doesn't destroy our /var setup'. > > The only symlink created with meta-angstrom/files/fs-perms-angstrom.txt > is the one Khem added '/var/run -> /run'. > But that one somehow magically appears also without my patch. Seems > like base_files does create it already in do_install. > > So would you like me to send a v2 of the patch which removes the > '/var/run -> /run' symlink creation? > > Max > ___ Angstrom-distro-devel mailing list Angstrom-distro-devel@linuxtogo.org http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/angstrom-distro-devel
Re: [Angstrom-devel] [meta-angstrom][PATCH] angstrom.inc: set arm instruction set with existing machine overrides
Ping. Max 2016-07-27 20:10 GMT+02:00 Khem Raj : > >> On Jul 27, 2016, at 10:55 AM, Max Krummenacher wrote: >> >> _armv4t, _armv5te are not (no longer?) set as MACHINEOVERRIDES if an armv4t >> or armv5te tune is set. >> Thus the default ARM_INSTRUCTION_SET ??= "thumb" kicks in. >> >> Set the override to _armv4, _armv5 to get the expected ARM_INSTRUCTION_SET. >> >> While at it cleanup comment to keep up with changes. >> >> Build tested with a build for a _armv5te machine. > > this is ok. > >> >> Signed-off-by: Max Krummenacher >> --- >> conf/distro/include/angstrom.inc | 19 +-- >> 1 file changed, 9 insertions(+), 10 deletions(-) >> >> diff --git a/conf/distro/include/angstrom.inc >> b/conf/distro/include/angstrom.inc >> index 778ab10..09f6121 100644 >> --- a/conf/distro/include/angstrom.inc >> +++ b/conf/distro/include/angstrom.inc >> @@ -46,16 +46,6 @@ require conf/distro/include/angstrom-${TCLIBC}.inc >> require conf/distro/include/angstrom-java.inc >> require conf/distro/include/angstrom-jalimo.conf >> >> -# ARM920T and up can use thumb mode to decrease binary size at the expense >> of speed >> -# (the complete story is a bit more nuanced due to cache starvation) >> -# Angstrom turns on thumb for armv4t machine according to this RFC: >> -# >> http://lists.linuxtogo.org/pipermail/angstrom-distro-devel/2008-October/002714.html >> - >> -ARM_INSTRUCTION_SET ??= "thumb" >> -ARM_INSTRUCTION_SET_armv4t ?= "arm" >> -ARM_INSTRUCTION_SET_armv5te ?= "arm" >> -ARM_INSTRUCTION_SET_armv6 ?= "arm" >> - >> # "arm" "thumb" >> #The instruction set the compiler should use when generating application >> #code. The kernel is always compiled with arm code at present. arm code >> @@ -63,6 +53,15 @@ ARM_INSTRUCTION_SET_armv6 ?= "arm" >> #encoded RISC sub-set. Thumb code is smaller (maybe 70% of the ARM size) >> #but requires more instructions (140% for 70% smaller code) so may be >> #slower. >> +#thumb 2 is an evolution of the original thumb instruction set which >> strives >> +#to keep the code size advantages without the performance penality. >> +#When setting thumb the compiler will use thumb or thumb2 depending on >> what >> +#the selected arm architecture. >> + >> +ARM_INSTRUCTION_SET ??= "thumb" >> +ARM_INSTRUCTION_SET_armv4 ?= "arm" >> +ARM_INSTRUCTION_SET_armv5 ?= "arm" >> +ARM_INSTRUCTION_SET_armv6 ?= "arm" >> >> #Use this variable in feeds and other parts that need a URI >> ANGSTROM_URI ?= "http://feeds.angstrom-distribution.org"; >> -- >> 2.5.5 >> >> >> ___ >> Angstrom-distro-devel mailing list >> Angstrom-distro-devel@linuxtogo.org >> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/angstrom-distro-devel > > > ___ > Angstrom-distro-devel mailing list > Angstrom-distro-devel@linuxtogo.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/angstrom-distro-devel > ___ Angstrom-distro-devel mailing list Angstrom-distro-devel@linuxtogo.org http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/angstrom-distro-devel