Re: [Angstrom-devel] [meta-angstrom][PATCH 2/2] fs-perms-angstrom.txt: backport patches from openembedded-core

2016-08-27 Thread Max Krummenacher
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

2016-08-27 Thread Max Krummenacher
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

2016-08-27 Thread Max Krummenacher
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