Re: [OE-core] [PATCH] mkefidisk: Create interactive menu for the script

[Bottazzini, Bruno]

Ping

On Fri, 2015-09-04 at 19:19 -0300, Bruno Bottazzini wrote:
> If it is passed only the image parameter, it will show some interactive
> options to help to conclude the script.
> 
> With this patch it will detect devices available in the machine, asking
> to choose what it is wanted to be flashed.
> 
> it will suggest the target device. If it is a SD card the suggestion
> is /dev/mmcblk0. If it is a pendrive it will suggest to use /dev/sda.
> 
> Signed-off-by: Bruno Bottazzini 
> ---
>  scripts/contrib/mkefidisk.sh | 63 
> 
>  1 file changed, 58 insertions(+), 5 deletions(-)
> 
> diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
> index 55f72b0..66e0e58 100755
> --- a/scripts/contrib/mkefidisk.sh
> +++ b/scripts/contrib/mkefidisk.sh
> @@ -152,8 +152,55 @@ unmount() {
>  # Parse and validate arguments
>  #
>  if [ $# -lt 3 ] || [ $# -gt 4 ]; then
> - usage
> - exit 1
> +if [ $# -eq 1 ]; then
> +AVAILABLE_DISK=`lsblk | grep "disk" | cut -f 1 -d " "`
> +X=0
> +for disk in `echo $AVAILABLE_DISK`; do
> +mounted=`blkid -o list | grep "/dev/$disk" | grep "not mounted"`
> +if [ -n "$mounted" ]; then
> +UNMOUNTED_AVAILABLES="$UNMOUNTED_AVAILABLES /dev/$disk\n"
> +info "$X - /dev/$disk"
> +X=`expr $X + 1`
> +fi
> +done
> +if [ $X -eq 0 ]; then
> +die "No unmounted device found."
> +fi
> +read -p "Choose unmounted device number: " DISK_NUMBER
> +X=0
> +for line in `echo $UNMOUNTED_AVAILABLES`; do
> +if [ $DISK_NUMBER -eq $X ]; then
> +DISK_TO_BE_FLASHED=$line
> +break
> +else
> +X=`expr $X + 1`
> +fi
> +done
> +if [ -z "$DISK_TO_BE_FLASHED" ]; then
> +die "Option \"$DISK_NUMBER\" is invalid. Choose a valid option"
> +else
> +if [ -z `echo $DISK_TO_BE_FLASHED | grep "mmc"` ]; then
> +TARGET_TO_BE_BOOT="/dev/sda"
> +else
> +TARGET_TO_BE_BOOT="/dev/mmcblk0"
> +fi
> +fi
> +echo ""
> +echo "Choose a name of the device that will be boot from"
> +echo -n "Recommended name is: "
> +info "$TARGET_TO_BE_BOOT\n"
> +read -p "Is target device okay? [y/N]: " RESPONSE
> +if [ "$RESPONSE" != "y" ]; then
> +read -p "Choose target device name: " TARGET_TO_BE_BOOT
> +fi
> +echo ""
> +if [ -z "$TARGET_TO_BE_BOOT" ]; then
> +die "Error: choose a valid target name"
> +fi
> +else
> +usage
> + exit 1
> +fi
>  fi
>  
>  if [ "$1" = "-v" ]; then
> @@ -162,9 +209,15 @@ if [ "$1" = "-v" ]; then
>   shift
>  fi
>  
> -DEVICE=$1
> -HDDIMG=$2
> -TARGET_DEVICE=$3
> +if [ -z "$AVAILABLE_DISK" ]; then
> +DEVICE=$1
> +HDDIMG=$2
> +TARGET_DEVICE=$3
> +else
> +DEVICE=$DISK_TO_BE_FLASHED
> +HDDIMG=$1
> +TARGET_DEVICE=$TARGET_TO_BE_BOOT
> +fi
>  
>  LINK=$(readlink $DEVICE)
>  if [ $? -eq 0 ]; then


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] mkefidisk: Create interactive menu for the script

[Bottazzini, Bruno]

On Tue, 2015-09-08 at 08:15 -0700, Saul Wold wrote:
> On 09/04/2015 03:19 PM, Bruno Bottazzini wrote:
> > If it is passed only the image parameter, it will show some interactive
> > options to help to conclude the script.
> >
> > With this patch it will detect devices available in the machine, asking
> > to choose what it is wanted to be flashed.
> >
> > it will suggest the target device. If it is a SD card the suggestion
> > is /dev/mmcblk0. If it is a pendrive it will suggest to use /dev/sda.
> >
> > Signed-off-by: Bruno Bottazzini 
> > ---
> >   scripts/contrib/mkefidisk.sh | 63 
> > 
> >   1 file changed, 58 insertions(+), 5 deletions(-)
> >
> > diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
> > index 55f72b0..66e0e58 100755
> > --- a/scripts/contrib/mkefidisk.sh
> > +++ b/scripts/contrib/mkefidisk.sh
> > @@ -152,8 +152,55 @@ unmount() {
> >   # Parse and validate arguments
> >   #
> >   if [ $# -lt 3 ] || [ $# -gt 4 ]; then
> > -   usage
> > -   exit 1
> > +if [ $# -eq 1 ]; then
> > +AVAILABLE_DISK=`lsblk | grep "disk" | cut -f 1 -d " "`
> > +X=0
> > +for disk in `echo $AVAILABLE_DISK`; do
> > +mounted=`blkid -o list | grep "/dev/$disk" | grep "not 
> > mounted"`
> Is there any way to not call blkid multiple time (ie save the output and 
> use it once?

Unfortunately no, both outputs are different.

> 
> 
> Are the lsblk and blkid commands guaranteed to be installed on most 
> machines?

I double checked it before sending this patch and the answer was yes.

> 
> Also this seems to have matched my root disk, which again concerns me, I 
> am still not convinced that a selection system like this is safe.

This is odd. My output shows only unmounted disks.

> 
> > +if [ -n "$mounted" ]; then
> > +UNMOUNTED_AVAILABLES="$UNMOUNTED_AVAILABLES /dev/$disk\n"
> 
> I think the \n is not needed here, what shell are you using, with Bash I 
> get the following output:
> 
> 0 - /dev/sda
> 1 - /dev/sdb
> 2 - /dev/mmcblk0
> Choose unmounted device number: 1

This output is strange mine shows only (without sda and sdb which i do
have them also, but mounted):
contrib > ./mkefidisk.sh image
0 - /dev/mmcblk0
Choose unmounted device number:

Are you sure you have updated the script with the changes? What distro
are you using?

> 
> Choose a name of the device that will be boot from
> Recommended name is: /dev/sda\n
> Is target device okay? [y/N]: y
> 
> Usage: mkefidisk.sh [-v] DEVICE HDDIMG TARGET_DEVICE
> -v: Verbose debug
> DEVICE: The device to write the image to, e.g. /dev/sdh
> HDDIMG: The hddimg file to generate the efi disk from
> TARGET_DEVICE: The device the target will boot from, e.g. 
> /dev/mmcblk0
> Device /dev/sdb\n cannot be found
> 
> Note the \n above in the choose a name and then the trailing \n here 
> with the device not being found.

You are right I am removing \n. Strangely here it works perfectly:
0 - /dev/sdc
1 - /dev/mmcblk0
Choose unmounted device number: 1

Choose a name of the device that will be boot from
Recommended name is: /dev/mmcblk0
Is target device okay? [y/N]: y


> 
> 
> 
> > +info "$X - /dev/$disk"
> > +X=`expr $X + 1`
> > +fi
> > +done
> > +if [ $X -eq 0 ]; then
> > +die "No unmounted device found."
> > +fi
> > +read -p "Choose unmounted device number: " DISK_NUMBER
> > +X=0
> > +for line in `echo $UNMOUNTED_AVAILABLES`; do
> > +if [ $DISK_NUMBER -eq $X ]; then
> > +DISK_TO_BE_FLASHED=$line
> > +break
> > +else
> > +X=`expr $X + 1`
> > +fi
> > +done
> > +if [ -z "$DISK_TO_BE_FLASHED" ]; then
> > +die "Option \"$DISK_NUMBER\" is invalid. Choose a valid option"
> > +else
> > +if [ -z `echo $DISK_TO_BE_FLASHED | grep "mmc"` ]; then
> > +TARGET_TO_BE_BOOT="/dev/sda"
> > +else
> > +TARGET_TO_BE_BOOT="/dev/mmcblk0"
> > +fi
> > +fi
> > +echo ""
> > +echo "Choose a name of the device that will be boot from"
> > +echo -n "Recommended name is: "
> > +info "$TARGET_TO_BE_BOOT\n"
> 
> You are still outputting the \n here

Removed too

Thank you a lot,

Bruno

> 
> > +read -p "Is target device okay? [y/N]: " RESPONSE
> > +if [ "$RESPONSE" != "y" ]; then
> > +read -p "Choose target device name: " TARGET_TO_BE_BOOT
> > +fi
> > +echo ""
> > +if [ -z "$TARGET_TO_BE_BOOT" ]; then
> > +die "Error: choose a valid target name"
> > +fi
> > +else
> > +usage
> > +   exit 1
> > +fi
> >   fi
> >
> >   if [ "$1" = "-v" 

Re: [OE-core] [PATCH] scripts: Create interactive menu for mkefidisk script

[Bottazzini, Bruno]

On Fri, 2015-09-04 at 10:32 -0300, Bottazzini, Bruno wrote:
> On Thu, 2015-09-03 at 16:36 -0700, Saul Wold wrote:
> > On 09/01/2015 12:22 PM, Bruno Bottazzini wrote:
> > > If it is passed only the image parameter, it will show some interactive
> > > options to help to conclude the script.
> > >
> > > With this patch it will detect devices available in the machine, asking
> > > to choose what it is wanted to be flashed.
> > >
> > > it will suggest the target device. If it is a SD card the suggestion
> > > is /dev/mmcblk0. If it is a pendrive it will suggest to use /dev/sda.
> > >
> > 
> > I am little concerned by this patch since it allows the root disk to be 
> > selected, even if someone can type it on the command line, but offering 
> > it in list would make someone believe that it was OK.
> 
> You are totally right. 
> 
> However, the script already warn and handles this kind of situation. 
> Also, this gives free will to the user. I don't see this as problem but
> something, besides as you have told the user may pass it as argument

Sorry correcting: s/I don't see this as problem but something/I don't
see this as problem but as feature/

> too. Maybe we can change the header message telling "all detect storage"
> or something like this?
> 
> I was wondering if the user wants to flash a H.D. when running a live CD
> and then maybe the root may be a right choice.
> 
> Since I have started using this patch, it got really faster when it
> comes flashing the device (I don't have to remember parameters every
> time). I think this will be a really good new feature for this script.
> 
> > 
> > 
> > > Signed-off-by: Bruno Bottazzini <bruno.bottazz...@intel.com>
> > > ---
> > >   scripts/contrib/mkefidisk.sh | 60 
> > > 
> > >   1 file changed, 55 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
> > > index 55f72b0..7628de7 100755
> > > --- a/scripts/contrib/mkefidisk.sh
> > > +++ b/scripts/contrib/mkefidisk.sh
> > > @@ -152,8 +152,52 @@ unmount() {
> > >   # Parse and validate arguments
> > >   #
> > >   if [ $# -lt 3 ] || [ $# -gt 4 ]; then
> > > - usage
> > > - exit 1
> > > +if [ $# -eq 1 ]; then
> > > +AVAILABLE_DISK=`fdisk -l | grep "Disk /" | cut -f 1 -d ':' | cut 
> > > -f 2 -d ' '`
> > > +if [ -z "$AVAILABLE_DISK" ]; then
> > > +die "Starting mkefidisk.sh as root is required"
> > > +fi
> > > +echo "Available disks are"
> > > +X=0
> > > +for line in `echo $AVAILABLE_DISK`; do
> > > +info "$X - $line"
> > > +X=`expr $X + 1`
> > > +done
> > > +read -p "Choose flashable device number: " DISK_NUMBER
> > > +X=0
> > > +for line in `echo $AVAILABLE_DISK`; do
> > > +if [ $DISK_NUMBER -eq $X ]; then
> > > +DISK_TO_BE_FLASHED=$line
> > > +break
> > > +else
> > > +X=`expr $X + 1`
> > > +fi
> > > +done
> > > +if [ -z "$DISK_TO_BE_FLASHED" ]; then
> > > +die "Option \"$DISK_NUMBER\" is invalid. Choose a valid 
> > > option"
> > > +else
> > > +if [ -z `echo $DISK_TO_BE_FLASHED | grep "mmc"` ]; then
> > > +TARGET_TO_BE_BOOT="/dev/sda"
> > > +else
> > > +TARGET_TO_BE_BOOT="/dev/mmcblk0"
> > > +fi
> > > +fi
> > > +echo ""
> > > +echo "Choose a name of the device that will be boot from"
> > > +echo -n "Recommended name is: "
> > > +info "$TARGET_TO_BE_BOOT\n"
> > The \n usage is probably wrong here.
> > 
> > > +read -p "Is target device okay? [y/N]: " RESPONSE
> > > +if [ "$RESPONSE" != "y" ]; then
> > > +read -p "Choose target device name: " TARGET_TO_BE_BOOT
> > > +fi
> > > +echo ""
> > > +if [ -z "$TARGET_TO_BE_BOOT" ]; then
> > > +die "Error: choose a valid target name"
> > > +fi
> > > +else
> > > +usage
> > > + exit 1
> > > +fi
> > >   fi
> > >
> > >   if [ "$1" = "-v" ]; then
> > > @@ -162,9 +206,15 @@ if [ "$1" = "-v" ]; then
> > >   shift
> > >   fi
> > >
> > > -DEVICE=$1
> > > -HDDIMG=$2
> > > -TARGET_DEVICE=$3
> > > +if [ -z "$AVAILABLE_DISK" ]; then
> > > +DEVICE=$1
> > > +HDDIMG=$2
> > > +TARGET_DEVICE=$3
> > > +else
> > > +DEVICE=$DISK_TO_BE_FLASHED
> > > +HDDIMG=$1
> > > +TARGET_DEVICE=$TARGET_TO_BE_BOOT
> > > +fi
> > >
> > >   LINK=$(readlink $DEVICE)
> > >   if [ $? -eq 0 ]; then
> > >
> 
> 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] scripts: Create interactive menu for mkefidisk script

[Bottazzini, Bruno]

On Thu, 2015-09-03 at 16:36 -0700, Saul Wold wrote:
> On 09/01/2015 12:22 PM, Bruno Bottazzini wrote:
> > If it is passed only the image parameter, it will show some interactive
> > options to help to conclude the script.
> >
> > With this patch it will detect devices available in the machine, asking
> > to choose what it is wanted to be flashed.
> >
> > it will suggest the target device. If it is a SD card the suggestion
> > is /dev/mmcblk0. If it is a pendrive it will suggest to use /dev/sda.
> >
> 
> I am little concerned by this patch since it allows the root disk to be 
> selected, even if someone can type it on the command line, but offering 
> it in list would make someone believe that it was OK.

You are totally right. 

However, the script already warn and handles this kind of situation. 
Also, this gives free will to the user. I don't see this as problem but
something, besides as you have told the user may pass it as argument
too. Maybe we can change the header message telling "all detect storage"
or something like this?

I was wondering if the user wants to flash a H.D. when running a live CD
and then maybe the root may be a right choice.

Since I have started using this patch, it got really faster when it
comes flashing the device (I don't have to remember parameters every
time). I think this will be a really good new feature for this script.

> 
> 
> > Signed-off-by: Bruno Bottazzini 
> > ---
> >   scripts/contrib/mkefidisk.sh | 60 
> > 
> >   1 file changed, 55 insertions(+), 5 deletions(-)
> >
> > diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
> > index 55f72b0..7628de7 100755
> > --- a/scripts/contrib/mkefidisk.sh
> > +++ b/scripts/contrib/mkefidisk.sh
> > @@ -152,8 +152,52 @@ unmount() {
> >   # Parse and validate arguments
> >   #
> >   if [ $# -lt 3 ] || [ $# -gt 4 ]; then
> > -   usage
> > -   exit 1
> > +if [ $# -eq 1 ]; then
> > +AVAILABLE_DISK=`fdisk -l | grep "Disk /" | cut -f 1 -d ':' | cut 
> > -f 2 -d ' '`
> > +if [ -z "$AVAILABLE_DISK" ]; then
> > +die "Starting mkefidisk.sh as root is required"
> > +fi
> > +echo "Available disks are"
> > +X=0
> > +for line in `echo $AVAILABLE_DISK`; do
> > +info "$X - $line"
> > +X=`expr $X + 1`
> > +done
> > +read -p "Choose flashable device number: " DISK_NUMBER
> > +X=0
> > +for line in `echo $AVAILABLE_DISK`; do
> > +if [ $DISK_NUMBER -eq $X ]; then
> > +DISK_TO_BE_FLASHED=$line
> > +break
> > +else
> > +X=`expr $X + 1`
> > +fi
> > +done
> > +if [ -z "$DISK_TO_BE_FLASHED" ]; then
> > +die "Option \"$DISK_NUMBER\" is invalid. Choose a valid option"
> > +else
> > +if [ -z `echo $DISK_TO_BE_FLASHED | grep "mmc"` ]; then
> > +TARGET_TO_BE_BOOT="/dev/sda"
> > +else
> > +TARGET_TO_BE_BOOT="/dev/mmcblk0"
> > +fi
> > +fi
> > +echo ""
> > +echo "Choose a name of the device that will be boot from"
> > +echo -n "Recommended name is: "
> > +info "$TARGET_TO_BE_BOOT\n"
> The \n usage is probably wrong here.
> 
> > +read -p "Is target device okay? [y/N]: " RESPONSE
> > +if [ "$RESPONSE" != "y" ]; then
> > +read -p "Choose target device name: " TARGET_TO_BE_BOOT
> > +fi
> > +echo ""
> > +if [ -z "$TARGET_TO_BE_BOOT" ]; then
> > +die "Error: choose a valid target name"
> > +fi
> > +else
> > +usage
> > +   exit 1
> > +fi
> >   fi
> >
> >   if [ "$1" = "-v" ]; then
> > @@ -162,9 +206,15 @@ if [ "$1" = "-v" ]; then
> > shift
> >   fi
> >
> > -DEVICE=$1
> > -HDDIMG=$2
> > -TARGET_DEVICE=$3
> > +if [ -z "$AVAILABLE_DISK" ]; then
> > +DEVICE=$1
> > +HDDIMG=$2
> > +TARGET_DEVICE=$3
> > +else
> > +DEVICE=$DISK_TO_BE_FLASHED
> > +HDDIMG=$1
> > +TARGET_DEVICE=$TARGET_TO_BE_BOOT
> > +fi
> >
> >   LINK=$(readlink $DEVICE)
> >   if [ $? -eq 0 ]; then
> >


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] scripts: Create interactive menu for mkefidisk script

[Bottazzini, Bruno]

Bump

On Tue, 2015-09-01 at 16:22 -0300, Bruno Bottazzini wrote:
> If it is passed only the image parameter, it will show some interactive
> options to help to conclude the script.
> 
> With this patch it will detect devices available in the machine, asking
> to choose what it is wanted to be flashed.
> 
> it will suggest the target device. If it is a SD card the suggestion
> is /dev/mmcblk0. If it is a pendrive it will suggest to use /dev/sda.
> 
> Signed-off-by: Bruno Bottazzini 
> ---
>  scripts/contrib/mkefidisk.sh | 60 
> 
>  1 file changed, 55 insertions(+), 5 deletions(-)
> 
> diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
> index 55f72b0..7628de7 100755
> --- a/scripts/contrib/mkefidisk.sh
> +++ b/scripts/contrib/mkefidisk.sh
> @@ -152,8 +152,52 @@ unmount() {
>  # Parse and validate arguments
>  #
>  if [ $# -lt 3 ] || [ $# -gt 4 ]; then
> - usage
> - exit 1
> +if [ $# -eq 1 ]; then
> +AVAILABLE_DISK=`fdisk -l | grep "Disk /" | cut -f 1 -d ':' | cut -f 
> 2 -d ' '`
> +if [ -z "$AVAILABLE_DISK" ]; then
> +die "Starting mkefidisk.sh as root is required"
> +fi
> +echo "Available disks are"
> +X=0
> +for line in `echo $AVAILABLE_DISK`; do
> +info "$X - $line"
> +X=`expr $X + 1`
> +done
> +read -p "Choose flashable device number: " DISK_NUMBER
> +X=0
> +for line in `echo $AVAILABLE_DISK`; do
> +if [ $DISK_NUMBER -eq $X ]; then
> +DISK_TO_BE_FLASHED=$line
> +break
> +else
> +X=`expr $X + 1`
> +fi
> +done
> +if [ -z "$DISK_TO_BE_FLASHED" ]; then
> +die "Option \"$DISK_NUMBER\" is invalid. Choose a valid option"
> +else
> +if [ -z `echo $DISK_TO_BE_FLASHED | grep "mmc"` ]; then
> +TARGET_TO_BE_BOOT="/dev/sda"
> +else
> +TARGET_TO_BE_BOOT="/dev/mmcblk0"
> +fi
> +fi
> +echo ""
> +echo "Choose a name of the device that will be boot from"
> +echo -n "Recommended name is: "
> +info "$TARGET_TO_BE_BOOT\n"
> +read -p "Is target device okay? [y/N]: " RESPONSE
> +if [ "$RESPONSE" != "y" ]; then
> +read -p "Choose target device name: " TARGET_TO_BE_BOOT
> +fi
> +echo ""
> +if [ -z "$TARGET_TO_BE_BOOT" ]; then
> +die "Error: choose a valid target name"
> +fi
> +else
> +usage
> + exit 1
> +fi
>  fi
>  
>  if [ "$1" = "-v" ]; then
> @@ -162,9 +206,15 @@ if [ "$1" = "-v" ]; then
>   shift
>  fi
>  
> -DEVICE=$1
> -HDDIMG=$2
> -TARGET_DEVICE=$3
> +if [ -z "$AVAILABLE_DISK" ]; then
> +DEVICE=$1
> +HDDIMG=$2
> +TARGET_DEVICE=$3
> +else
> +DEVICE=$DISK_TO_BE_FLASHED
> +HDDIMG=$1
> +TARGET_DEVICE=$TARGET_TO_BE_BOOT
> +fi
>  
>  LINK=$(readlink $DEVICE)
>  if [ $? -eq 0 ]; then


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH V5 3/3] systemd: split modules into packages

[Bottazzini, Bruno]

Ping

On Sex, 2015-06-05 at 13:52 -0300, Bottazzini, Bruno wrote:
 Ping.
 
 On Ter, 2015-05-19 at 10:18 -0300, Bottazzini, Bruno wrote:
  On Qui, 2015-05-14 at 00:41 +0200, Andreas Oberritter wrote:
   Hello Bruno,
   
   On 13.05.2015 23:51, Bruno Bottazzini wrote:
+
+# Aggregation of Split Packages
+
+PACKAGES =+ ${PN}-services-base
+SUMMARY_${PN}-services-base = Base services aggregation
+ALLOW_EMPTY_${PN}-services-base = 1
+RDEPENDS_${PN}-services-base =  \
   
   I think it would be better to use RRECOMMENDS, in order to support
   BAD_RECOMMENDATIONS per image. This would also remove the need to use
   bb.utils.contains, because unavailable recommended packages get ignored
   by the package managers.
  
  I don't think if we should use RRECOMENDS. 
  
  There are some packages described in services-base that it is not on the
  default PACKAGECONFIG.
  
  With RRECOMENDS and with out bb.utils.contains, it will install all the
  packages that it is described on the recipe and this is not the behavior
  we are looking for.
  
   
   Besides that, I wonder whether ${PN}-services would be a better name,
   because the -base suffix suggests that it contained only the most
   important services.
  
  Sure, we can change it.
  
   
   Btw., sorry for commenting late again, but it's quite a big patch
   
   Regards,
   Andreas
   
+${@bb.utils.contains('PACKAGECONFIG', 'ask-password', 
'${PN}-services-ask-password', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'backlight', 
'${PN}-services-backlight', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'binfmt', 
'${PN}-services-binfmt', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'dbus', 
'${PN}-services-dbus', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'firstboot', 
'${PN}-services-firstboot', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'fuse', 
'${PN}-services-fuse', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'quota', 
'${PN}-services-quota', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'rfkill', 
'${PN}-services-rfkill', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'timesyncd', 
'${PN}-services-timesyncd', '', d)}  \
+${@bb.utils.contains('PACKAGECONFIG', 'update', 
'${PN}-services-update', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'vconsole', 
'${PN}-services-vconsole', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'filesystems', 
'${PN}-generators-filesystems', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'preset', 
'${PN}-preset', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'fsck', 
'${PN}-services-fsck', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'journal', 
'${PN}-services-journal', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'modules-load', 
'${PN}-services-modules-load', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'randomseed', 
'${PN}-services-randomseed', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'sleep', 
'${PN}-services-sleep', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'sysctl', 
'${PN}-services-sysctl', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'sysusers', 
'${PN}-services-sysusers', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'tmpfiles', 
'${PN}-services-tmpfiles', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'udev', 
'${PN}-services-udev', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'tools', '${PN}-tools', 
'', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'bash', '${PN}-bash', 
'', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'initramfs', 
'${PN}-initramfs', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'kernel-install', 
'${PN}-kernel-install', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'rpm-macros', 
'${PN}-rpm-macros', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'zsh', '${PN}-zsh', '', 
d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'bootchart', 
'${PN}-services-bootchart', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'debug', 
'${PN}-services-debug', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'machined', 
'${PN}-services-machined', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'nspawn', 
'${PN}-services-nspawn', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'sysvcompat', 
'${PN}-services-sysvcompat', '', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'getty', 
'${PN}-generators-getty', '', d

Re: [OE-core] [PATCH V5 3/3] systemd: split modules into packages

[Bottazzini, Bruno]

Ping.

On Ter, 2015-05-19 at 10:18 -0300, Bottazzini, Bruno wrote:
 On Qui, 2015-05-14 at 00:41 +0200, Andreas Oberritter wrote:
  Hello Bruno,
  
  On 13.05.2015 23:51, Bruno Bottazzini wrote:
   +
   +# Aggregation of Split Packages
   +
   +PACKAGES =+ ${PN}-services-base
   +SUMMARY_${PN}-services-base = Base services aggregation
   +ALLOW_EMPTY_${PN}-services-base = 1
   +RDEPENDS_${PN}-services-base =  \
  
  I think it would be better to use RRECOMMENDS, in order to support
  BAD_RECOMMENDATIONS per image. This would also remove the need to use
  bb.utils.contains, because unavailable recommended packages get ignored
  by the package managers.
 
 I don't think if we should use RRECOMENDS. 
 
 There are some packages described in services-base that it is not on the
 default PACKAGECONFIG.
 
 With RRECOMENDS and with out bb.utils.contains, it will install all the
 packages that it is described on the recipe and this is not the behavior
 we are looking for.
 
  
  Besides that, I wonder whether ${PN}-services would be a better name,
  because the -base suffix suggests that it contained only the most
  important services.
 
 Sure, we can change it.
 
  
  Btw., sorry for commenting late again, but it's quite a big patch
  
  Regards,
  Andreas
  
   +${@bb.utils.contains('PACKAGECONFIG', 'ask-password', 
   '${PN}-services-ask-password', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'backlight', 
   '${PN}-services-backlight', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'binfmt', 
   '${PN}-services-binfmt', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'dbus', 
   '${PN}-services-dbus', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'firstboot', 
   '${PN}-services-firstboot', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'fuse', 
   '${PN}-services-fuse', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'quota', 
   '${PN}-services-quota', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'rfkill', 
   '${PN}-services-rfkill', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'timesyncd', 
   '${PN}-services-timesyncd', '', d)}  \
   +${@bb.utils.contains('PACKAGECONFIG', 'update', 
   '${PN}-services-update', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'vconsole', 
   '${PN}-services-vconsole', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'filesystems', 
   '${PN}-generators-filesystems', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'preset', '${PN}-preset', 
   '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'fsck', 
   '${PN}-services-fsck', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'journal', 
   '${PN}-services-journal', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'modules-load', 
   '${PN}-services-modules-load', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'randomseed', 
   '${PN}-services-randomseed', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'sleep', 
   '${PN}-services-sleep', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'sysctl', 
   '${PN}-services-sysctl', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'sysusers', 
   '${PN}-services-sysusers', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'tmpfiles', 
   '${PN}-services-tmpfiles', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'udev', 
   '${PN}-services-udev', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'tools', '${PN}-tools', 
   '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'bash', '${PN}-bash', '', 
   d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'initramfs', 
   '${PN}-initramfs', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'kernel-install', 
   '${PN}-kernel-install', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'rpm-macros', 
   '${PN}-rpm-macros', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'zsh', '${PN}-zsh', '', 
   d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'bootchart', 
   '${PN}-services-bootchart', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'debug', 
   '${PN}-services-debug', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'machined', 
   '${PN}-services-machined', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'nspawn', 
   '${PN}-services-nspawn', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'sysvcompat', 
   '${PN}-services-sysvcompat', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'getty', 
   '${PN}-generators-getty', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'getty', 
   '${PN}-services-getty', '', d)} \
   +${@bb.utils.contains('PACKAGECONFIG', 'ldconfig', 
   '${PN}-services-ldconfig

Re: [OE-core] [PATCH V5 3/3] systemd: split modules into packages

[Bottazzini, Bruno]

On Qui, 2015-05-14 at 00:41 +0200, Andreas Oberritter wrote:
 Hello Bruno,
 
 On 13.05.2015 23:51, Bruno Bottazzini wrote:
  +
  +# Aggregation of Split Packages
  +
  +PACKAGES =+ ${PN}-services-base
  +SUMMARY_${PN}-services-base = Base services aggregation
  +ALLOW_EMPTY_${PN}-services-base = 1
  +RDEPENDS_${PN}-services-base =  \
 
 I think it would be better to use RRECOMMENDS, in order to support
 BAD_RECOMMENDATIONS per image. This would also remove the need to use
 bb.utils.contains, because unavailable recommended packages get ignored
 by the package managers.

I don't think if we should use RRECOMENDS. 

There are some packages described in services-base that it is not on the
default PACKAGECONFIG.

With RRECOMENDS and with out bb.utils.contains, it will install all the
packages that it is described on the recipe and this is not the behavior
we are looking for.

 
 Besides that, I wonder whether ${PN}-services would be a better name,
 because the -base suffix suggests that it contained only the most
 important services.

Sure, we can change it.

 
 Btw., sorry for commenting late again, but it's quite a big patch
 
 Regards,
 Andreas
 
  +${@bb.utils.contains('PACKAGECONFIG', 'ask-password', 
  '${PN}-services-ask-password', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'backlight', 
  '${PN}-services-backlight', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'binfmt', 
  '${PN}-services-binfmt', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'dbus', 
  '${PN}-services-dbus', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'firstboot', 
  '${PN}-services-firstboot', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'fuse', 
  '${PN}-services-fuse', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'quota', 
  '${PN}-services-quota', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'rfkill', 
  '${PN}-services-rfkill', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'timesyncd', 
  '${PN}-services-timesyncd', '', d)}  \
  +${@bb.utils.contains('PACKAGECONFIG', 'update', 
  '${PN}-services-update', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'vconsole', 
  '${PN}-services-vconsole', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'filesystems', 
  '${PN}-generators-filesystems', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'preset', '${PN}-preset', 
  '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'fsck', 
  '${PN}-services-fsck', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'journal', 
  '${PN}-services-journal', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'modules-load', 
  '${PN}-services-modules-load', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'randomseed', 
  '${PN}-services-randomseed', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'sleep', 
  '${PN}-services-sleep', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'sysctl', 
  '${PN}-services-sysctl', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'sysusers', 
  '${PN}-services-sysusers', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'tmpfiles', 
  '${PN}-services-tmpfiles', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'udev', 
  '${PN}-services-udev', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'tools', '${PN}-tools', '', 
  d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'bash', '${PN}-bash', '', 
  d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'initramfs', 
  '${PN}-initramfs', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'kernel-install', 
  '${PN}-kernel-install', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'rpm-macros', 
  '${PN}-rpm-macros', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'zsh', '${PN}-zsh', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'bootchart', 
  '${PN}-services-bootchart', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'debug', 
  '${PN}-services-debug', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'machined', 
  '${PN}-services-machined', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'nspawn', 
  '${PN}-services-nspawn', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'sysvcompat', 
  '${PN}-services-sysvcompat', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'getty', 
  '${PN}-generators-getty', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'getty', 
  '${PN}-services-getty', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'ldconfig', 
  '${PN}-services-ldconfig', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'polkit', 
  '${PN}-services-hostnamed', '', d)} \
  +${@bb.utils.contains('PACKAGECONFIG', 'polkit', 
  

Re: [OE-core] [PATCH V3 3/3] systemd: split modules into packages

[Bottazzini, Bruno]

On Seg, 2015-05-11 at 08:06 +0200, Anders Darander wrote:
 * Bottazzini, Bruno bruno.bottazz...@intel.com [150508 00:46]:
 
  On Qui, 2015-05-07 at 20:58 +0200, Andreas Oberritter wrote:
   On 07.05.2015 18:40, Bruno Bottazzini wrote:
+   vconsole \
+   update \
+   timesyncd \
+   rfkill \
+   quota \
+   fuse \
+   firstboot \
+   dbus \
+   binfmt \
+   backlight \
+   ask-password \
 
   I think it may be better to sort this quite long list alphabetically.
 
   Are these options already enabled today? 
 
  Yes, it is enabled. 
 
  This is the default packages enabled in the current recipe.
 
   Because of the new
   packageconfig entries, I would have expected changes to EXTRA_OECONF and
   DEPENDS, which I don't see in your patch.
 
  Some packages does not have EXTRA_OECONF.
 
  You can see what this recipe DEPENDS by looking at ${PN}-services-base.
 
 What Andreas is asking about, is e.g. for the kmod PACKAGECONFIG:
 
+PACKAGECONFIG[kmod] = --enable-kmod,--disable-kmod,kmod
 
 Here, if kmod is enabled in PACKAGECONFIG, kmod (the 3rd parameter),
 will be added to DEPENDS. Thus, you should remove kmod from the static
 DEPENDS (as now it's dynamically added using PACKAGECONFIGS).

You're right. I will make DEPENDS to be dynamically too.

 
 The same is likely true for a number of the other PACKAGECONFIGs that
 you're adding (which, by the way is a _good_ thing to add).

Yeap! I will check the others

Thanks

 
 Cheers,
 Anders
 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH V4 1/3] dbus: split tools package.

[Bottazzini, Bruno]

On Seg, 2015-05-11 at 23:04 +0100, Phil Blundell wrote:
 On Mon, 2015-05-11 at 15:02 -0300, Bruno Bottazzini wrote:
  +FILES_${PN}-tools += ${bindir}/dbus-uuidgen \
 
Hello Phil,

It is a post-install script of a Dbus package.

Best Regards,

Bruno Bottazzini


 Doesn't the dbus startup script use dbus-uuidgen?
 
 p.
 
 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH V3 3/3] systemd: split modules into packages

[Bottazzini, Bruno]

On Qui, 2015-05-07 at 20:58 +0200, Andreas Oberritter wrote:
 On 07.05.2015 18:40, Bruno Bottazzini wrote:
  if one wants to launch a simple deamon, most modules are not
  required.
  He will be able to save space and exclude unwanted packages
  from the final image.
  
  Signed-off-by: Bruno Bottazzini bruno.bottazz...@intel.com
  ---
   meta/recipes-core/systemd/systemd_219.bb | 1063 
  ++
   1 file changed, 938 insertions(+), 125 deletions(-)
  
  diff --git a/meta/recipes-core/systemd/systemd_219.bb 
  b/meta/recipes-core/systemd/systemd_219.bb
  index b4bff18..629c7fc 100644
  --- a/meta/recipes-core/systemd/systemd_219.bb
  +++ b/meta/recipes-core/systemd/systemd_219.bb
  @@ -60,9 +60,55 @@ LDFLAGS_append_libc-uclibc =  -lrt
   GTKDOC_DOCDIR = ${S}/docs/
   
   PACKAGECONFIG ??= xz ldconfig \
  +   getty \
  +   sysvcompat \
  +   nspawn \
  +   machined \
  +   bootchart \
  +   zsh \
  +   rpm-macros \
  +   kernel-install \
  +   initramfs \
  +   bash \
  +   tools \
  +   udev \
  +   tmpfiles \
  +   sysusers \
  +   sysctl \
  +   sleep \
  +   randomseed \
  +   modules-load \
  +   journal \
  +   fsck \
  +   preset \
  +   filesystems \
  +   vconsole \
  +   update \
  +   timesyncd \
  +   rfkill \
  +   quota \
  +   fuse \
  +   firstboot \
  +   dbus \
  +   binfmt \
  +   backlight \
  +   ask-password \
 
 I think it may be better to sort this quite long list alphabetically.
 
 Are these options already enabled today? 

Yes, it is enabled. 

This is the default packages enabled in the current recipe.

 Because of the new
 packageconfig entries, I would have expected changes to EXTRA_OECONF and
 DEPENDS, which I don't see in your patch.

Some packages does not have EXTRA_OECONF.

You can see what this recipe DEPENDS by looking at ${PN}-services-base.

Regards,

 
 Regards,
 Andreas
 
  ${@bb.utils.contains('DISTRO_FEATURES', 'pam', 'pam', 
  '', d)} \
  ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 
  'xkbcommon', '', d)}
   
  +PACKAGECONFIG[glib] = --enable-gudev,--disable-gudev,glib-2.0
  +PACKAGECONFIG[acl] = --enable-acl,--disable-acl,acl
  +PACKAGECONFIG[blkid] = --enable-blkid,--disable-blkid,util-linux
  +PACKAGECONFIG[efi] = --enable-efi,--disable-efi
  +PACKAGECONFIG[kmod] = --enable-kmod,--disable-kmod,kmod
  +PACKAGECONFIG[polkit] = --enable-polkit,--disable-polkit,polkit
  +PACKAGECONFIG[selinux] = --enable-selinux,--disable-selinux,libselinux
  +PACKAGECONFIG[smack] = --enable-smack,--disable-smack
  +PACKAGECONFIG[ima] = --enable-ima,--disable-ima
  +PACKAGECONFIG[apparmor] = 
  --enable-apparmor,--disable-apparmor,libapparmor
  +PACKAGECONFIG[seccomp] = --enable-seccomp,--disable-seccomp,libseccomp
  +PACKAGECONFIG[qrencode] = 
  --enable-qrencode,--disable-qrencode,libqrencode
  +PACKAGECONFIG[lz4] = --enable-lz4,--disable-lz4,lz4
   PACKAGECONFIG[journal-upload] = --enable-libcurl,--disable-libcurl,curl
   # Sign the journal for anti-tampering
   PACKAGECONFIG[gcrypt] = --enable-gcrypt,--disable-gcrypt,libgcrypt
  @@ -206,158 +252,742 @@ do_install_ptest () {
   
   python populate_packages_prepend (){
   systemdlibdir = d.getVar(rootlibdir, True)
  -do_split_packages(d, systemdlibdir, '^lib(.*)\.so\.*', 'lib%s', 
  'Systemd %s library', extra_depends='', allow_links=True)
  +do_split_packages(d, systemdlibdir, 
  '^lib(udev|gudev|systemd|nss)\.so\.*', 'lib%s', 'Systemd %s library', 
  extra_depends='', allow_links=True)
   }
  -PACKAGES_DYNAMIC += ^lib(udev|systemd).*
  +PACKAGES_DYNAMIC += ^lib(udev|gudev|systemd|nss).*
  +
  +
  +# Base Packages
  +
  +
  +PACKAGES =+ ${PN}-generators-filesystems
  +SUMMARY_${PN}-generators-filesystems = Unit generator for /etc/fstab
  +RDEPENDS_${PN}-generators-filesystems = ${PN}-services-fsck
  +FILES_${PN}-generators-filesystems = \
  +
  ${rootlibexecdir}/systemd/system-generators/systemd-fstab-generator \
  +
  ${rootlibexecdir}/systemd/system-generators/systemd-gpt-auto-generator \
  +${rootlibexecdir}/systemd/systemd-remount-fs \
  +
  ${systemd_unitdir}/system/local-fs.target.wants/systemd-remount-fs.service \
  +${systemd_unitdir}/system/systemd-remount-fs.service \
  +
   
  -PACKAGES =+ ${PN}-gui ${PN}-vconsole-setup ${PN}-initramfs 

Re: [OE-core] [PATCH V3 2/3] systemd: Verify if journal-flush.service exists.

[Bottazzini, Bruno]

On Qui, 2015-05-07 at 20:49 +0200, Andreas Oberritter wrote:
 Hello Bruno,
 
 On 07.05.2015 18:40, Bruno Bottazzini wrote:
  If the users wants to enable networkd this file will not be generate and the
  installation will fail.
  
  Signed-off-by: Bruno Bottazzini bruno.bottazz...@intel.com
  ---
   meta/recipes-core/systemd/systemd_219.bb | 7 +--
   1 file changed, 5 insertions(+), 2 deletions(-)
  
  diff --git a/meta/recipes-core/systemd/systemd_219.bb 
  b/meta/recipes-core/systemd/systemd_219.bb
  index c196017..b4bff18 100644
  --- a/meta/recipes-core/systemd/systemd_219.bb
  +++ b/meta/recipes-core/systemd/systemd_219.bb
  @@ -177,8 +177,11 @@ do_install() {
  sed -i -e 's/.*ForwardToSyslog.*/ForwardToSyslog=yes/' 
  ${D}${sysconfdir}/systemd/journald.conf
  # its needed in 216 upstream has fixed it with 
  919699ec301ea507edce4a619141ed22e789ac0d
  # don't order journal flushing afte remote-fs.target
  -   sed -i -e 's/ remote-fs.target$//' 
  ${D}${systemd_unitdir}/system/systemd-journal-flush.service
  -   # this file is needed to exist if networkd is disabled but timesyncd is 
  still in use since timesyncd checks it
  +   if [ -s ${D}${systemd_unitdir}/system/systemd-journal-flush.service ]; 
  then
  +# this file is needed to exist if networkd is disabled but 
  timesyncd is still in use since timesyncd checks it
  +sed -i -e 's/ remote-fs.target$//' 
  ${D}${systemd_unitdir}/system/systemd-journal-flush.service
  +fi
  +
 
 According to the comment above, this is a workaround specific to 216.
 Commit 919699ec301ea507edce4a619141ed22e789ac0d made it into 218. So
 maybe these lines should just get removed.

Andreas,

You are right.

I have checked and tested. It is upstreamed and It is not needed.

I will remove it and send another patch.

Best Regards,

 Regards,
 Andreas


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH V2 3/3] systemd: split modules into packages

[Bottazzini, Bruno]

On Ter, 2015-05-05 at 17:38 -0300, Leandro Dorileo wrote:
 On 05/05/2015 05:01 PM, Anders Darander wrote:
  * Bruno Bottazzini bruno.bottazz...@intel.com [150504 23:43]:
 
  diff --git a/meta/recipes-core/systemd/systemd_219.bb 
  b/meta/recipes-core/systemd/systemd_219.bb
  index b4bff18..2da74dc 100644
  --- a/meta/recipes-core/systemd/systemd_219.bb
  +++ b/meta/recipes-core/systemd/systemd_219.bb
  @@ -60,9 +60,89 @@ LDFLAGS_append_libc-uclibc =  -lrt
GTKDOC_DOCDIR = ${S}/docs/
 
PACKAGECONFIG ??= xz ldconfig \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'gcrypt', 
  'gcrypt', , d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'kmod', 
  'kmod', , d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'blkid', 
  'blkid', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'efi', 'efi', 
  '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'lz4', 'lz4', 
  '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'libidn', 
  'libidn', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'acl', 'acl', 
  '', d)} \
   ${@bb.utils.contains('DISTRO_FEATURES', 'pam', 'pam', 
  '', d)} \
  -   ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 
  'xkbcommon', '', d)}
  -
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'glib', 
  'glib', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'acl', 'acl', 
  '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'blkid', 
  'blkid', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'efi', 'efi', 
  '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'kmod', 
  'kmod', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'polkit', 
  'polkit', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'smack', 
  'smack', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'ima', 'ima', 
  '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'apparmor', 
  'apparmor', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'selinux', 
  'selinux', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'seccomp', 
  'seccomp', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'qrencode', 
  'qrencode', '', d)} \
 
  NAK.
 
  I like the split of systemd into smaller packages, but NAK for this
  version. We're not going to add all this to DISTRO_FEATURES...
 
  Add all the PACKAGCONFIG options, but enable only those that either were
  enabled before, or where we have a real DISTRO_FEATURE to check for.
  Dpn't add a new DISTRO_FEATURE for every PACKAGECONFIG.
 
 
 Just to have it documented here via ML, Anders and I talked on IRC
 and we agreed to add PACKAGECONFIG[feature] definitions for both
 features and have the current features enabled by default without
 using  DISTRO_FEATURES - since we don't want to add these
 DISTRO_FEATURES.
 
 For distros wanting to use the new packages/features we do so by
 adding PACKAGECONFIG_pn-systemd... on the distro side.
 
 Regards...
 
 --
 Dorileo

I agree with the changes as soon as everyone has reviewed I will send a
new one version.

Best Regards,

Bruno Bottazzini


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] upstart init manager

[Bottazzini, Bruno]

Andrea,

I would totally suggest you to use systemd which it does as the same as
upstart but it has more features and it is a lot faster.

However, if you really want to use it, you should verify if you have a
recipe for upstart on your yocto.

If not, you could create it, or search if there is one created. I think
you will find it on meta-freescale or something like this.

Regards,

On Ter, 2015-05-05 at 18:56 +0200, Andrea Di Chiara wrote:
 Hi,
 I'm building my own distribution using Yocto Project and I'm trying to 
 setup Upstart as init manager. In my local.conf I have added these lines:
 
 DISTRO_FEATURES_append = “ upstart”
 VIRTUAL-RUNTIME_init_manager = “upstart”
 
 but no recipe seems to exist. Am I wrong? I have searched through 
 openembedded layers but I found only a recipe for WebOS distribution. Is 
 it possible to use it with Yocto Dora or Yocto Daisy? There is another 
 recipe?
 
 thank you for the attention,
 best regards,
 Andrea
 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] systemd: bring back the patch to customise root's $HOME

[Bottazzini, Bruno]

Thanks Ross!

This slipped trough my fingers when I have remade the first version to
use the stable branch.

Martin,

The patch removed will be applied automatically because they are
upstreamed on the stable branch.

Best Regards,

On Ter, 2015-04-28 at 23:30 +0200, Martin Jansa wrote:
 On Tue, Apr 28, 2015 at 09:12:02PM +0100, Ross Burton wrote:
  This patch appears to have been accidently dropped in the move to 
  219-stable,
  probably because it didn't apply.  Update the patch and re-add it.
 
 what about the unused .patch files reported in:
 http://lists.openembedded.org/pipermail/openembedded-core/2015-April/104197.html
 ?
 
  Signed-off-by: Ross Burton ross.bur...@intel.com
  ---
   .../systemd/systemd/0010-Make-root-s-home-directory-configurable.patch | 3 
  ++-
   meta/recipes-core/systemd/systemd_219.bb   | 1 
  +
   2 files changed, 3 insertions(+), 1 deletion(-)
  
  diff --git 
  a/meta/recipes-core/systemd/systemd/0010-Make-root-s-home-directory-configurable.patch
   
  b/meta/recipes-core/systemd/systemd/0010-Make-root-s-home-directory-configurable.patch
  index 41b9039..b65fe42 100644
  --- 
  a/meta/recipes-core/systemd/systemd/0010-Make-root-s-home-directory-configurable.patch
  +++ 
  b/meta/recipes-core/systemd/systemd/0010-Make-root-s-home-directory-configurable.patch
  @@ -26,10 +26,11 @@ diff --git a/Makefile.am b/Makefile.am
   index 0fb3f9f..4623963 100644
   --- a/Makefile.am
   +++ b/Makefile.am
  -@@ -199,6 +199,7 @@ AM_CPPFLAGS = \
  +@@ -199,7 +199,8 @@ AM_CPPFLAGS = \
  -DKEXEC=\$(KEXEC)\ \
  -DLIBDIR=\$(libdir)\ \
  -DROOTLIBDIR=\$(rootlibdir)\ \
  +   -DROOTLIBEXECDIR=\$(rootlibexecdir)\ \
   +  -DROOTHOMEDIR=\$(roothomedir)\ \
  -DTEST_DIR=\$(abs_top_srcdir)/test\ \
  -I $(top_srcdir)/src \
  diff --git a/meta/recipes-core/systemd/systemd_219.bb 
  b/meta/recipes-core/systemd/systemd_219.bb
  index d1303f7..c196017 100644
  --- a/meta/recipes-core/systemd/systemd_219.bb
  +++ b/meta/recipes-core/systemd/systemd_219.bb
  @@ -38,6 +38,7 @@ SRC_URI = 
  git://anongit.freedesktop.org/systemd/systemd-stable;branch=v219-stab
  file://0007-util-Use-mkostemp-only-if-libc-supports-it.patch \
  
  file://0008-util-bypass-unimplemented-_SC_PHYS_PAGES-system-conf.patch \
  
  file://0009-sysv-generator-add-support-for-executing-scripts-und.patch \
  +   file://0010-Make-root-s-home-directory-configurable.patch \
  file://0011-systemd-user-avoid-using-system-auth.patch \
  
  file://0012-systemd-tmpfiles.c-Honor-ordering-within-files-as-th.patch \
  file://0014-Revert-rules-remove-firmware-loading-rules.patch \
  -- 
  2.1.4
  
  -- 
  ___
  Openembedded-core mailing list
  Openembedded-core@lists.openembedded.org
  http://lists.openembedded.org/mailman/listinfo/openembedded-core
 
 -- 
 Martin 'JaMa' Jansa jabber: martin.ja...@gmail.com


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] systemd: v219 with stable fixes

[Bottazzini, Bruno]

Ping

On Seg, 2015-04-13 at 18:21 -0300, Bottazzini, Bruno wrote:
 Ping
 
 On Qui, 2015-04-09 at 09:57 -0300, Bottazzini, Bruno wrote:
  Good! 
  
  When will this patch be merged with master?
  
  Thanks and best regards
  
  On Ter, 2015-04-07 at 12:33 -0700, Khem Raj wrote:
   this looks ok
   
On Apr 7, 2015, at 6:10 AM, Bottazzini, Bruno 
bruno.bottazz...@intel.com wrote:

Ping

On Qua, 2015-04-01 at 14:14 -0700, Khem Raj wrote:
I will try it out. give me a day

On Apr 1, 2015, at 2:08 PM, Bottazzini, Bruno 
bruno.bottazz...@intel.com wrote:

Khem,

Could you verify this patch and see if this satisfies what you have
pointed previously ?

Do you think this patch is ready to go to master ?

Regards,

On Ter, 2015-03-31 at 17:25 -0300, Otavio Salvador wrote:
I am not a heavy user of systemd so I prefer other people (Khem?) to
comment on this ...

On Tue, Mar 31, 2015 at 5:21 PM, Bottazzini, Bruno
bruno.bottazz...@intel.com wrote:
Ping

On Seg, 2015-03-30 at 15:13 -0300, Bruno Bottazzini wrote:
Adding patches that fix bugs for 219 version.
This will get the same consistency of the stable systemd 219 
version.

More details:
http://cgit.freedesktop.org/systemd/systemd-stable/log/?h=v219-stable
---
...remote-fix-certificate-status-memory-leak.patch |  31 +
...ournal-remote-fix-client_cert-memory-leak.patch |  35 +
...0003-tmpfiles-Fix-parse_acl-error-message.patch |  28 +
...-test-utf8-fix-utf16-tests-on-BE-machines.patch |  26 +
...iles-avoid-creating-duplicate-acl-entries.patch | 131 +++
.../0006-shared-time-util-fix-gcc5-warning.patch   |  32 +
...time-test-infinity-parsing-in-nanoseconds.patch |  36 +
.../0008-bootchart-fix-default-init-path.patch |  44 +
...emctl-bump-NOFILE-only-for-systemctl_main.patch |  44 +
...0-Make-root-s-home-directory-configurable.patch |  89 +-
...-util-avoid-freeing-uninitialized-pointer.patch |  37 +
...11-bootchart-svg-fix-checking-of-list-end.patch |  28 +
...md-add-getrandom-syscall-numbers-for-MIPS.patch |  38 +
...aker-dependencies-between-mount-and-devic.patch |  33 +
...topping-due-to-BindsTo-log-which-unit-cau.patch |  43 +
...grade-message-about-sysctl-overrides-to-d.patch |  30 +
...l-add-some-hints-how-to-override-settings.patch |  39 +
.../0017-core-rework-device-state-logic.patch  | 912 
+
.../0018-core-fix-return-value-on-OOM.patch|  26 +
...e-x-machine-unix-prefix-for-the-container.patch |  33 +
...0-shared-AFS-is-also-a-network-filesystem.patch |  25 +
...downgrade-unit-type-not-supported-message.patch |  31 +
...ournal-remote-fix-saving-of-binary-fields.patch |  97 +++
...ix-Inappropriate-ioctl-for-device-on-ext4.patch |  37 +
...eplace-VLA-with-alloca-to-make-llvm-happy.patch |  53 ++
...ietly-ignore-ACLs-on-unsupported-filesyst.patch |  84 ++
...-assume-ac-when-sys-class-power_supply-is.patch |  30 +
meta/recipes-core/systemd/systemd_219.bb   |  33 +-
28 files changed, 2050 insertions(+), 55 deletions(-)
create mode 100644 
meta/recipes-core/systemd/systemd/0001-journal-remote-fix-certificate-status-memory-leak.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0002-journal-remote-fix-client_cert-memory-leak.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0003-tmpfiles-Fix-parse_acl-error-message.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0004-test-utf8-fix-utf16-tests-on-BE-machines.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0005-tmpfiles-avoid-creating-duplicate-acl-entries.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0006-shared-time-util-fix-gcc5-warning.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0007-test-time-test-infinity-parsing-in-nanoseconds.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0008-bootchart-fix-default-init-path.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0009-systemctl-bump-NOFILE-only-for-systemctl_main.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0010-acl-util-avoid-freeing-uninitialized-pointer.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0011-bootchart-svg-fix-checking-of-list-end.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0012-systemd-add-getrandom-syscall-numbers-for-MIPS.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0013-unit-use-weaker-dependencies-between-mount-and-devic.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0014-unit-When-stopping-due-to-BindsTo-log-which-unit-cau.patch
create mode 100644 
meta/recipes-core/systemd/systemd/0015-sysctl-downgrade-message-about-sysctl-overrides-to-d.patch
create mode

Re: [OE-core] [PATCH 4/4] systemd: v219 with stable fixes

[Bottazzini, Bruno]

Hello Ross,

No problem, I will change the patch to use it the stable branch which I
think it will be better.

I will send this asap...

Thanks a lot for the reply! 

Best Regards,

On Qua, 2015-04-15 at 16:01 +0100, Burton, Ross wrote:
 Hi,
 
 On 1 April 2015 at 13:52, Bottazzini, Bruno
 bruno.bottazz...@intel.com wrote:
 The previous patch was done in the way you said but, Khem Raj
 suggested
 that we should use it as patches.
 
 Copying and Pasting what he said:
 Thats good, although we have to be open on stable branch to
 do SRCREV
  updates which we hardly do. We only backport fixed via master
  so I see this as pointless from that angle.
 
 
 At the time - when master was frozen - that was true.  But
 unfortunately this didn't get applied in time for Fido, as a
 non-trivial patch that doesn't fix existing bugs there's a large
 barrier to pass.  Now that master is open the systemd recipe can in
 fact track the stable git branch instead, and insert the git hash into
 the PV so it's obvious that it's taken from git.
 
 
 Assuming that patch gets merged and doesn't cause problems, then it
 would be up to the Fido maintainer (that's Joshua Lock) to decide
 whether to merge a backport, which could well end up being this patch.
 
 
 Sorry about this, the final stage of the release was quite busy and
 this slipped straight through my fingers.
 
 
 Ross


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 4/4] systemd: v219 with stable fixes

[Bottazzini, Bruno]

On Qua, 2015-04-15 at 16:53 +0100, Burton, Ross wrote:
 
 On 15 April 2015 at 16:37, Bottazzini, Bruno
 bruno.bottazz...@intel.com wrote:
 No problem, I will change the patch to use it the stable
 branch which I
 think it will be better.
 
 I will send this asap...
 
 
 Patches for the stable branch are generally only accepted once they've
 been proven in master, so a patch changing the master systemd to use
 the git branch to verify they don't break something is the first step.

Yes, the master will also add new features and since it is not a
official release may contain bugs. 

So the main proposal is to use the official version of systemd with
backport fixes (stable branch).

In the end we will get the release 219 version with the fixes which is
really good and it'll be more secure.

When the version 220 is released we can change the branch to master with
SRCREV pointing for the release and later we can do the same thing that
I'm trying to do. To add backport fixes for the current version by using
the branch stable or patch files.

 
 
 Ross


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] systemd: v219 with stable fixes

[Bottazzini, Bruno]

Ping

On Qui, 2015-04-09 at 09:57 -0300, Bottazzini, Bruno wrote:
 Good! 
 
 When will this patch be merged with master?
 
 Thanks and best regards
 
 On Ter, 2015-04-07 at 12:33 -0700, Khem Raj wrote:
  this looks ok
  
   On Apr 7, 2015, at 6:10 AM, Bottazzini, Bruno 
   bruno.bottazz...@intel.com wrote:
   
   Ping
   
   On Qua, 2015-04-01 at 14:14 -0700, Khem Raj wrote:
   I will try it out. give me a day
   
   On Apr 1, 2015, at 2:08 PM, Bottazzini, Bruno 
   bruno.bottazz...@intel.com wrote:
   
   Khem,
   
   Could you verify this patch and see if this satisfies what you have
   pointed previously ?
   
   Do you think this patch is ready to go to master ?
   
   Regards,
   
   On Ter, 2015-03-31 at 17:25 -0300, Otavio Salvador wrote:
   I am not a heavy user of systemd so I prefer other people (Khem?) to
   comment on this ...
   
   On Tue, Mar 31, 2015 at 5:21 PM, Bottazzini, Bruno
   bruno.bottazz...@intel.com wrote:
   Ping
   
   On Seg, 2015-03-30 at 15:13 -0300, Bruno Bottazzini wrote:
   Adding patches that fix bugs for 219 version.
   This will get the same consistency of the stable systemd 219 
   version.
   
   More details:
   http://cgit.freedesktop.org/systemd/systemd-stable/log/?h=v219-stable
   ---
   ...remote-fix-certificate-status-memory-leak.patch |  31 +
   ...ournal-remote-fix-client_cert-memory-leak.patch |  35 +
   ...0003-tmpfiles-Fix-parse_acl-error-message.patch |  28 +
   ...-test-utf8-fix-utf16-tests-on-BE-machines.patch |  26 +
   ...iles-avoid-creating-duplicate-acl-entries.patch | 131 +++
   .../0006-shared-time-util-fix-gcc5-warning.patch   |  32 +
   ...time-test-infinity-parsing-in-nanoseconds.patch |  36 +
   .../0008-bootchart-fix-default-init-path.patch |  44 +
   ...emctl-bump-NOFILE-only-for-systemctl_main.patch |  44 +
   ...0-Make-root-s-home-directory-configurable.patch |  89 +-
   ...-util-avoid-freeing-uninitialized-pointer.patch |  37 +
   ...11-bootchart-svg-fix-checking-of-list-end.patch |  28 +
   ...md-add-getrandom-syscall-numbers-for-MIPS.patch |  38 +
   ...aker-dependencies-between-mount-and-devic.patch |  33 +
   ...topping-due-to-BindsTo-log-which-unit-cau.patch |  43 +
   ...grade-message-about-sysctl-overrides-to-d.patch |  30 +
   ...l-add-some-hints-how-to-override-settings.patch |  39 +
   .../0017-core-rework-device-state-logic.patch  | 912 
   +
   .../0018-core-fix-return-value-on-OOM.patch|  26 +
   ...e-x-machine-unix-prefix-for-the-container.patch |  33 +
   ...0-shared-AFS-is-also-a-network-filesystem.patch |  25 +
   ...downgrade-unit-type-not-supported-message.patch |  31 +
   ...ournal-remote-fix-saving-of-binary-fields.patch |  97 +++
   ...ix-Inappropriate-ioctl-for-device-on-ext4.patch |  37 +
   ...eplace-VLA-with-alloca-to-make-llvm-happy.patch |  53 ++
   ...ietly-ignore-ACLs-on-unsupported-filesyst.patch |  84 ++
   ...-assume-ac-when-sys-class-power_supply-is.patch |  30 +
   meta/recipes-core/systemd/systemd_219.bb   |  33 +-
   28 files changed, 2050 insertions(+), 55 deletions(-)
   create mode 100644 
   meta/recipes-core/systemd/systemd/0001-journal-remote-fix-certificate-status-memory-leak.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0002-journal-remote-fix-client_cert-memory-leak.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0003-tmpfiles-Fix-parse_acl-error-message.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0004-test-utf8-fix-utf16-tests-on-BE-machines.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0005-tmpfiles-avoid-creating-duplicate-acl-entries.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0006-shared-time-util-fix-gcc5-warning.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0007-test-time-test-infinity-parsing-in-nanoseconds.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0008-bootchart-fix-default-init-path.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0009-systemctl-bump-NOFILE-only-for-systemctl_main.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0010-acl-util-avoid-freeing-uninitialized-pointer.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0011-bootchart-svg-fix-checking-of-list-end.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0012-systemd-add-getrandom-syscall-numbers-for-MIPS.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0013-unit-use-weaker-dependencies-between-mount-and-devic.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0014-unit-When-stopping-due-to-BindsTo-log-which-unit-cau.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0015-sysctl-downgrade-message-about-sysctl-overrides-to-d.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0016-sysctl-add-some-hints-how-to-override-settings.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0017-core-rework

Re: [OE-core] [PATCH] systemd: v219 with stable fixes

[Bottazzini, Bruno]

Good! 

When will this patch be merged with master?

Thanks and best regards

On Ter, 2015-04-07 at 12:33 -0700, Khem Raj wrote:
 this looks ok
 
  On Apr 7, 2015, at 6:10 AM, Bottazzini, Bruno bruno.bottazz...@intel.com 
  wrote:
  
  Ping
  
  On Qua, 2015-04-01 at 14:14 -0700, Khem Raj wrote:
  I will try it out. give me a day
  
  On Apr 1, 2015, at 2:08 PM, Bottazzini, Bruno 
  bruno.bottazz...@intel.com wrote:
  
  Khem,
  
  Could you verify this patch and see if this satisfies what you have
  pointed previously ?
  
  Do you think this patch is ready to go to master ?
  
  Regards,
  
  On Ter, 2015-03-31 at 17:25 -0300, Otavio Salvador wrote:
  I am not a heavy user of systemd so I prefer other people (Khem?) to
  comment on this ...
  
  On Tue, Mar 31, 2015 at 5:21 PM, Bottazzini, Bruno
  bruno.bottazz...@intel.com wrote:
  Ping
  
  On Seg, 2015-03-30 at 15:13 -0300, Bruno Bottazzini wrote:
  Adding patches that fix bugs for 219 version.
  This will get the same consistency of the stable systemd 219 version.
  
  More details:
  http://cgit.freedesktop.org/systemd/systemd-stable/log/?h=v219-stable
  ---
  ...remote-fix-certificate-status-memory-leak.patch |  31 +
  ...ournal-remote-fix-client_cert-memory-leak.patch |  35 +
  ...0003-tmpfiles-Fix-parse_acl-error-message.patch |  28 +
  ...-test-utf8-fix-utf16-tests-on-BE-machines.patch |  26 +
  ...iles-avoid-creating-duplicate-acl-entries.patch | 131 +++
  .../0006-shared-time-util-fix-gcc5-warning.patch   |  32 +
  ...time-test-infinity-parsing-in-nanoseconds.patch |  36 +
  .../0008-bootchart-fix-default-init-path.patch |  44 +
  ...emctl-bump-NOFILE-only-for-systemctl_main.patch |  44 +
  ...0-Make-root-s-home-directory-configurable.patch |  89 +-
  ...-util-avoid-freeing-uninitialized-pointer.patch |  37 +
  ...11-bootchart-svg-fix-checking-of-list-end.patch |  28 +
  ...md-add-getrandom-syscall-numbers-for-MIPS.patch |  38 +
  ...aker-dependencies-between-mount-and-devic.patch |  33 +
  ...topping-due-to-BindsTo-log-which-unit-cau.patch |  43 +
  ...grade-message-about-sysctl-overrides-to-d.patch |  30 +
  ...l-add-some-hints-how-to-override-settings.patch |  39 +
  .../0017-core-rework-device-state-logic.patch  | 912 
  +
  .../0018-core-fix-return-value-on-OOM.patch|  26 +
  ...e-x-machine-unix-prefix-for-the-container.patch |  33 +
  ...0-shared-AFS-is-also-a-network-filesystem.patch |  25 +
  ...downgrade-unit-type-not-supported-message.patch |  31 +
  ...ournal-remote-fix-saving-of-binary-fields.patch |  97 +++
  ...ix-Inappropriate-ioctl-for-device-on-ext4.patch |  37 +
  ...eplace-VLA-with-alloca-to-make-llvm-happy.patch |  53 ++
  ...ietly-ignore-ACLs-on-unsupported-filesyst.patch |  84 ++
  ...-assume-ac-when-sys-class-power_supply-is.patch |  30 +
  meta/recipes-core/systemd/systemd_219.bb   |  33 +-
  28 files changed, 2050 insertions(+), 55 deletions(-)
  create mode 100644 
  meta/recipes-core/systemd/systemd/0001-journal-remote-fix-certificate-status-memory-leak.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0002-journal-remote-fix-client_cert-memory-leak.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0003-tmpfiles-Fix-parse_acl-error-message.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0004-test-utf8-fix-utf16-tests-on-BE-machines.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0005-tmpfiles-avoid-creating-duplicate-acl-entries.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0006-shared-time-util-fix-gcc5-warning.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0007-test-time-test-infinity-parsing-in-nanoseconds.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0008-bootchart-fix-default-init-path.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0009-systemctl-bump-NOFILE-only-for-systemctl_main.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0010-acl-util-avoid-freeing-uninitialized-pointer.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0011-bootchart-svg-fix-checking-of-list-end.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0012-systemd-add-getrandom-syscall-numbers-for-MIPS.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0013-unit-use-weaker-dependencies-between-mount-and-devic.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0014-unit-When-stopping-due-to-BindsTo-log-which-unit-cau.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0015-sysctl-downgrade-message-about-sysctl-overrides-to-d.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0016-sysctl-add-some-hints-how-to-override-settings.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0017-core-rework-device-state-logic.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0018-core-fix-return-value-on-OOM.patch
  create mode 100644 
  meta/recipes-core/systemd

Re: [OE-core] [PATCH] systemd: v219 with stable fixes

[Bottazzini, Bruno]

Ping

 On Qua, 2015-04-01 at 14:14 -0700, Khem Raj wrote:
  I will try it out. give me a day
  
   On Apr 1, 2015, at 2:08 PM, Bottazzini, Bruno 
   bruno.bottazz...@intel.com wrote:
   
   Khem,
   
   Could you verify this patch and see if this satisfies what you have
   pointed previously ?
   
   Do you think this patch is ready to go to master ?
   
   Regards,
   
   On Ter, 2015-03-31 at 17:25 -0300, Otavio Salvador wrote:
   I am not a heavy user of systemd so I prefer other people (Khem?) to
   comment on this ...
   
   On Tue, Mar 31, 2015 at 5:21 PM, Bottazzini, Bruno
   bruno.bottazz...@intel.com wrote:
   Ping
   
   On Seg, 2015-03-30 at 15:13 -0300, Bruno Bottazzini wrote:
   Adding patches that fix bugs for 219 version.
   This will get the same consistency of the stable systemd 219 version.
   
   More details:
   http://cgit.freedesktop.org/systemd/systemd-stable/log/?h=v219-stable
   ---
   ...remote-fix-certificate-status-memory-leak.patch |  31 +
   ...ournal-remote-fix-client_cert-memory-leak.patch |  35 +
   ...0003-tmpfiles-Fix-parse_acl-error-message.patch |  28 +
   ...-test-utf8-fix-utf16-tests-on-BE-machines.patch |  26 +
   ...iles-avoid-creating-duplicate-acl-entries.patch | 131 +++
   .../0006-shared-time-util-fix-gcc5-warning.patch   |  32 +
   ...time-test-infinity-parsing-in-nanoseconds.patch |  36 +
   .../0008-bootchart-fix-default-init-path.patch |  44 +
   ...emctl-bump-NOFILE-only-for-systemctl_main.patch |  44 +
   ...0-Make-root-s-home-directory-configurable.patch |  89 +-
   ...-util-avoid-freeing-uninitialized-pointer.patch |  37 +
   ...11-bootchart-svg-fix-checking-of-list-end.patch |  28 +
   ...md-add-getrandom-syscall-numbers-for-MIPS.patch |  38 +
   ...aker-dependencies-between-mount-and-devic.patch |  33 +
   ...topping-due-to-BindsTo-log-which-unit-cau.patch |  43 +
   ...grade-message-about-sysctl-overrides-to-d.patch |  30 +
   ...l-add-some-hints-how-to-override-settings.patch |  39 +
   .../0017-core-rework-device-state-logic.patch  | 912 
   +
   .../0018-core-fix-return-value-on-OOM.patch|  26 +
   ...e-x-machine-unix-prefix-for-the-container.patch |  33 +
   ...0-shared-AFS-is-also-a-network-filesystem.patch |  25 +
   ...downgrade-unit-type-not-supported-message.patch |  31 +
   ...ournal-remote-fix-saving-of-binary-fields.patch |  97 +++
   ...ix-Inappropriate-ioctl-for-device-on-ext4.patch |  37 +
   ...eplace-VLA-with-alloca-to-make-llvm-happy.patch |  53 ++
   ...ietly-ignore-ACLs-on-unsupported-filesyst.patch |  84 ++
   ...-assume-ac-when-sys-class-power_supply-is.patch |  30 +
   meta/recipes-core/systemd/systemd_219.bb   |  33 +-
   28 files changed, 2050 insertions(+), 55 deletions(-)
   create mode 100644 
   meta/recipes-core/systemd/systemd/0001-journal-remote-fix-certificate-status-memory-leak.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0002-journal-remote-fix-client_cert-memory-leak.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0003-tmpfiles-Fix-parse_acl-error-message.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0004-test-utf8-fix-utf16-tests-on-BE-machines.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0005-tmpfiles-avoid-creating-duplicate-acl-entries.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0006-shared-time-util-fix-gcc5-warning.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0007-test-time-test-infinity-parsing-in-nanoseconds.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0008-bootchart-fix-default-init-path.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0009-systemctl-bump-NOFILE-only-for-systemctl_main.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0010-acl-util-avoid-freeing-uninitialized-pointer.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0011-bootchart-svg-fix-checking-of-list-end.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0012-systemd-add-getrandom-syscall-numbers-for-MIPS.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0013-unit-use-weaker-dependencies-between-mount-and-devic.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0014-unit-When-stopping-due-to-BindsTo-log-which-unit-cau.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0015-sysctl-downgrade-message-about-sysctl-overrides-to-d.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0016-sysctl-add-some-hints-how-to-override-settings.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0017-core-rework-device-state-logic.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0018-core-fix-return-value-on-OOM.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0019-machined-use-x-machine-unix-prefix-for-the-container.patch
   create mode 100644 
   meta/recipes-core/systemd/systemd/0020-shared-AFS

Re: [OE-core] [PATCH] systemd: v219 with stable fixes

[Bottazzini, Bruno]

Thanks!!! 

On Qua, 2015-04-01 at 14:14 -0700, Khem Raj wrote:
 I will try it out. give me a day
 
  On Apr 1, 2015, at 2:08 PM, Bottazzini, Bruno bruno.bottazz...@intel.com 
  wrote:
  
  Khem,
  
  Could you verify this patch and see if this satisfies what you have
  pointed previously ?
  
  Do you think this patch is ready to go to master ?
  
  Regards,
  
  On Ter, 2015-03-31 at 17:25 -0300, Otavio Salvador wrote:
  I am not a heavy user of systemd so I prefer other people (Khem?) to
  comment on this ...
  
  On Tue, Mar 31, 2015 at 5:21 PM, Bottazzini, Bruno
  bruno.bottazz...@intel.com wrote:
  Ping
  
  On Seg, 2015-03-30 at 15:13 -0300, Bruno Bottazzini wrote:
  Adding patches that fix bugs for 219 version.
  This will get the same consistency of the stable systemd 219 version.
  
  More details:
  http://cgit.freedesktop.org/systemd/systemd-stable/log/?h=v219-stable
  ---
  ...remote-fix-certificate-status-memory-leak.patch |  31 +
  ...ournal-remote-fix-client_cert-memory-leak.patch |  35 +
  ...0003-tmpfiles-Fix-parse_acl-error-message.patch |  28 +
  ...-test-utf8-fix-utf16-tests-on-BE-machines.patch |  26 +
  ...iles-avoid-creating-duplicate-acl-entries.patch | 131 +++
  .../0006-shared-time-util-fix-gcc5-warning.patch   |  32 +
  ...time-test-infinity-parsing-in-nanoseconds.patch |  36 +
  .../0008-bootchart-fix-default-init-path.patch |  44 +
  ...emctl-bump-NOFILE-only-for-systemctl_main.patch |  44 +
  ...0-Make-root-s-home-directory-configurable.patch |  89 +-
  ...-util-avoid-freeing-uninitialized-pointer.patch |  37 +
  ...11-bootchart-svg-fix-checking-of-list-end.patch |  28 +
  ...md-add-getrandom-syscall-numbers-for-MIPS.patch |  38 +
  ...aker-dependencies-between-mount-and-devic.patch |  33 +
  ...topping-due-to-BindsTo-log-which-unit-cau.patch |  43 +
  ...grade-message-about-sysctl-overrides-to-d.patch |  30 +
  ...l-add-some-hints-how-to-override-settings.patch |  39 +
  .../0017-core-rework-device-state-logic.patch  | 912 
  +
  .../0018-core-fix-return-value-on-OOM.patch|  26 +
  ...e-x-machine-unix-prefix-for-the-container.patch |  33 +
  ...0-shared-AFS-is-also-a-network-filesystem.patch |  25 +
  ...downgrade-unit-type-not-supported-message.patch |  31 +
  ...ournal-remote-fix-saving-of-binary-fields.patch |  97 +++
  ...ix-Inappropriate-ioctl-for-device-on-ext4.patch |  37 +
  ...eplace-VLA-with-alloca-to-make-llvm-happy.patch |  53 ++
  ...ietly-ignore-ACLs-on-unsupported-filesyst.patch |  84 ++
  ...-assume-ac-when-sys-class-power_supply-is.patch |  30 +
  meta/recipes-core/systemd/systemd_219.bb   |  33 +-
  28 files changed, 2050 insertions(+), 55 deletions(-)
  create mode 100644 
  meta/recipes-core/systemd/systemd/0001-journal-remote-fix-certificate-status-memory-leak.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0002-journal-remote-fix-client_cert-memory-leak.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0003-tmpfiles-Fix-parse_acl-error-message.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0004-test-utf8-fix-utf16-tests-on-BE-machines.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0005-tmpfiles-avoid-creating-duplicate-acl-entries.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0006-shared-time-util-fix-gcc5-warning.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0007-test-time-test-infinity-parsing-in-nanoseconds.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0008-bootchart-fix-default-init-path.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0009-systemctl-bump-NOFILE-only-for-systemctl_main.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0010-acl-util-avoid-freeing-uninitialized-pointer.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0011-bootchart-svg-fix-checking-of-list-end.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0012-systemd-add-getrandom-syscall-numbers-for-MIPS.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0013-unit-use-weaker-dependencies-between-mount-and-devic.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0014-unit-When-stopping-due-to-BindsTo-log-which-unit-cau.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0015-sysctl-downgrade-message-about-sysctl-overrides-to-d.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0016-sysctl-add-some-hints-how-to-override-settings.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0017-core-rework-device-state-logic.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0018-core-fix-return-value-on-OOM.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0019-machined-use-x-machine-unix-prefix-for-the-container.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd/0020-shared-AFS-is-also-a-network-filesystem.patch
  create mode 100644 
  meta/recipes-core/systemd/systemd

Re: [OE-core] [PATCH] systemd: v219 with stable fixes

[Bottazzini, Bruno]

Khem,

Could you verify this patch and see if this satisfies what you have
pointed previously ?

Do you think this patch is ready to go to master ?

Regards,

On Ter, 2015-03-31 at 17:25 -0300, Otavio Salvador wrote:
 I am not a heavy user of systemd so I prefer other people (Khem?) to
 comment on this ...
 
 On Tue, Mar 31, 2015 at 5:21 PM, Bottazzini, Bruno
 bruno.bottazz...@intel.com wrote:
  Ping
 
  On Seg, 2015-03-30 at 15:13 -0300, Bruno Bottazzini wrote:
  Adding patches that fix bugs for 219 version.
  This will get the same consistency of the stable systemd 219 version.
 
  More details:
  http://cgit.freedesktop.org/systemd/systemd-stable/log/?h=v219-stable
  ---
   ...remote-fix-certificate-status-memory-leak.patch |  31 +
   ...ournal-remote-fix-client_cert-memory-leak.patch |  35 +
   ...0003-tmpfiles-Fix-parse_acl-error-message.patch |  28 +
   ...-test-utf8-fix-utf16-tests-on-BE-machines.patch |  26 +
   ...iles-avoid-creating-duplicate-acl-entries.patch | 131 +++
   .../0006-shared-time-util-fix-gcc5-warning.patch   |  32 +
   ...time-test-infinity-parsing-in-nanoseconds.patch |  36 +
   .../0008-bootchart-fix-default-init-path.patch |  44 +
   ...emctl-bump-NOFILE-only-for-systemctl_main.patch |  44 +
   ...0-Make-root-s-home-directory-configurable.patch |  89 +-
   ...-util-avoid-freeing-uninitialized-pointer.patch |  37 +
   ...11-bootchart-svg-fix-checking-of-list-end.patch |  28 +
   ...md-add-getrandom-syscall-numbers-for-MIPS.patch |  38 +
   ...aker-dependencies-between-mount-and-devic.patch |  33 +
   ...topping-due-to-BindsTo-log-which-unit-cau.patch |  43 +
   ...grade-message-about-sysctl-overrides-to-d.patch |  30 +
   ...l-add-some-hints-how-to-override-settings.patch |  39 +
   .../0017-core-rework-device-state-logic.patch  | 912 
  +
   .../0018-core-fix-return-value-on-OOM.patch|  26 +
   ...e-x-machine-unix-prefix-for-the-container.patch |  33 +
   ...0-shared-AFS-is-also-a-network-filesystem.patch |  25 +
   ...downgrade-unit-type-not-supported-message.patch |  31 +
   ...ournal-remote-fix-saving-of-binary-fields.patch |  97 +++
   ...ix-Inappropriate-ioctl-for-device-on-ext4.patch |  37 +
   ...eplace-VLA-with-alloca-to-make-llvm-happy.patch |  53 ++
   ...ietly-ignore-ACLs-on-unsupported-filesyst.patch |  84 ++
   ...-assume-ac-when-sys-class-power_supply-is.patch |  30 +
   meta/recipes-core/systemd/systemd_219.bb   |  33 +-
   28 files changed, 2050 insertions(+), 55 deletions(-)
   create mode 100644 
  meta/recipes-core/systemd/systemd/0001-journal-remote-fix-certificate-status-memory-leak.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0002-journal-remote-fix-client_cert-memory-leak.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0003-tmpfiles-Fix-parse_acl-error-message.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0004-test-utf8-fix-utf16-tests-on-BE-machines.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0005-tmpfiles-avoid-creating-duplicate-acl-entries.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0006-shared-time-util-fix-gcc5-warning.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0007-test-time-test-infinity-parsing-in-nanoseconds.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0008-bootchart-fix-default-init-path.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0009-systemctl-bump-NOFILE-only-for-systemctl_main.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0010-acl-util-avoid-freeing-uninitialized-pointer.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0011-bootchart-svg-fix-checking-of-list-end.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0012-systemd-add-getrandom-syscall-numbers-for-MIPS.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0013-unit-use-weaker-dependencies-between-mount-and-devic.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0014-unit-When-stopping-due-to-BindsTo-log-which-unit-cau.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0015-sysctl-downgrade-message-about-sysctl-overrides-to-d.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0016-sysctl-add-some-hints-how-to-override-settings.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0017-core-rework-device-state-logic.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0018-core-fix-return-value-on-OOM.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0019-machined-use-x-machine-unix-prefix-for-the-container.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0020-shared-AFS-is-also-a-network-filesystem.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0021-core-downgrade-unit-type-not-supported-message.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0022-journal-remote-fix-saving-of-binary

Re: [OE-core] [PATCH] systemd: v219 with stable fixes

[Bottazzini, Bruno]

Ping

On Seg, 2015-03-30 at 15:13 -0300, Bruno Bottazzini wrote:
 Adding patches that fix bugs for 219 version.
 This will get the same consistency of the stable systemd 219 version.
 
 More details:
 http://cgit.freedesktop.org/systemd/systemd-stable/log/?h=v219-stable
 ---
  ...remote-fix-certificate-status-memory-leak.patch |  31 +
  ...ournal-remote-fix-client_cert-memory-leak.patch |  35 +
  ...0003-tmpfiles-Fix-parse_acl-error-message.patch |  28 +
  ...-test-utf8-fix-utf16-tests-on-BE-machines.patch |  26 +
  ...iles-avoid-creating-duplicate-acl-entries.patch | 131 +++
  .../0006-shared-time-util-fix-gcc5-warning.patch   |  32 +
  ...time-test-infinity-parsing-in-nanoseconds.patch |  36 +
  .../0008-bootchart-fix-default-init-path.patch |  44 +
  ...emctl-bump-NOFILE-only-for-systemctl_main.patch |  44 +
  ...0-Make-root-s-home-directory-configurable.patch |  89 +-
  ...-util-avoid-freeing-uninitialized-pointer.patch |  37 +
  ...11-bootchart-svg-fix-checking-of-list-end.patch |  28 +
  ...md-add-getrandom-syscall-numbers-for-MIPS.patch |  38 +
  ...aker-dependencies-between-mount-and-devic.patch |  33 +
  ...topping-due-to-BindsTo-log-which-unit-cau.patch |  43 +
  ...grade-message-about-sysctl-overrides-to-d.patch |  30 +
  ...l-add-some-hints-how-to-override-settings.patch |  39 +
  .../0017-core-rework-device-state-logic.patch  | 912 
 +
  .../0018-core-fix-return-value-on-OOM.patch|  26 +
  ...e-x-machine-unix-prefix-for-the-container.patch |  33 +
  ...0-shared-AFS-is-also-a-network-filesystem.patch |  25 +
  ...downgrade-unit-type-not-supported-message.patch |  31 +
  ...ournal-remote-fix-saving-of-binary-fields.patch |  97 +++
  ...ix-Inappropriate-ioctl-for-device-on-ext4.patch |  37 +
  ...eplace-VLA-with-alloca-to-make-llvm-happy.patch |  53 ++
  ...ietly-ignore-ACLs-on-unsupported-filesyst.patch |  84 ++
  ...-assume-ac-when-sys-class-power_supply-is.patch |  30 +
  meta/recipes-core/systemd/systemd_219.bb   |  33 +-
  28 files changed, 2050 insertions(+), 55 deletions(-)
  create mode 100644 
 meta/recipes-core/systemd/systemd/0001-journal-remote-fix-certificate-status-memory-leak.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0002-journal-remote-fix-client_cert-memory-leak.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0003-tmpfiles-Fix-parse_acl-error-message.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0004-test-utf8-fix-utf16-tests-on-BE-machines.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0005-tmpfiles-avoid-creating-duplicate-acl-entries.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0006-shared-time-util-fix-gcc5-warning.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0007-test-time-test-infinity-parsing-in-nanoseconds.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0008-bootchart-fix-default-init-path.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0009-systemctl-bump-NOFILE-only-for-systemctl_main.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0010-acl-util-avoid-freeing-uninitialized-pointer.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0011-bootchart-svg-fix-checking-of-list-end.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0012-systemd-add-getrandom-syscall-numbers-for-MIPS.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0013-unit-use-weaker-dependencies-between-mount-and-devic.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0014-unit-When-stopping-due-to-BindsTo-log-which-unit-cau.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0015-sysctl-downgrade-message-about-sysctl-overrides-to-d.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0016-sysctl-add-some-hints-how-to-override-settings.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0017-core-rework-device-state-logic.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0018-core-fix-return-value-on-OOM.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0019-machined-use-x-machine-unix-prefix-for-the-container.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0020-shared-AFS-is-also-a-network-filesystem.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0021-core-downgrade-unit-type-not-supported-message.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0022-journal-remote-fix-saving-of-binary-fields.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0023-journal-fix-Inappropriate-ioctl-for-device-on-ext4.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0024-sd-daemon-replace-VLA-with-alloca-to-make-llvm-happy.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0025-tmpfiles-quietly-ignore-ACLs-on-unsupported-filesyst.patch
  create mode 100644 
 

Re: [OE-core] [PATCH 4/4] systemd: v219 with stable fixes

[Bottazzini, Bruno]

Hello,

What about this patch ?

Did you guys check this out ?

Best Regards,

On Qua, 2015-03-25 at 18:49 -0300, Bruno Bottazzini wrote:
 Adding patches that fix bugs for 219 version.
 This will get the same consistency of the stable systemd 219 version.
 
 More details:
 http://cgit.freedesktop.org/systemd/systemd-stable/log/?h=v219-stable
 ---
  ...remote-fix-certificate-status-memory-leak.patch |  31 +
  ...ournal-remote-fix-client_cert-memory-leak.patch |  35 +
  ...0003-tmpfiles-Fix-parse_acl-error-message.patch |  28 +
  ...-test-utf8-fix-utf16-tests-on-BE-machines.patch |  26 +
  ...iles-avoid-creating-duplicate-acl-entries.patch | 131 +++
  .../0006-shared-time-util-fix-gcc5-warning.patch   |  32 +
  ...time-test-infinity-parsing-in-nanoseconds.patch |  36 +
  .../0008-bootchart-fix-default-init-path.patch |  44 +
  ...emctl-bump-NOFILE-only-for-systemctl_main.patch |  44 +
  ...0-Make-root-s-home-directory-configurable.patch |  89 +-
  ...-util-avoid-freeing-uninitialized-pointer.patch |  37 +
  ...11-bootchart-svg-fix-checking-of-list-end.patch |  28 +
  ...md-add-getrandom-syscall-numbers-for-MIPS.patch |  38 +
  ...aker-dependencies-between-mount-and-devic.patch |  33 +
  ...topping-due-to-BindsTo-log-which-unit-cau.patch |  43 +
  ...grade-message-about-sysctl-overrides-to-d.patch |  30 +
  ...l-add-some-hints-how-to-override-settings.patch |  39 +
  .../0017-core-rework-device-state-logic.patch  | 912 
 +
  .../0018-core-fix-return-value-on-OOM.patch|  26 +
  ...e-x-machine-unix-prefix-for-the-container.patch |  33 +
  ...0-shared-AFS-is-also-a-network-filesystem.patch |  25 +
  ...downgrade-unit-type-not-supported-message.patch |  31 +
  ...ournal-remote-fix-saving-of-binary-fields.patch |  97 +++
  ...ix-Inappropriate-ioctl-for-device-on-ext4.patch |  37 +
  ...eplace-VLA-with-alloca-to-make-llvm-happy.patch |  53 ++
  ...ietly-ignore-ACLs-on-unsupported-filesyst.patch |  84 ++
  ...-assume-ac-when-sys-class-power_supply-is.patch |  30 +
  meta/recipes-core/systemd/systemd_219.bb   |  37 +-
  meta/recipes-core/util-linux/util-linux.inc|   2 +-
  29 files changed, 2052 insertions(+), 59 deletions(-)
  create mode 100644 
 meta/recipes-core/systemd/systemd/0001-journal-remote-fix-certificate-status-memory-leak.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0002-journal-remote-fix-client_cert-memory-leak.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0003-tmpfiles-Fix-parse_acl-error-message.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0004-test-utf8-fix-utf16-tests-on-BE-machines.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0005-tmpfiles-avoid-creating-duplicate-acl-entries.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0006-shared-time-util-fix-gcc5-warning.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0007-test-time-test-infinity-parsing-in-nanoseconds.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0008-bootchart-fix-default-init-path.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0009-systemctl-bump-NOFILE-only-for-systemctl_main.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0010-acl-util-avoid-freeing-uninitialized-pointer.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0011-bootchart-svg-fix-checking-of-list-end.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0012-systemd-add-getrandom-syscall-numbers-for-MIPS.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0013-unit-use-weaker-dependencies-between-mount-and-devic.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0014-unit-When-stopping-due-to-BindsTo-log-which-unit-cau.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0015-sysctl-downgrade-message-about-sysctl-overrides-to-d.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0016-sysctl-add-some-hints-how-to-override-settings.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0017-core-rework-device-state-logic.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0018-core-fix-return-value-on-OOM.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0019-machined-use-x-machine-unix-prefix-for-the-container.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0020-shared-AFS-is-also-a-network-filesystem.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0021-core-downgrade-unit-type-not-supported-message.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0022-journal-remote-fix-saving-of-binary-fields.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0023-journal-fix-Inappropriate-ioctl-for-device-on-ext4.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0024-sd-daemon-replace-VLA-with-alloca-to-make-llvm-happy.patch
  create mode 100644 
 

Re: [OE-core] [PATCH 4/4] systemd: v219 with stable fixes

[Bottazzini, Bruno]

On Seg, 2015-03-30 at 10:40 -0300, Otavio Salvador wrote:
 On Mon, Mar 30, 2015 at 10:05 AM, Bottazzini, Bruno
 bruno.bottazz...@intel.com wrote:
  Hello,
 
  What about this patch ?
 
 What is this change in util-linux? Why this is necessary?

It is not. 

This is some garbage that it shouldn't have been in the patch.

Sorry for this, I will remake and send it separately soon.





-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] systemd: v219 with stable fixes

[Bottazzini, Bruno]

Otavio,

Here it is the patch without garbage that I have told you before.

Could you review it ?

Best Regards,

Bruno Bottazzini

On Seg, 2015-03-30 at 15:13 -0300, Bruno Bottazzini wrote:
 Adding patches that fix bugs for 219 version.
 This will get the same consistency of the stable systemd 219 version.
 
 More details:
 http://cgit.freedesktop.org/systemd/systemd-stable/log/?h=v219-stable
 ---
  ...remote-fix-certificate-status-memory-leak.patch |  31 +
  ...ournal-remote-fix-client_cert-memory-leak.patch |  35 +
  ...0003-tmpfiles-Fix-parse_acl-error-message.patch |  28 +
  ...-test-utf8-fix-utf16-tests-on-BE-machines.patch |  26 +
  ...iles-avoid-creating-duplicate-acl-entries.patch | 131 +++
  .../0006-shared-time-util-fix-gcc5-warning.patch   |  32 +
  ...time-test-infinity-parsing-in-nanoseconds.patch |  36 +
  .../0008-bootchart-fix-default-init-path.patch |  44 +
  ...emctl-bump-NOFILE-only-for-systemctl_main.patch |  44 +
  ...0-Make-root-s-home-directory-configurable.patch |  89 +-
  ...-util-avoid-freeing-uninitialized-pointer.patch |  37 +
  ...11-bootchart-svg-fix-checking-of-list-end.patch |  28 +
  ...md-add-getrandom-syscall-numbers-for-MIPS.patch |  38 +
  ...aker-dependencies-between-mount-and-devic.patch |  33 +
  ...topping-due-to-BindsTo-log-which-unit-cau.patch |  43 +
  ...grade-message-about-sysctl-overrides-to-d.patch |  30 +
  ...l-add-some-hints-how-to-override-settings.patch |  39 +
  .../0017-core-rework-device-state-logic.patch  | 912 
 +
  .../0018-core-fix-return-value-on-OOM.patch|  26 +
  ...e-x-machine-unix-prefix-for-the-container.patch |  33 +
  ...0-shared-AFS-is-also-a-network-filesystem.patch |  25 +
  ...downgrade-unit-type-not-supported-message.patch |  31 +
  ...ournal-remote-fix-saving-of-binary-fields.patch |  97 +++
  ...ix-Inappropriate-ioctl-for-device-on-ext4.patch |  37 +
  ...eplace-VLA-with-alloca-to-make-llvm-happy.patch |  53 ++
  ...ietly-ignore-ACLs-on-unsupported-filesyst.patch |  84 ++
  ...-assume-ac-when-sys-class-power_supply-is.patch |  30 +
  meta/recipes-core/systemd/systemd_219.bb   |  33 +-
  28 files changed, 2050 insertions(+), 55 deletions(-)
  create mode 100644 
 meta/recipes-core/systemd/systemd/0001-journal-remote-fix-certificate-status-memory-leak.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0002-journal-remote-fix-client_cert-memory-leak.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0003-tmpfiles-Fix-parse_acl-error-message.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0004-test-utf8-fix-utf16-tests-on-BE-machines.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0005-tmpfiles-avoid-creating-duplicate-acl-entries.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0006-shared-time-util-fix-gcc5-warning.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0007-test-time-test-infinity-parsing-in-nanoseconds.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0008-bootchart-fix-default-init-path.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0009-systemctl-bump-NOFILE-only-for-systemctl_main.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0010-acl-util-avoid-freeing-uninitialized-pointer.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0011-bootchart-svg-fix-checking-of-list-end.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0012-systemd-add-getrandom-syscall-numbers-for-MIPS.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0013-unit-use-weaker-dependencies-between-mount-and-devic.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0014-unit-When-stopping-due-to-BindsTo-log-which-unit-cau.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0015-sysctl-downgrade-message-about-sysctl-overrides-to-d.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0016-sysctl-add-some-hints-how-to-override-settings.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0017-core-rework-device-state-logic.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0018-core-fix-return-value-on-OOM.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0019-machined-use-x-machine-unix-prefix-for-the-container.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0020-shared-AFS-is-also-a-network-filesystem.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0021-core-downgrade-unit-type-not-supported-message.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0022-journal-remote-fix-saving-of-binary-fields.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0023-journal-fix-Inappropriate-ioctl-for-device-on-ext4.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0024-sd-daemon-replace-VLA-with-alloca-to-make-llvm-happy.patch
  create mode 100644 
 

Re: [OE-core] [PATCH 3/4] systemd: split modules into packages

[Bottazzini, Bruno]

On Sex, 2015-03-27 at 14:11 -0300, Otavio Salvador wrote:
 On Fri, Mar 27, 2015 at 2:09 PM, Bottazzini, Bruno
 bruno.bottazz...@intel.com wrote:
  On Sex, 2015-03-27 at 15:57 +0100, Anders Darander wrote:
  * Bottazzini, Bruno bruno.bottazz...@intel.com [150326 14:40]:
 
   On Qui, 2015-03-26 at 08:56 -0300, Otavio Salvador wrote:
On Thu, Mar 26, 2015 at 5:29 AM, Anders Darander 
and...@chargestorm.se wrote:
 * Bruno Bottazzini bruno.bottazz...@intel.com [150325 22:50]:
 
 if one wants to launch a simple deamon, most modules are not
 required.
 He will be able to save space and exclude unwanted packages
 from the final image.
 
 I like this, though I've got a few questions that I just noticed.
 
 -PACKAGECONFIG ??= xz ldconfig \
 +PACKAGECONFIG ??=  \
 +   gcrypt \
 +   kmod \
 +   ldconfig \
 +   ${@bb.utils.contains('DISTRO_FEATURES', 
 'blkid', 'blkid', '', d)} \
 +   ${@bb.utils.contains('DISTRO_FEATURES', 'efi', 
 'efi', '', d)} \
 +   ${@bb.utils.contains('DISTRO_FEATURES', 'lz4', 
 'lz4', '', d)} \
 +   ${@bb.utils.contains('DISTRO_FEATURES', 'xz', 
 'xz', '', d)} \
 +   ${@bb.utils.contains('DISTRO_FEATURES', 
 'libidn', 'libidn', '', d)} \
 +   ${@bb.utils.contains('DISTRO_FEATURES', 'acl', 
 'acl', '', d)} \
 ${@bb.utils.contains('DISTRO_FEATURES', 'pam', 
 'pam', '', d)} \
 ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 
 'xkbcommon', '', d)}
 
 It might be worth noting that xz has gone from being explicitly 
 enabled,
 to depend on a DISTRO_FEATURES.
 
Agreed and we shouldn't explode the number of possible dsitro
features. I'd also prefer if xz were kept enable by default so we
don't make a behavior change under the hood.
 
...
  PACKAGECONFIG[resolved] = --enable-resolved,--disable-resolved
 -PACKAGECONFIG[networkd] = --enable-networkd,--disable-networkd
 
 Why do you remove networkd as a PACKAGECONFIG?
 
If there is a real reason for this, it must be recorded in commit log 
as well.
 
   Guys, if you continue this patch you will see that networkd will always
   be enabled. Systemd will always configure/make it however, the package
   will not be installed if the user wants to.
 
   With PACKAGECONFIG, we may not get everything for free as some data
   files will be installed regardless as well as some components from
   systemd cannot be disabled by their build system but we can run without
   them, for instance we can run without journald.
 
  The advantage of also keeping the PACKAGECONFIG for e.g. networkd (and
  as much other things as possible) is that we're also reducing the build
  time and size. Sure, it might not be by much, but all small bits are
  valuable.
 
 
  You are right! Maybe we should mix them ?
 
  e.g. if PACKAGECONFIG networkd is disabled then the package
  systemd-services-networkd will not be included.
 
  What do you think?
 
 You can add the files conditionally so these empty packages won't be 
 generated.

Indeed, do you know how to do this condition or have you got any
example?

 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH V2 0/4] systemd and dbus split into packages

[Bottazzini, Bruno]

On Sex, 2015-03-27 at 12:38 +, Burton, Ross wrote:
 
 On 18 March 2015 at 21:40, Bottazzini, Bruno
 bruno.bottazz...@intel.com wrote:
 Sorry, I didn't get why we can't consider this patchset to be
 included
 into master now.
 
 
 Because when Khem said that, master was freezing so that we can make
 the 1.8 release.  We've now branched off the release branch so master
 is open, but the current focus is on the release.
 

Alright! Thanks

 
 Ross


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 3/4] systemd: split modules into packages

[Bottazzini, Bruno]

On Sex, 2015-03-27 at 15:57 +0100, Anders Darander wrote:
 * Bottazzini, Bruno bruno.bottazz...@intel.com [150326 14:40]:
 
  On Qui, 2015-03-26 at 08:56 -0300, Otavio Salvador wrote:
   On Thu, Mar 26, 2015 at 5:29 AM, Anders Darander and...@chargestorm.se 
   wrote:
* Bruno Bottazzini bruno.bottazz...@intel.com [150325 22:50]:
 
if one wants to launch a simple deamon, most modules are not
required.
He will be able to save space and exclude unwanted packages
from the final image.
 
I like this, though I've got a few questions that I just noticed.
 
-PACKAGECONFIG ??= xz ldconfig \
+PACKAGECONFIG ??=  \
+   gcrypt \
+   kmod \
+   ldconfig \
+   ${@bb.utils.contains('DISTRO_FEATURES', 'blkid', 
'blkid', '', d)} \
+   ${@bb.utils.contains('DISTRO_FEATURES', 'efi', 
'efi', '', d)} \
+   ${@bb.utils.contains('DISTRO_FEATURES', 'lz4', 
'lz4', '', d)} \
+   ${@bb.utils.contains('DISTRO_FEATURES', 'xz', 
'xz', '', d)} \
+   ${@bb.utils.contains('DISTRO_FEATURES', 'libidn', 
'libidn', '', d)} \
+   ${@bb.utils.contains('DISTRO_FEATURES', 'acl', 
'acl', '', d)} \
${@bb.utils.contains('DISTRO_FEATURES', 'pam', 
'pam', '', d)} \
${@bb.utils.contains('DISTRO_FEATURES', 'x11', 
'xkbcommon', '', d)}
 
It might be worth noting that xz has gone from being explicitly enabled,
to depend on a DISTRO_FEATURES.
 
   Agreed and we shouldn't explode the number of possible dsitro
   features. I'd also prefer if xz were kept enable by default so we
   don't make a behavior change under the hood.
 
   ...
 PACKAGECONFIG[resolved] = --enable-resolved,--disable-resolved
-PACKAGECONFIG[networkd] = --enable-networkd,--disable-networkd
 
Why do you remove networkd as a PACKAGECONFIG?
 
   If there is a real reason for this, it must be recorded in commit log as 
   well.
 
  Guys, if you continue this patch you will see that networkd will always
  be enabled. Systemd will always configure/make it however, the package
  will not be installed if the user wants to.
 
  With PACKAGECONFIG, we may not get everything for free as some data
  files will be installed regardless as well as some components from
  systemd cannot be disabled by their build system but we can run without
  them, for instance we can run without journald.
 
 The advantage of also keeping the PACKAGECONFIG for e.g. networkd (and
 as much other things as possible) is that we're also reducing the build
 time and size. Sure, it might not be by much, but all small bits are
 valuable.
 

You are right! Maybe we should mix them ? 

e.g. if PACKAGECONFIG networkd is disabled then the package
systemd-services-networkd will not be included.

What do you think?

  The problem is understanding that although systemd is a single
  repository it contains multiple services and daemons in it that can run
  even without the core PID1, udev or the many helpers used to configure
  the system such as resolved, 
  timedated, localed... 
 
 Yes, and as systemd have configure options for quite a lot of those
 things, we should make use of those options, as long as possible.
 
  All of these components are runtime independent, we can install or
  remove them and they should not create problems.
 
 
 That's right, but still, if we even can avoid to build them, it's
 better...
 
 Cheers,
 Anders
 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 3/4] systemd: split modules into packages

[Bottazzini, Bruno]

On Qui, 2015-03-26 at 09:52 +0100, Andreas Oberritter wrote:
 Hi Bruno,
 
 On 25.03.2015 22:49, Bruno Bottazzini wrote:
  if one wants to launch a simple deamon, most modules are not
  required.
  He will be able to save space and exclude unwanted packages
  from the final image.
  ---
   meta/recipes-core/systemd/systemd_219.bb | 1073 
  ++
   1 file changed, 943 insertions(+), 130 deletions(-)
  
  diff --git a/meta/recipes-core/systemd/systemd_219.bb 
  b/meta/recipes-core/systemd/systemd_219.bb
  index e2ea81e..9bb104f 100644
  --- a/meta/recipes-core/systemd/systemd_219.bb
  +++ b/meta/recipes-core/systemd/systemd_219.bb
  @@ -19,7 +19,7 @@ PROVIDES = udev
   
   PE = 1
   
  -DEPENDS = kmod docbook-sgml-dtd-4.1-native intltool-native gperf-native 
  acl readline dbus libcap libcgroup glib-2.0 qemu-native util-linux
  +DEPENDS = intltool-native gperf-native readline libcap libcgroup 
  qemu-native
   
   SECTION = base/shell
   
  @@ -62,23 +62,45 @@ LDFLAGS_append_libc-uclibc =  -lrt
   
   GTKDOC_DOCDIR = ${S}/docs/
   
  -PACKAGECONFIG ??= xz ldconfig \
  +PACKAGECONFIG ??=  \
  +   gcrypt \
  +   kmod \
  +   ldconfig \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'blkid', 
  'blkid', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'efi', 'efi', 
  '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'lz4', 'lz4', 
  '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'xz', 'xz', '', 
  d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'libidn', 
  'libidn', '', d)} \
 
 please don't invent new distro features for packageconfig of a single
 recipe.
 
 Regards,
 Andreas

Andreas,

Firstly, thank you a lot, for reviewing this patch


How would you suggest to change this code ?

Should I do the following ?

PACKAGECONFIG ??=  \
   gcrypt \
   kmod \
   ldconfig \
   blkid \
   efi \
   ... \
an so on ?

Best Regards,

Bruno Bottazzini


 
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'acl', 'acl', 
  '', d)} \
  ${@bb.utils.contains('DISTRO_FEATURES', 'pam', 'pam', 
  '', d)} \
  ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 
  'xkbcommon', '', d)}
   
  +PACKAGECONFIG[glib] = --enable-gudev,--disable-gudev,glib-2.0
  +PACKAGECONFIG[acl] = --enable-acl,--disable-acl,acl
  +PACKAGECONFIG[blkid] = --enable-blkid,--disable-blkid,util-linux
  +PACKAGECONFIG[efi] = --enable-efi,--disable-efi
  +PACKAGECONFIG[kmod] = --enable-kmod,--disable-kmod,kmod
  +PACKAGECONFIG[polkit] = --enable-polkit,--disable-polkit,,polkit
  +PACKAGECONFIG[selinux] = --enable-selinux,--disable-selinux,libselinux
  +PACKAGECONFIG[smack] = --enable-smack,--disable-smack
  +PACKAGECONFIG[ima] = --enable-ima,--disable-ima
  +PACKAGECONFIG[apparmor] = 
  --enable-apparmor,--disable-apparmor,libapparmor
  +PACKAGECONFIG[seccomp] = --enable-seccomp,--disable-seccomp,libseccomp
  +PACKAGECONFIG[gcrypt] = --enable-gcrypt,--disable-gcrypt,libgcrypt
   PACKAGECONFIG[journal-upload] = --enable-libcurl,--disable-libcurl,curl
   # Sign the journal for anti-tampering
   PACKAGECONFIG[gcrypt] = --enable-gcrypt,--disable-gcrypt,libgcrypt
   # regardless of PACKAGECONFIG, libgcrypt is always required to expand
   # the AM_PATH_LIBGCRYPT autoconf macro
  -DEPENDS += libgcrypt
   # Compress the journal
  +PACKAGECONFIG[lz4] = --enable-lz4,--disable-lz4,lz4
   PACKAGECONFIG[xz] = --enable-xz,--disable-xz,xz
  +PACKAGECONFIG[qrencode] = 
  --enable-qrencode,--disable-qrencode,libqrencode
  +PACKAGECONFIG[libidn] = --enable-libidn,--disable-libidn,libidn
   PACKAGECONFIG[cryptsetup] = 
  --enable-libcryptsetup,--disable-libcryptsetup,cryptsetup
   PACKAGECONFIG[microhttpd] = 
  --enable-microhttpd,--disable-microhttpd,libmicrohttpd
   PACKAGECONFIG[elfutils] = --enable-elfutils,--disable-elfutils,elfutils
   PACKAGECONFIG[resolved] = --enable-resolved,--disable-resolved
  -PACKAGECONFIG[networkd] = --enable-networkd,--disable-networkd
   PACKAGECONFIG[libidn] = --enable-libidn,--disable-libidn,libidn
   PACKAGECONFIG[audit] = --enable-audit,--disable-audit,audit
   PACKAGECONFIG[manpages] = 
  --enable-manpages,--disable-manpages,libxslt-native xmlto-native 
  docbook-xml-dtd4-native docbook-xsl-stylesheets-native
  @@ -101,17 +123,36 @@ rootprefix ?= ${base_prefix}
   rootlibdir ?= ${base_libdir}
   rootlibexecdir = ${rootprefix}/lib
   
  -# The gtk+ tools should get built as a separate recipe e.g. systemd-tools
   EXTRA_OECONF =  --with-rootprefix=${rootprefix} \
--with-rootlibdir=${rootlibdir} \
--with-roothomedir=${ROOT_HOME} \
  - --disable-coredump \
  + --enable-coredump \
  + --disable-introspection \
  + 

Re: [OE-core] [PATCH 3/4] systemd: split modules into packages

[Bottazzini, Bruno]

On Qui, 2015-03-26 at 08:56 -0300, Otavio Salvador wrote:
 On Thu, Mar 26, 2015 at 5:29 AM, Anders Darander and...@chargestorm.se 
 wrote:
  * Bruno Bottazzini bruno.bottazz...@intel.com [150325 22:50]:
 
  if one wants to launch a simple deamon, most modules are not
  required.
  He will be able to save space and exclude unwanted packages
  from the final image.
 
  I like this, though I've got a few questions that I just noticed.
 
  -PACKAGECONFIG ??= xz ldconfig \
  +PACKAGECONFIG ??=  \
  +   gcrypt \
  +   kmod \
  +   ldconfig \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'blkid', 
  'blkid', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'efi', 'efi', 
  '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'lz4', 'lz4', 
  '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'xz', 'xz', 
  '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'libidn', 
  'libidn', '', d)} \
  +   ${@bb.utils.contains('DISTRO_FEATURES', 'acl', 'acl', 
  '', d)} \
  ${@bb.utils.contains('DISTRO_FEATURES', 'pam', 'pam', 
  '', d)} \
  ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 
  'xkbcommon', '', d)}
 
  It might be worth noting that xz has gone from being explicitly enabled,
  to depend on a DISTRO_FEATURES.
 
 Agreed and we shouldn't explode the number of possible dsitro
 features. I'd also prefer if xz were kept enable by default so we
 don't make a behavior change under the hood.
 
 ...
   PACKAGECONFIG[resolved] = --enable-resolved,--disable-resolved
  -PACKAGECONFIG[networkd] = --enable-networkd,--disable-networkd
 
  Why do you remove networkd as a PACKAGECONFIG?
 
 If there is a real reason for this, it must be recorded in commit log as well.

Firstly, 

Thank you a lot for reviewing this Patch. I'd like to ask you guys to
review the others patches too.
I know this on will be the more complicated but there others will be
faster to review.


Now:

Guys, if you continue this patch you will see that networkd will always
be enabled. Systemd will always configure/make it however, the package
will not be installed if the user wants to.

With PACKAGECONFIG, we may not get everything for free as some data
files will be installed regardless as well as some components from
systemd cannot be disabled by their build system but we can run without
them, for instance we can run without journald.

The problem is understanding that although systemd is a single
repository it contains multiple services and daemons in it that can run
even without the core PID1, udev or the many helpers used to configure
the system such as resolved, 
timedated, localed... 

All of these components are runtime independent, we can install or
remove them and they should not create problems.

 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 3/4] systemd: split modules into packages

[Bottazzini, Bruno]

On Qui, 2015-03-26 at 10:43 -0300, Otavio Salvador wrote:
 On Thu, Mar 26, 2015 at 10:40 AM, Bottazzini, Bruno
 bruno.bottazz...@intel.com wrote:
  On Qui, 2015-03-26 at 08:56 -0300, Otavio Salvador wrote:
  On Thu, Mar 26, 2015 at 5:29 AM, Anders Darander and...@chargestorm.se 
  wrote:
   * Bruno Bottazzini bruno.bottazz...@intel.com [150325 22:50]:
  
   if one wants to launch a simple deamon, most modules are not
   required.
   He will be able to save space and exclude unwanted packages
   from the final image.
  
   I like this, though I've got a few questions that I just noticed.
  
   -PACKAGECONFIG ??= xz ldconfig \
   +PACKAGECONFIG ??=  \
   +   gcrypt \
   +   kmod \
   +   ldconfig \
   +   ${@bb.utils.contains('DISTRO_FEATURES', 'blkid', 
   'blkid', '', d)} \
   +   ${@bb.utils.contains('DISTRO_FEATURES', 'efi', 
   'efi', '', d)} \
   +   ${@bb.utils.contains('DISTRO_FEATURES', 'lz4', 
   'lz4', '', d)} \
   +   ${@bb.utils.contains('DISTRO_FEATURES', 'xz', 'xz', 
   '', d)} \
   +   ${@bb.utils.contains('DISTRO_FEATURES', 'libidn', 
   'libidn', '', d)} \
   +   ${@bb.utils.contains('DISTRO_FEATURES', 'acl', 
   'acl', '', d)} \
   ${@bb.utils.contains('DISTRO_FEATURES', 'pam', 
   'pam', '', d)} \
   ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 
   'xkbcommon', '', d)}
  
   It might be worth noting that xz has gone from being explicitly enabled,
   to depend on a DISTRO_FEATURES.
 
  Agreed and we shouldn't explode the number of possible dsitro
  features. I'd also prefer if xz were kept enable by default so we
  don't make a behavior change under the hood.
 
  ...
PACKAGECONFIG[resolved] = --enable-resolved,--disable-resolved
   -PACKAGECONFIG[networkd] = --enable-networkd,--disable-networkd
  
   Why do you remove networkd as a PACKAGECONFIG?
 
  If there is a real reason for this, it must be recorded in commit log as 
  well.
 
  Firstly,
 
  Thank you a lot for reviewing this Patch. I'd like to ask you guys to
  review the others patches too.
  I know this on will be the more complicated but there others will be
  faster to review.
 
 
  Now:
 
  Guys, if you continue this patch you will see that networkd will always
  be enabled. Systemd will always configure/make it however, the package
  will not be installed if the user wants to.
 
  With PACKAGECONFIG, we may not get everything for free as some data
  files will be installed regardless as well as some components from
  systemd cannot be disabled by their build system but we can run without
  them, for instance we can run without journald.
 
  The problem is understanding that although systemd is a single
  repository it contains multiple services and daemons in it that can run
  even without the core PID1, udev or the many helpers used to configure
  the system such as resolved,
  timedated, localed...
 
  All of these components are runtime independent, we can install or
  remove them and they should not create problems.
 
 I understand however you can remove the files if networkd, for
 example, is not in PACKAGECONFIG.

Nice! I can adapt the recipe.

How would I do this ?

 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH V2 0/4] systemd and dbus split into packages

[Bottazzini, Bruno]

On Ter, 2015-03-17 at 15:14 -0700, Khem Raj wrote:
 On Tue, Mar 17, 2015 at 11:58 AM, Bottazzini, Bruno
 bruno.bottazz...@intel.com wrote:
  Could you guys look at this patch ?
 
 
 I would hold off this until after 1.8 and consider this for master and
 backport immediately for 1.8.1 release.

Sorry, I didn't get why we can't consider this patchset to be included
into master now. 

I mean, if there is something to be changed I will be glad to change it
until we merge them into master. 

Is there any special reason ?

Thanks for answering.

Best Regards,

Bruno Bottazzini

 
  On Qui, 2015-03-12 at 11:26 -0300, Bottazzini, Bruno wrote:
  Ping
 
  On Seg, 2015-03-09 at 18:10 -0300, Bruno Bottazzini wrote:
   Differences from v1:
  
   dbus: Adding change that Andreas Oberritter recommended.
   -Setting RRECOMMENDS_${PN}_class-native instead RPROVIDES
dbus-native-tools-native.
  
   systemd: Adding change that Anders Darander and Khem Raj recommended
   -Modifying PACKAGECONFIG as requested to use it with bbutils.
  
   systemd: Adding change that Khem Raj requested:
   -Porting patches that was used on the release of the 219 stable version
instead of changing the repository.
  
  
   ---
   In this patchset we include the following:
  
   1- dbus: split tools package
  Most tools are not required if one is launching a simple daemon.
  So, we split it into two packages dbus and dbus-tools which the user
  will be able to exclude it from the image if he wants to.
  
   2- systemd: verify if files test-udev and systemd-journal-flush exists
  If the user put --disable-tests on EXTRA_OECONF for systemd, some 
   files will
  not be created and systemd do_install phase
  will fail.
  This patch will give this flexibility for the user
  
   3- systemd: split modules into packages
  Since systemd repo grows with more and more tools, the recipe will 
   provide
  a customizable layer and he will be able to choose what he wants to 
   ship
  to the final image.
  He will be able to save space and exclude unwanted packages from the
  final image.
  
   4- systemd: add patches from systemd 219 - stable
  Adding patches that fix bugs for 219 version.
  This will get the same consistency of the stable systemd 219 version.
  
  
   Bruno Bottazzini (3):
 systemd: verify if files test-udev and systemd-journal-flush exists
 systemd: split modules into packages
 systemd: v219 with stable fixes
  
   Gustavo Sverzut Barbieri (1):
 dbus: split tools package.
  
   Bruno Bottazzini (3):
 systemd: verify if files test-udev and systemd-journal-flush exists
 systemd: split modules into packages
 systemd: v219 with stable fixes
  
   Gustavo Sverzut Barbieri (1):
 dbus: split tools package.
  
meta/recipes-core/dbus/dbus.inc|9 +-
...remote-fix-certificate-status-memory-leak.patch |   31 +
...ournal-remote-fix-client_cert-memory-leak.patch |   35 +
...0003-tmpfiles-Fix-parse_acl-error-message.patch |   28 +
...-test-utf8-fix-utf16-tests-on-BE-machines.patch |   26 +
...iles-avoid-creating-duplicate-acl-entries.patch |  131 +++
.../0006-shared-time-util-fix-gcc5-warning.patch   |   32 +
...time-test-infinity-parsing-in-nanoseconds.patch |   36 +
.../0008-bootchart-fix-default-init-path.patch |   44 +
...emctl-bump-NOFILE-only-for-systemctl_main.patch |   44 +
...0-Make-root-s-home-directory-configurable.patch |   89 +-
...-util-avoid-freeing-uninitialized-pointer.patch |   37 +
...11-bootchart-svg-fix-checking-of-list-end.patch |   28 +
...md-add-getrandom-syscall-numbers-for-MIPS.patch |   38 +
...aker-dependencies-between-mount-and-devic.patch |   33 +
...topping-due-to-BindsTo-log-which-unit-cau.patch |   43 +
...grade-message-about-sysctl-overrides-to-d.patch |   30 +
...l-add-some-hints-how-to-override-settings.patch |   39 +
.../0017-core-rework-device-state-logic.patch  |  912 
   
.../0018-core-fix-return-value-on-OOM.patch|   26 +
...e-x-machine-unix-prefix-for-the-container.patch |   33 +
...0-shared-AFS-is-also-a-network-filesystem.patch |   25 +
...downgrade-unit-type-not-supported-message.patch |   31 +
...ournal-remote-fix-saving-of-binary-fields.patch |   97 ++
...ix-Inappropriate-ioctl-for-device-on-ext4.patch |   37 +
...eplace-VLA-with-alloca-to-make-llvm-happy.patch |   53 +
...ietly-ignore-ACLs-on-unsupported-filesyst.patch |   84 ++
...-assume-ac-when-sys-class-power_supply-is.patch |   30 +
meta/recipes-core/systemd/systemd_219.bb   |  
   +---
29 files changed, 3006 insertions(+), 186 deletions(-)
create mode 100644 
   meta/recipes-core/systemd/systemd/0001-journal-remote-fix-certificate-status-memory-leak.patch
create mode 100644

Re: [OE-core] [PATCH V2 0/4] systemd and dbus split into packages

[Bottazzini, Bruno]

Could you guys look at this patch ?

On Qui, 2015-03-12 at 11:26 -0300, Bottazzini, Bruno wrote:
 Ping
 
 On Seg, 2015-03-09 at 18:10 -0300, Bruno Bottazzini wrote:
  Differences from v1:
  
  dbus: Adding change that Andreas Oberritter recommended.
  -Setting RRECOMMENDS_${PN}_class-native instead RPROVIDES 
   dbus-native-tools-native.
  
  systemd: Adding change that Anders Darander and Khem Raj recommended
  -Modifying PACKAGECONFIG as requested to use it with bbutils.
  
  systemd: Adding change that Khem Raj requested:
  -Porting patches that was used on the release of the 219 stable version
   instead of changing the repository.
  
  
  ---
  In this patchset we include the following:
  
  1- dbus: split tools package
 Most tools are not required if one is launching a simple daemon.
 So, we split it into two packages dbus and dbus-tools which the user
 will be able to exclude it from the image if he wants to.
  
  2- systemd: verify if files test-udev and systemd-journal-flush exists
 If the user put --disable-tests on EXTRA_OECONF for systemd, some files 
  will
 not be created and systemd do_install phase
 will fail.
 This patch will give this flexibility for the user
  
  3- systemd: split modules into packages
 Since systemd repo grows with more and more tools, the recipe will 
  provide
 a customizable layer and he will be able to choose what he wants to ship
 to the final image.
 He will be able to save space and exclude unwanted packages from the
 final image.
  
  4- systemd: add patches from systemd 219 - stable
 Adding patches that fix bugs for 219 version.
 This will get the same consistency of the stable systemd 219 version.
  
  
  Bruno Bottazzini (3):
systemd: verify if files test-udev and systemd-journal-flush exists
systemd: split modules into packages
systemd: v219 with stable fixes
  
  Gustavo Sverzut Barbieri (1):
dbus: split tools package.
  
  Bruno Bottazzini (3):
systemd: verify if files test-udev and systemd-journal-flush exists
systemd: split modules into packages
systemd: v219 with stable fixes
  
  Gustavo Sverzut Barbieri (1):
dbus: split tools package.
  
   meta/recipes-core/dbus/dbus.inc|9 +-
   ...remote-fix-certificate-status-memory-leak.patch |   31 +
   ...ournal-remote-fix-client_cert-memory-leak.patch |   35 +
   ...0003-tmpfiles-Fix-parse_acl-error-message.patch |   28 +
   ...-test-utf8-fix-utf16-tests-on-BE-machines.patch |   26 +
   ...iles-avoid-creating-duplicate-acl-entries.patch |  131 +++
   .../0006-shared-time-util-fix-gcc5-warning.patch   |   32 +
   ...time-test-infinity-parsing-in-nanoseconds.patch |   36 +
   .../0008-bootchart-fix-default-init-path.patch |   44 +
   ...emctl-bump-NOFILE-only-for-systemctl_main.patch |   44 +
   ...0-Make-root-s-home-directory-configurable.patch |   89 +-
   ...-util-avoid-freeing-uninitialized-pointer.patch |   37 +
   ...11-bootchart-svg-fix-checking-of-list-end.patch |   28 +
   ...md-add-getrandom-syscall-numbers-for-MIPS.patch |   38 +
   ...aker-dependencies-between-mount-and-devic.patch |   33 +
   ...topping-due-to-BindsTo-log-which-unit-cau.patch |   43 +
   ...grade-message-about-sysctl-overrides-to-d.patch |   30 +
   ...l-add-some-hints-how-to-override-settings.patch |   39 +
   .../0017-core-rework-device-state-logic.patch  |  912 
   .../0018-core-fix-return-value-on-OOM.patch|   26 +
   ...e-x-machine-unix-prefix-for-the-container.patch |   33 +
   ...0-shared-AFS-is-also-a-network-filesystem.patch |   25 +
   ...downgrade-unit-type-not-supported-message.patch |   31 +
   ...ournal-remote-fix-saving-of-binary-fields.patch |   97 ++
   ...ix-Inappropriate-ioctl-for-device-on-ext4.patch |   37 +
   ...eplace-VLA-with-alloca-to-make-llvm-happy.patch |   53 +
   ...ietly-ignore-ACLs-on-unsupported-filesyst.patch |   84 ++
   ...-assume-ac-when-sys-class-power_supply-is.patch |   30 +
   meta/recipes-core/systemd/systemd_219.bb   |  
  +---
   29 files changed, 3006 insertions(+), 186 deletions(-)
   create mode 100644 
  meta/recipes-core/systemd/systemd/0001-journal-remote-fix-certificate-status-memory-leak.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0002-journal-remote-fix-client_cert-memory-leak.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0003-tmpfiles-Fix-parse_acl-error-message.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0004-test-utf8-fix-utf16-tests-on-BE-machines.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0005-tmpfiles-avoid-creating-duplicate-acl-entries.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0006-shared-time-util-fix-gcc5-warning.patch
   create mode 100644 
  meta/recipes-core/systemd/systemd/0007-test-time-test-infinity-parsing

Re: [OE-core] [PATCH V2 0/4] systemd and dbus split into packages

[Bottazzini, Bruno]

Ping

On Seg, 2015-03-09 at 18:10 -0300, Bruno Bottazzini wrote:
 Differences from v1:
 
 dbus: Adding change that Andreas Oberritter recommended.
 -Setting RRECOMMENDS_${PN}_class-native instead RPROVIDES 
  dbus-native-tools-native.
 
 systemd: Adding change that Anders Darander and Khem Raj recommended
 -Modifying PACKAGECONFIG as requested to use it with bbutils.
 
 systemd: Adding change that Khem Raj requested:
 -Porting patches that was used on the release of the 219 stable version
  instead of changing the repository.
 
 
 ---
 In this patchset we include the following:
 
 1- dbus: split tools package
Most tools are not required if one is launching a simple daemon.
So, we split it into two packages dbus and dbus-tools which the user
will be able to exclude it from the image if he wants to.
 
 2- systemd: verify if files test-udev and systemd-journal-flush exists
If the user put --disable-tests on EXTRA_OECONF for systemd, some files 
 will
not be created and systemd do_install phase
will fail.
This patch will give this flexibility for the user
 
 3- systemd: split modules into packages
Since systemd repo grows with more and more tools, the recipe will provide
a customizable layer and he will be able to choose what he wants to ship
to the final image.
He will be able to save space and exclude unwanted packages from the
final image.
 
 4- systemd: add patches from systemd 219 - stable
Adding patches that fix bugs for 219 version.
This will get the same consistency of the stable systemd 219 version.
 
 
 Bruno Bottazzini (3):
   systemd: verify if files test-udev and systemd-journal-flush exists
   systemd: split modules into packages
   systemd: v219 with stable fixes
 
 Gustavo Sverzut Barbieri (1):
   dbus: split tools package.
 
 Bruno Bottazzini (3):
   systemd: verify if files test-udev and systemd-journal-flush exists
   systemd: split modules into packages
   systemd: v219 with stable fixes
 
 Gustavo Sverzut Barbieri (1):
   dbus: split tools package.
 
  meta/recipes-core/dbus/dbus.inc|9 +-
  ...remote-fix-certificate-status-memory-leak.patch |   31 +
  ...ournal-remote-fix-client_cert-memory-leak.patch |   35 +
  ...0003-tmpfiles-Fix-parse_acl-error-message.patch |   28 +
  ...-test-utf8-fix-utf16-tests-on-BE-machines.patch |   26 +
  ...iles-avoid-creating-duplicate-acl-entries.patch |  131 +++
  .../0006-shared-time-util-fix-gcc5-warning.patch   |   32 +
  ...time-test-infinity-parsing-in-nanoseconds.patch |   36 +
  .../0008-bootchart-fix-default-init-path.patch |   44 +
  ...emctl-bump-NOFILE-only-for-systemctl_main.patch |   44 +
  ...0-Make-root-s-home-directory-configurable.patch |   89 +-
  ...-util-avoid-freeing-uninitialized-pointer.patch |   37 +
  ...11-bootchart-svg-fix-checking-of-list-end.patch |   28 +
  ...md-add-getrandom-syscall-numbers-for-MIPS.patch |   38 +
  ...aker-dependencies-between-mount-and-devic.patch |   33 +
  ...topping-due-to-BindsTo-log-which-unit-cau.patch |   43 +
  ...grade-message-about-sysctl-overrides-to-d.patch |   30 +
  ...l-add-some-hints-how-to-override-settings.patch |   39 +
  .../0017-core-rework-device-state-logic.patch  |  912 
  .../0018-core-fix-return-value-on-OOM.patch|   26 +
  ...e-x-machine-unix-prefix-for-the-container.patch |   33 +
  ...0-shared-AFS-is-also-a-network-filesystem.patch |   25 +
  ...downgrade-unit-type-not-supported-message.patch |   31 +
  ...ournal-remote-fix-saving-of-binary-fields.patch |   97 ++
  ...ix-Inappropriate-ioctl-for-device-on-ext4.patch |   37 +
  ...eplace-VLA-with-alloca-to-make-llvm-happy.patch |   53 +
  ...ietly-ignore-ACLs-on-unsupported-filesyst.patch |   84 ++
  ...-assume-ac-when-sys-class-power_supply-is.patch |   30 +
  meta/recipes-core/systemd/systemd_219.bb   |  
 +---
  29 files changed, 3006 insertions(+), 186 deletions(-)
  create mode 100644 
 meta/recipes-core/systemd/systemd/0001-journal-remote-fix-certificate-status-memory-leak.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0002-journal-remote-fix-client_cert-memory-leak.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0003-tmpfiles-Fix-parse_acl-error-message.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0004-test-utf8-fix-utf16-tests-on-BE-machines.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0005-tmpfiles-avoid-creating-duplicate-acl-entries.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0006-shared-time-util-fix-gcc5-warning.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0007-test-time-test-infinity-parsing-in-nanoseconds.patch
  create mode 100644 
 meta/recipes-core/systemd/systemd/0008-bootchart-fix-default-init-path.patch
  create mode 100644 
 

Re: [OE-core] [PATCH 3/3] systemd: split modules into packages

[Bottazzini, Bruno]

On Sex, 2015-03-06 at 14:51 -0800, Khem Raj wrote:
 On Fri, Mar 6, 2015 at 6:03 AM, Bottazzini, Bruno
 bruno.bottazz...@intel.com wrote:
  On Sex, 2015-03-06 at 09:23 +0100, Anders Darander wrote:
  * Bottazzini, Bruno bruno.bottazz...@intel.com [150305 17:15]:
 
   On Qui, 2015-03-05 at 15:28 +0100, Anders Darander wrote:
 
Just a quick question before I look into the patch in more detail.
 
Is the new setting of PACKAGECONFIG consistent with how systemd was
built previously? I guess it is.
 
   Hi Anders,
 
   it is consistent with how systemd was built previously. If you apply the
   patch and bitbake it. Systemd will be built and shipped normally.
 
   But now it will give some options on how to customize it by excluding
   packages you don't want to be with systemd.
 
  Nice, I'm really liking this! That's something I've planned on doing
  myself for a while.
 
Another comment, you should remove the dependcies that gets added using
PACKAGECONFIG from DEPENDS, e.g. acl etc. (Or are they required
nevertheless?)
 
   You mean I should do the following ?
   - PACKAGECONFIG[acl] = --enable-acl,--disable-acl,acl
   + PACKAGECONFIG[acl] = --enable-acl,--disable-acl
 
  No, I  meant to remove them from the long
  DEPENDS = kmod docbook-sgml-dtd-4.1-native intltool-native gperf-native 
  acl readline dbus libcap libcgroup glib-2.0 qemu-native util-linux
  line. (Unless I overlooked that part in your patch?)
 
   If I get what you said correctly, yes they are required.
 
  What I meant, was thas unless e.g. acl is required even when building
  with --disable-acl, it's better to add the acl dependency in the
  PACKAGECONFIG like you to. Though, at the same time, remove acl from the
  long DEPENDS-line. (As otherwise we'll build acl anyway).
 
  Anders,
 
  You are right.
 
  It is not needed to specify the libs on depends.
 
  I have removed it and as soon as we review the whole patch I will send a
  new version with this corrected
 
 
 On top of all this 'acl' is a distro feature so that should be
 accounted for too.

Hello Khem,

It will be accounted when acl flag is enabled on systemd.

Best Regards,





-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 3/3] systemd: split modules into packages

[Bottazzini, Bruno]

On Seg, 2015-03-09 at 08:32 -0700, Khem Raj wrote:
  On Mar 9, 2015, at 6:37 AM, Bottazzini, Bruno bruno.bottazz...@intel.com 
  wrote:
  
  On Sex, 2015-03-06 at 14:51 -0800, Khem Raj wrote:
  On Fri, Mar 6, 2015 at 6:03 AM, Bottazzini, Bruno
  bruno.bottazz...@intel.com wrote:
  On Sex, 2015-03-06 at 09:23 +0100, Anders Darander wrote:
  * Bottazzini, Bruno bruno.bottazz...@intel.com [150305 17:15]:
  
  On Qui, 2015-03-05 at 15:28 +0100, Anders Darander wrote:
  
  Just a quick question before I look into the patch in more detail.
  
  Is the new setting of PACKAGECONFIG consistent with how systemd was
  built previously? I guess it is.
  
  Hi Anders,
  
  it is consistent with how systemd was built previously. If you apply the
  patch and bitbake it. Systemd will be built and shipped normally.
  
  But now it will give some options on how to customize it by excluding
  packages you don't want to be with systemd.
  
  Nice, I'm really liking this! That's something I've planned on doing
  myself for a while.
  
  Another comment, you should remove the dependcies that gets added using
  PACKAGECONFIG from DEPENDS, e.g. acl etc. (Or are they required
  nevertheless?)
  
  You mean I should do the following ?
  - PACKAGECONFIG[acl] = --enable-acl,--disable-acl,acl
  + PACKAGECONFIG[acl] = --enable-acl,--disable-acl
  
  No, I  meant to remove them from the long
  DEPENDS = kmod docbook-sgml-dtd-4.1-native intltool-native gperf-native 
  acl readline dbus libcap libcgroup glib-2.0 qemu-native util-linux
  line. (Unless I overlooked that part in your patch?)
  
  If I get what you said correctly, yes they are required.
  
  What I meant, was thas unless e.g. acl is required even when building
  with --disable-acl, it's better to add the acl dependency in the
  PACKAGECONFIG like you to. Though, at the same time, remove acl from the
  long DEPENDS-line. (As otherwise we'll build acl anyway).
  
  Anders,
  
  You are right.
  
  It is not needed to specify the libs on depends.
  
  I have removed it and as soon as we review the whole patch I will send a
  new version with this corrected
  
  
  On top of all this 'acl' is a distro feature so that should be
  accounted for too.
  
  Hello Khem,
  
  It will be accounted when acl flag is enabled on systemd.
  
 
 
 I am saying enabling this packagconfig should depend upon distro features 
 something like
 PACKAGECONFIG ??= ${@bb.utils.contains('DISTRO_FEATURES', ‘acl', ‘acl', '', 
 d)}

Alright, I got what you said and it looks good to me.

Next version will have it.

 
  Best Regards,
  
  
  
  
  
 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] systemd: 219 - stable-219

[Bottazzini, Bruno]

On Sex, 2015-03-06 at 14:49 -0800, Khem Raj wrote:
 On Fri, Mar 6, 2015 at 1:51 PM, Bruno Bottazzini
 bruno.bottazz...@intel.com wrote:
  The upstream systemd git repo only contains the main systemd branch that
  progresses at a quick pace, continuously bringing both bugfixes and new 
  features.
 
  Distributions usually prefer basing their releases on stabilized versions
  that receive the bugfixes but not the features.
 
 Thats good, although we have to be open on stable branch to do SRCREV
 updates which we hardly do. We only backport fixed via master
 so I see this as pointless from that angle.

Alright! Can I get all the patches for the stable version and apply
it ? 
In the end we will get the stable version via master.

What do you think ? Would this be better ?


 
  ---
   ...0-Make-root-s-home-directory-configurable.patch | 89 
  +-
   meta/recipes-core/systemd/systemd_219.bb   |  6 +-
   2 files changed, 40 insertions(+), 55 deletions(-)
 
  diff --git 
  a/meta/recipes-core/systemd/systemd/0010-Make-root-s-home-directory-configurable.patch
   
  b/meta/recipes-core/systemd/systemd/0010-Make-root-s-home-directory-configurable.patch
  index 41b9039..ff2871b 100644
  --- 
  a/meta/recipes-core/systemd/systemd/0010-Make-root-s-home-directory-configurable.patch
  +++ 
  b/meta/recipes-core/systemd/systemd/0010-Make-root-s-home-directory-configurable.patch
  @@ -1,7 +1,7 @@
  -From 3dc731c1d270e2e143de621db9bd898299fd849d Mon Sep 17 00:00:00 2001
  -From: Khem Raj raj.k...@gmail.com
  -Date: Fri, 20 Feb 2015 05:24:49 +
  -Subject: [PATCH 10/11] Make root's home directory configurable
  +From dd843247b6c769a3983462f7e616dc43226974a0 Mon Sep 17 00:00:00 2001
  +From: Bruno Bottazzini bruno.bottazz...@intel.com
  +Date: Thu, 5 Mar 2015 17:38:12 -0300
  +Subject: [PATCH] Make root's home directory configurable
 
   OpenEmbedded has a configurable home directory for root. Allow
   systemd to be built using its idea of what root's home directory
  @@ -9,32 +9,31 @@ should be.
 
   Upstream-Status: Pending
 
  -Signed-off-by: Dan McGregor dan.mcgre...@usask.ca
  -Signed-off-by: Khem Raj raj.k...@gmail.com
  +Patch made by Khem Raj raj.k...@gmail.com updated to be
  +compatbile with systemd 219-stable
   ---
  - Makefile.am   | 2 ++
  - configure.ac  | 7 +++
  - src/core/unit-printf.c| 2 +-
  - src/nspawn/nspawn.c   | 4 ++--
  - src/shared/util.c | 4 ++--
  - units/console-shell.service.m4.in | 4 ++--
  - units/emergency.service.in| 4 ++--
  - units/rescue.service.in   | 4 ++--
  - 8 files changed, 20 insertions(+), 11 deletions(-)
  + Makefile.am| 2 ++
  + configure.ac   | 7 +++
  + src/core/unit-printf.c | 2 +-
  + src/nspawn/nspawn.c| 5 ++---
  + src/shared/util.c  | 4 ++--
  + units/emergency.service.in | 4 ++--
  + units/rescue.service.in| 4 ++--
  + 7 files changed, 18 insertions(+), 10 deletions(-)
 
   diff --git a/Makefile.am b/Makefile.am
  -index 0fb3f9f..4623963 100644
  +index 93c0509..31ea94d 100644
   --- a/Makefile.am
   +++ b/Makefile.am
  -@@ -199,6 +199,7 @@ AM_CPPFLAGS = \
  -   -DKEXEC=\$(KEXEC)\ \
  -   -DLIBDIR=\$(libdir)\ \
  +@@ -201,6 +201,7 @@ AM_CPPFLAGS = \
  -DROOTLIBDIR=\$(rootlibdir)\ \
  -+  -DROOTHOMEDIR=\$(roothomedir)\ \
  +   -DROOTLIBEXECDIR=\$(rootlibexecdir)\ \
  -DTEST_DIR=\$(abs_top_srcdir)/test\ \
  ++  -DROOTHOMEDIR=\$(roothomedir)\ \
  -I $(top_srcdir)/src \
  -I $(top_builddir)/src/shared \
  -@@ -6342,6 +6343,7 @@ EXTRA_DIST += \
  +   -I $(top_srcdir)/src/shared \
  +@@ -6350,6 +6351,7 @@ EXTRA_DIST += \
substitutions = \
   '|rootlibexecdir=$(rootlibexecdir)|' \
   '|rootbindir=$(rootbindir)|' \
  @@ -43,10 +42,10 @@ index 0fb3f9f..4623963 100644
   '|SYSTEMCTL=$(rootbindir)/systemctl|' \
   '|SYSTEMD_NOTIFY=$(rootbindir)/systemd-notify|' \
   diff --git a/configure.ac b/configure.ac
  -index a5b2e6e..55bb7d8 100644
  +index 01ee8dc..800a587 100644
   --- a/configure.ac
   +++ b/configure.ac
  -@@ -1428,6 +1428,11 @@ AC_ARG_WITH([rootlibdir],
  +@@ -1410,6 +1410,11 @@ AC_ARG_WITH([rootlibdir],
[],
[with_rootlibdir=${libdir}])
 
  @@ -58,7 +57,7 @@ index a5b2e6e..55bb7d8 100644
AC_ARG_WITH([pamlibdir],
AS_HELP_STRING([--with-pamlibdir=DIR], [Directory for PAM 
  modules]),
[],
  -@@ -1518,6 +1523,7 @@ AC_SUBST([pamlibdir], [$with_pamlibdir])
  +@@ -1500,6 +1505,7 @@ AC_SUBST([pamlibdir], [$with_pamlibdir])
AC_SUBST([pamconfdir], [$with_pamconfdir])
AC_SUBST([rootprefix], [$with_rootprefix])
AC_SUBST([rootlibdir], [$with_rootlibdir])
  @@ -66,7 +65,7 @@ index a5b2e6e..55bb7d8 100644
 
AC_CONFIG_FILES([
Makefile po/Makefile.in
  -@@ -1617,6 +1623,7 @@ AC_MSG_RESULT([
  +@@ -1599,6 +1605,7 @@ AC_MSG_RESULT([
  

Re: [OE-core] [PATCH 3/3] systemd: split modules into packages

[Bottazzini, Bruno]

On Sex, 2015-03-06 at 09:23 +0100, Anders Darander wrote:
 * Bottazzini, Bruno bruno.bottazz...@intel.com [150305 17:15]:
 
  On Qui, 2015-03-05 at 15:28 +0100, Anders Darander wrote:
 
   Just a quick question before I look into the patch in more detail.
 
   Is the new setting of PACKAGECONFIG consistent with how systemd was
   built previously? I guess it is.
 
  Hi Anders,
 
  it is consistent with how systemd was built previously. If you apply the
  patch and bitbake it. Systemd will be built and shipped normally.
 
  But now it will give some options on how to customize it by excluding
  packages you don't want to be with systemd.
 
 Nice, I'm really liking this! That's something I've planned on doing
 myself for a while.
 
   Another comment, you should remove the dependcies that gets added using
   PACKAGECONFIG from DEPENDS, e.g. acl etc. (Or are they required
   nevertheless?)
 
  You mean I should do the following ?
  - PACKAGECONFIG[acl] = --enable-acl,--disable-acl,acl
  + PACKAGECONFIG[acl] = --enable-acl,--disable-acl
 
 No, I  meant to remove them from the long 
 DEPENDS = kmod docbook-sgml-dtd-4.1-native intltool-native gperf-native acl 
 readline dbus libcap libcgroup glib-2.0 qemu-native util-linux
 line. (Unless I overlooked that part in your patch?)
 
  If I get what you said correctly, yes they are required. 
 
 What I meant, was thas unless e.g. acl is required even when building
 with --disable-acl, it's better to add the acl dependency in the
 PACKAGECONFIG like you to. Though, at the same time, remove acl from the
 long DEPENDS-line. (As otherwise we'll build acl anyway).

Anders,

You are right. 

It is not needed to specify the libs on depends. 

I have removed it and as soon as we review the whole patch I will send a
new version with this corrected

Best Regards,
Bruno Bottazzini
 
 Cheers,
 Anders
 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/3] dbus: split tools package.

[Bottazzini, Bruno]

On Qui, 2015-03-05 at 15:49 +0100, Andreas Oberritter wrote:
 On 05.03.2015 15:29, Bottazzini, Bruno wrote:
  I receive the following problem:
  ERROR: Nothing RPROVIDES 'dbus-native-tools-native' (but
  virtual:native:PATH/poky/meta/recipes-core/dbus/dbus_1.8.10.bb RDEPENDS
  on or otherwise requires it
  
  Any suggestion of how to fix it?
 
 Try setting RRECOMMENDS_${PN}_class-native = .

Nice! it worked perfectly and looks better, indeed.
As soon as we review the patch set, I will send a V2 with this
correction.

 
 I still wonder why -native appears twice inside this dependency, but I
 fail to spot the cause inside the dbus recipe.

You are right, it is strange.

 
 Regards,
 Andreas


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 3/3] systemd: split modules into packages

[Bottazzini, Bruno]

On Qui, 2015-03-05 at 15:28 +0100, Anders Darander wrote:
 * Bruno Bottazzini bruno.bottazz...@intel.com [150304 22:28]:
  if one wants to launch a simple deamon, most modules are not
  required.
  He will be able to save space and exclude unwanted packages
  from the final image.
  ---
   meta/recipes-core/systemd/systemd_219.bb | 1071 
  ++
   1 file changed, 942 insertions(+), 129 deletions(-)
 
  diff --git a/meta/recipes-core/systemd/systemd_219.bb 
  b/meta/recipes-core/systemd/systemd_219.bb
  index 6a8b5b8..fd6cf57 100644
  --- a/meta/recipes-core/systemd/systemd_219.bb
  +++ b/meta/recipes-core/systemd/systemd_219.bb
  @@ -56,10 +56,31 @@ LDFLAGS_append_libc-uclibc =  -lrt
 
   GTKDOC_DOCDIR = ${S}/docs/
 
  -PACKAGECONFIG ??= xz \
  +DEPENDS += curl
  +
  +PACKAGECONFIG ??= acl \
  +   blkid \
  +   efi \
  +   kmod \
  +   gcrypt \
  +   lz4 \
  +   xz \
  +   libidn \
  ${@bb.utils.contains('DISTRO_FEATURES', 'pam', 'pam', 
  '', d)} \
  ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 
  'xkbcommon', '', d)}
 
 Just a quick question before I look into the patch in more detail.
 
 Is the new setting of PACKAGECONFIG consistent with how systemd was
 built previously? I guess it is.

Hi Anders,

it is consistent with how systemd was built previously. If you apply the
patch and bitbake it. Systemd will be built and shipped normally.

But now it will give some options on how to customize it by excluding
packages you don't want to be with systemd.

 
 Another comment, you should remove the dependcies that gets added using
 PACKAGECONFIG from DEPENDS, e.g. acl etc. (Or are they required
 nevertheless?)

You mean I should do the following ?
- PACKAGECONFIG[acl] = --enable-acl,--disable-acl,acl
+ PACKAGECONFIG[acl] = --enable-acl,--disable-acl

If I get what you said correctly, yes they are required. 

Best Regards,

 
 Cheers,
 Anders
 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/3] dbus: split tools package.

[Bottazzini, Bruno]

On Qui, 2015-03-05 at 15:09 +0100, Andreas Oberritter wrote:
 On 05.03.2015 14:16, Bottazzini, Bruno wrote:
  On Qui, 2015-03-05 at 12:01 +0100, Andreas Oberritter wrote:
  Hello Gustavo,
 
  On 04.03.2015 22:27, Bruno Bottazzini wrote:
  From: Gustavo Sverzut Barbieri gustavo.barbi...@intel.com
 
  Most tools are not required if one is launching a simple daemon.
  The user will be able to exclude dbus-tool and save some space in his
  build
 
  Signed-off-by: Bruno Bottazzini bruno.bottazz...@intel.com
  ---
   meta/recipes-core/dbus/dbus.inc | 10 --
   1 file changed, 8 insertions(+), 2 deletions(-)
 
  diff --git a/meta/recipes-core/dbus/dbus.inc 
  b/meta/recipes-core/dbus/dbus.inc
  index fb5d017..f975dc9 100644
  --- a/meta/recipes-core/dbus/dbus.inc
  +++ b/meta/recipes-core/dbus/dbus.inc
  @@ -39,13 +39,13 @@ CONFFILES_${PN} = ${sysconfdir}/dbus-1/system.conf 
  ${sysconfdir}/dbus-1/session
   
   DEBIANNAME_${PN} = dbus-1
   
  -PACKAGES =+ ${PN}-lib
  +PACKAGES =+ ${PN}-lib ${PN}-tools
   
   OLDPKGNAME = dbus-x11
   OLDPKGNAME_class-nativesdk = 
   
   # for compatibility
  -RPROVIDES_${PN} = ${OLDPKGNAME}
  +RPROVIDES_${PN} = ${OLDPKGNAME} ${PN}-native-tools-native
 
  this line seems to be wrong. What are you trying to solve with it?
 
  Regards,
  Andreas
  
  Hello Andreas, 
  
  Could you be more specific what is wrong with this line ?
  
  Have you tried removing it and compile it ?
 
 No, I haven't. What kind of problem will it raise?

I receive the following problem:
ERROR: Nothing RPROVIDES 'dbus-native-tools-native' (but
virtual:native:PATH/poky/meta/recipes-core/dbus/dbus_1.8.10.bb RDEPENDS
on or otherwise requires it

Any suggestion of how to fix it?
 
 It looks wrong, because the package dbus doesn't provide dbus-tools, but
 dbus-tools is instead provided by dbus-tools itself (with your patch
 applied).
 
 Furthermore, having native twice in a package name seems odd, but
 there's no reason for target packages to provide anything native in
 the first place.
 
 Regards,
 Andreas
 
  Try compiling this patch and see what it provides.
  
  If you want to exclude the dbus-tools just add the following at your 
  local conf file: 
  PACKAGE_EXCLUDE = dbus-tools.
  
  Best Regards,
  
  Bruno Bottazzini
  
 
   RREPLACES_${PN} += ${OLDPKGNAME}
   
   FILES_${PN} = ${bindir}/dbus-daemon* \
  @@ -64,6 +64,12 @@ FILES_${PN} = ${bindir}/dbus-daemon* \
   FILES_${PN}-lib = ${libdir}/lib*.so.*
   RRECOMMENDS_${PN}-lib = ${PN}
   FILES_${PN}-dev += ${libdir}/dbus-1.0/include ${bindir}/dbus-glib-tool
  +FILES_${PN}-tools += ${bindir}/dbus-uuidgen \
  +  ${bindir}/dbus-cleanup-sockets \
  +  ${bindir}/dbus-monitor \
  +  ${bindir}/dbus-launch \
  +  ${bindir}/dbus-run-session
  +RRECOMMENDS_${PN} = ${PN}-tools
   
   pkg_postinst_dbus() {
# If both systemd and sysvinit are enabled, mask the dbus-1 init script
 
 
  
  
 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/3] dbus: split tools package.

[Bottazzini, Bruno]

On Qui, 2015-03-05 at 12:01 +0100, Andreas Oberritter wrote:
 Hello Gustavo,
 
 On 04.03.2015 22:27, Bruno Bottazzini wrote:
  From: Gustavo Sverzut Barbieri gustavo.barbi...@intel.com
  
  Most tools are not required if one is launching a simple daemon.
  The user will be able to exclude dbus-tool and save some space in his
  build
  
  Signed-off-by: Bruno Bottazzini bruno.bottazz...@intel.com
  ---
   meta/recipes-core/dbus/dbus.inc | 10 --
   1 file changed, 8 insertions(+), 2 deletions(-)
  
  diff --git a/meta/recipes-core/dbus/dbus.inc 
  b/meta/recipes-core/dbus/dbus.inc
  index fb5d017..f975dc9 100644
  --- a/meta/recipes-core/dbus/dbus.inc
  +++ b/meta/recipes-core/dbus/dbus.inc
  @@ -39,13 +39,13 @@ CONFFILES_${PN} = ${sysconfdir}/dbus-1/system.conf 
  ${sysconfdir}/dbus-1/session
   
   DEBIANNAME_${PN} = dbus-1
   
  -PACKAGES =+ ${PN}-lib
  +PACKAGES =+ ${PN}-lib ${PN}-tools
   
   OLDPKGNAME = dbus-x11
   OLDPKGNAME_class-nativesdk = 
   
   # for compatibility
  -RPROVIDES_${PN} = ${OLDPKGNAME}
  +RPROVIDES_${PN} = ${OLDPKGNAME} ${PN}-native-tools-native
 
 this line seems to be wrong. What are you trying to solve with it?
 
 Regards,
 Andreas

Hello Andreas, 

Could you be more specific what is wrong with this line ?

Have you tried removing it and compile it ?

Try compiling this patch and see what it provides.

If you want to exclude the dbus-tools just add the following at your 
local conf file: 
PACKAGE_EXCLUDE = dbus-tools.

Best Regards,

Bruno Bottazzini

 
   RREPLACES_${PN} += ${OLDPKGNAME}
   
   FILES_${PN} = ${bindir}/dbus-daemon* \
  @@ -64,6 +64,12 @@ FILES_${PN} = ${bindir}/dbus-daemon* \
   FILES_${PN}-lib = ${libdir}/lib*.so.*
   RRECOMMENDS_${PN}-lib = ${PN}
   FILES_${PN}-dev += ${libdir}/dbus-1.0/include ${bindir}/dbus-glib-tool
  +FILES_${PN}-tools += ${bindir}/dbus-uuidgen \
  +  ${bindir}/dbus-cleanup-sockets \
  +  ${bindir}/dbus-monitor \
  +  ${bindir}/dbus-launch \
  +  ${bindir}/dbus-run-session
  +RRECOMMENDS_${PN} = ${PN}-tools
   
   pkg_postinst_dbus() {
  # If both systemd and sysvinit are enabled, mask the dbus-1 init script
  
 


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH v2 2/2] systemd: split modules into packages

[Bottazzini, Bruno]

Hello Randy,

On Sex, 2015-02-06 at 10:45 -0800, Randy Witt wrote:
 On 02/04/2015 09:04 AM, Bruno Bottazzini wrote:
  It wil be able to choose what systemd module to be installed.
  The final result may get smaller, if the user wanted to.
  By default it will install the whole systemd which may be big.
  ---
meta/recipes-core/systemd/systemd_218.bb | 1059 
  ++
1 file changed, 914 insertions(+), 145 deletions(-)
 
 This patch will push a lot of maintenance work onto the Yocto systemd 
 maintainer 
 every time there is an upgrade. It's also easy to miss dependencies because 
 of 
 the inter-dependencies that can exist between services.

This is not a problem, we can give this maintenance.

 
 For items such as systemd-cgls and the tools, a change like this would be 
 maintainable since those items are self-contained. But yanking out the 
 services 
 into separate packages means the maintainer will have to inspect them on each 
 upgrade to make sure they still work.
 
 Would it be sufficient to use PACKAGECONFIG for each of the items that can be 
 disabled via configure? That way the granularity configuration is done by the 
 systemd maintainers rather than the Yocto maintainer.

With PACKAGECONFIG, we may not get everything for free as some data files 
will be installed regardless as well as some components from systemd cannot 
be disabled by their build system but we can run without them, for instance 
we can run without journald.

The problem is understanding that although systemd is a single repository 
it contains multiple services and daemons in it that can run even without the 
core PID1, udev or the many helpers used to configure the system such as 
resolved, 
timedated, localed... 

All of these components are runtime independent, we can install or remove them 
and 
they should not create problems.

Even Ubuntu is shipping with some of these daemons even if systemd is not used 
(yet).
If they were different GIT repos then it would be clear that they are separate 
daemons, 
but as they are all in a single repository people can make a confusion about it.

I see that in the master branch it has already the version 219.

I will make a split packages for the newest version that was just released.

Best Regards,

Bruno Bottazzini

 
 
 



-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] mkefidisk: change filesystem to be writeable on grub

[Bottazzini, Bruno]

On Qui, 2015-02-26 at 13:44 -0800, Randy Witt wrote:
 On 02/26/2015 11:29 AM, Bruno Bottazzini wrote:
  This patch changes grub configuration so the filesystem will be writeable
  instead, just read only when running the script.
  ---
scripts/contrib/mkefidisk.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
 
  diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
  index b96b7d4..9b84831 100755
  --- a/scripts/contrib/mkefidisk.sh
  +++ b/scripts/contrib/mkefidisk.sh
  @@ -342,7 +342,7 @@ if [ -e $GRUB_CFG ]; then
  sed -i s/ LABEL=[^ ]*/ / $GRUB_CFG
 
  sed -i s@ root=[^ ]*@ @ $GRUB_CFG
  -   sed -i s@vmlinuz @vmlinuz root=$TARGET_ROOTFS ro rootwait quiet @ 
  $GRUB_CFG
  +   sed -i s@vmlinuz @vmlinuz root=$TARGET_ROOTFS rw rootwait quiet @ 
  $GRUB_CFG
fi
 
# Look for a gummiboot installation
 
 
 This change basically reverts 
 http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=24cd3ddeb70a3d3f2985ec71c87ae0108f3d2777
  
 by Darren. It also doesn't change the ro option when using gummiboot instead.
 
 What is this change trying to fix?

Hello Randy,

I got the following message:
 file system is read-only 

if I try to touch a file that is not on /tmp, or when I start a systemd
service that tries to write on the file system.

After some research, I go it fixed by changing to read/write on the
script that was with read only.

I thought this would be useful for someone else.

Reading the patch you sent me, Darren says that a proper fstab will ensure
the rootfs is rw. 
Maybe I'm missing this. I will look for this to make sure that my fstab OK.

Thanks




-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] systemd: update version from 216 to 218

[Bottazzini, Bruno]

On Wed, 4 Feb 2015 12:48:35 +0100
Enrico Scholz enrico.sch...@sigma-chemnitz.de wrote:

 Bruno Bottazzini
 bruno.bottazzini-ral2jqcrhueavxtiumw...@public.gmane.org writes:
 
  +PACKAGES =+ ${PN}-services-ask-password
  +SUMMARY_${PN}-services-ask-password = systemd's service and tool
  to query the user for a system password
  +RRECOMMENDS_${PN}-services-ask-password = ${PN}-services-base
  +FILES_${PN}-services-ask-password = \
  +${rootprefix}/bin/systemd-ask-password \
  +${rootprefix}/bin/systemd-tty-ask-password-agent \
  +
  ${systemd_unitdir}/system/multi-user.target.wants/systemd-ask-password-wall.path
  \
  +
  ${systemd_unitdir}/system/sysinit.target.wants/systemd-ask-password-console.path
  \
  +
  ${systemd_unitdir}/system/systemd-ask-password-console.path \
  +
  ${systemd_unitdir}/system/systemd-ask-password-console.service \
  +${systemd_unitdir}/system/systemd-ask-password-wall.path \
  +
  ${systemd_unitdir}/system/systemd-ask-password-wall.service \ +
 
 I think, enumerating services in this kind becomes a maintenance
 nightmare.  E.g. you have to check for every future systemd version
 whether service gets listed in a new .wants.  Using wildcards can
 simplify this; e.g. see systemd_elito_populate_packages() in
 
   https://www.cvg.de/people/ensc/systemd_216.bbappend
 
 
 Enrico

Enrico,

On your example, wouldn't we have to enumerate the packages too ? 
But now it is on a different place.

Where did you get this file ? I looked into master branch but I
couldn't find it.

Thanks,

Bruno Bottazzini


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] systemd: update version from 216 to 218

[Bottazzini, Bruno]

On Tue, 3 Feb 2015 13:23:32 -0800
Randy Witt randy.e.w...@linux.intel.com wrote:

 On 02/03/2015 10:21 AM, Bruno Bottazzini wrote:
  Spliting into packages to be able to choose what to be installed.
  The final result may get smaller, if the user wanted to.
  By default it will install the whole systemd which may be big.
  ---
...r-executing-scripts-under-etc-systemd-218.patch |  131 ++
.../systemd/systemd_218-pam-fix-fallocate.patch|   91 ++
meta/recipes-core/systemd/systemd_218.bb   | 1251
   3 files changed, 1473 insertions(+)
create mode 100644
  meta/recipes-core/systemd/systemd/0001-add-support-for-executing-scripts-under-etc-systemd-218.patch
  create mode 100644
  meta/recipes-core/systemd/systemd/systemd_218-pam-fix-fallocate.patch
  create mode 100644 meta/recipes-core/systemd/systemd_218.bb
 
  diff --git
  a/meta/recipes-core/systemd/systemd/0001-add-support-for-executing-scripts-under-etc-systemd-218.patch
  b/meta/recipes-core/systemd/systemd/0001-add-support-for-executing-scripts-under-etc-systemd-218.patch
  new file mode 100644 index 000..d50f2cb --- /dev/null
  +++
  b/meta/recipes-core/systemd/systemd/0001-add-support-for-executing-scripts-under-etc-systemd-218.patch
  @@ -0,0 +1,131 @@ +From 0dec519c563654148d3cdd363d2598b50313de60
  Mon Sep 17 00:00:00 2001 +From: Bruno Bottazzini
  bruno.bottazz...@intel.com +Date: Mon, 2 Feb 2015 13:53:24 -0200
  +Subject: [PATCH 1/1] add support for executing scripts
  under /etc/rcS.d/ +
  +To be compatible, all services translated from scripts
  under /etc/rcS.d would +run before services translated from scripts
  under /etc/rcN.d. +---
  + src/sysv-generator/sysv-generator.c | 46
  -
  + 1 file changed, 35 insertions(+), 11 deletions(-)
  +
  +diff --git a/src/sysv-generator/sysv-generator.c
  b/src/sysv-generator/sysv-generator.c +index b8b77aa..9494afb 100644
  +--- a/src/sysv-generator/sysv-generator.c
   b/src/sysv-generator/sysv-generator.c
  +@@ -42,7 +42,8 @@
  +
  + typedef enum RunlevelType {
  + RUNLEVEL_UP,
  +-RUNLEVEL_DOWN
  ++RUNLEVEL_DOWN,
  ++RUNLEVEL_SYSINIT
  + } RunlevelType;
  +
  + static const struct {
  +@@ -57,6 +58,9 @@ static const struct {
  + { rc4.d,  SPECIAL_RUNLEVEL4_TARGET, RUNLEVEL_UP },
  + { rc5.d,  SPECIAL_RUNLEVEL5_TARGET, RUNLEVEL_UP },
  +
  ++/* Debian style rcS.d, also adopted by OE */
  ++{ rcS.d,  SPECIAL_SYSINIT_TARGET,   RUNLEVEL_SYSINIT},
  ++
  + /* Standard SysV runlevels for shutdown */
  + { rc0.d,  SPECIAL_POWEROFF_TARGET,  RUNLEVEL_DOWN },
  + { rc6.d,  SPECIAL_REBOOT_TARGET,RUNLEVEL_DOWN }
  +@@ -65,7 +69,7 @@ static const struct {
  +directories in this order, and we want to make sure
  that
  +sysv_start_priority is known when we first load the
  +unit. And that value we only know from S links. Hence
  +-   UP must be read before DOWN */
  ++   UP/SYSINIT must be read before DOWN */
  + };
  +
  + typedef struct SysvStub {
  +@@ -81,6 +85,8 @@ typedef struct SysvStub {
  + char **conflicts;
  + bool has_lsb;
  + bool reload;
  ++bool default_dependencies;
  ++bool from_rcsd;
  + } SysvStub;
  +
  + const char *arg_dest = /tmp;
  +@@ -189,6 +195,8 @@ static int generate_unit_file(SysvStub *s) {
  + Description=%s\n,
  + s-path, s-description);
  +
  ++if (!s-default_dependencies)
  ++fprintf(f, DefaultDependencies=no\n);
  + if (!isempty(before))
  + fprintf(f, Before=%s\n, before);
  + if (!isempty(after))
  +@@ -717,15 +725,26 @@ static int fix_order(SysvStub *s, Hashmap
  *all_services) {
  + r = strv_extend(s-after, other-name);
  + if (r  0)
  + return log_oom();
  +-}
  +-else if (other-sysv_start_priority 
  s-sysv_start_priority) { +-r =
  strv_extend(s-before, other-name); ++   } else if
  (other-from_rcsd  !s-from_rcsd) { ++r =
  strv_extend(s-after, other-name);
  + if (r  0)
  + return log_oom();
  +-}
  +-else
  +-continue;
  +-
  ++} else {
  ++  /* All scripts under /etc/rcS.d should execute before
  scripts under ++   * /etc/rcN.d */
  ++ if (!other-from_rcsd  s-from_rcsd) {
  ++ r = strv_extend(s-before, other-name);
  ++if (r  0)
  ++return log_oom();
  ++}
  ++else if (other-sysv_start_priority 
  s-sysv_start_priority) { ++r 

Re: [OE-core] [PATCH 0/2] kmod new packageconfig and dbus split package

[Bottazzini, Bruno]

Ping.

From: Burton, Ross [mailto:ross.bur...@intel.com]
Sent: Wednesday, January 28, 2015 5:58 PM
To: Dan McGregor
Cc: Bottazzini, Bruno; OE-core
Subject: Re: [OE-core] [PATCH 0/2] kmod new packageconfig and dbus split package


On 28 January 2015 at 18:55, Dan McGregor 
danismostlik...@gmail.commailto:danismostlik...@gmail.com wrote:
Is there a public place to see what you have queued?

Sure it, ross/mut in poky-contrib.  As often seen on the autobuilder in all 
red. :)

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/2] kmod new packageconfig and dbus split package

[Bottazzini, Bruno]

On Wed, 21 Jan 2015 18:00:56 -0200
Bruno Bottazzini bruno.bottazz...@intel.com wrote:

Guys, were you able to look at this patch ?

 In this patchset we include two changes:
 
 1- kmod: new PACKAGECONFIG debug and logging to help reduce binary
 size. 2- dbus: split it into two package. We will be able to choose
 if we want the full version or a smaller one.
 
 Gustavo Sverzut Barbieri (2):
   kmod: new PACKAGECONFIG debug and logging to help reduce binary
 size. dbus: split tools package.
 
  meta/recipes-core/dbus/dbus.inc   | 13 +++--
  meta/recipes-kernel/kmod/kmod.inc |  5 -
  2 files changed, 11 insertions(+), 7 deletions(-)
 
 --
 1.9.1
 

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] Removing support for udev cdrom

[Bottazzini, Bruno]

Hello there,

I'd like to remove udev-rules-cdrom support in our recipe. 

For some reasons this package is not needed at all.

When I include the following in our recipe:
PACKAGE_EXCLUDE = udev-rules-cdrom

I receive the following error:
Collected errors:
 * satisfy_dependencies_for: Cannot satisfy the following dependencies for 
packagegroup-core-boot:
 *  udev-rules-cdrom * 
 * opkg_install_cmd: Cannot install package packagegroup-core-boot

So, I went to the packagegroup-core-boot.bb and I saw this dependencies:
RDEPENDS_${PN} = \
base-files \
base-passwd \
busybox \
${@bb.utils.contains(DISTRO_FEATURES, sysvinit,
${SYSVINIT_SCRIPTS}, , d)} \
${@bb.utils.contains(MACHINE_FEATURES, keyboard,
${VIRTUAL-RUNTIME_keymaps}, , d)} \
netbase \
${VIRTUAL-RUNTIME_login_manager} \
${VIRTUAL-RUNTIME_init_manager} \
${VIRTUAL-RUNTIME_dev_manager} \
${VIRTUAL-RUNTIME_update-alternatives} \
${MACHINE_ESSENTIAL_EXTRA_RDEPENDS}

As you can see there is no udev-rules-cdrom, at least not explicitly.

Is there any way that I could remove this cdrom support from udev ?

Best Regards,

Bruno Bottazzini
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core