[OE-core] [Call to Action] Yocto Project Bug Triage

2018-05-29 Thread Stephano Cetola
Hi All, Sorry for the cross posting but I want to be sure to hit both the Yocto Project and OpenEmbedded communities with this information. It has been brought to our attention that the weekly bug triage has fallen out of the spotlight and we'd like to get it back into focus. We'd like to

Re: [OE-core] [PATCH 0/1] buildhistory_analysis.py: Check if RPROVIDES changed order

2018-04-02 Thread Stephano Cetola
On 3/15/18 2:04 PM, Amanda Brindle wrote: > The following changes since commit 0fd6d0dd0041929d60111868ca3575d9d9bf9ac3: > > linux-yocto/4.12: backport bugfixes for x86 (2018-03-15 06:27:20 -0700) > > are available in the git repository at: > > git://git.yoctoproject.org/poky-contrib

[OE-core] Yocto Project Status WW13’18

2018-03-26 Thread Stephano Cetola
Current Dev Position: YP 2.5 M3 final close out. Next Deadline: YP 2.5 M4 stabilization is 4/2/18 SWAT lead is currently Ross. SWAT team rotation: Ross -> Juro on March 30, 2018 SWAT team rotation: Juro -> Paul on April 6, 2018 https://wiki.yoctoproject.org/wiki/Yocto_Build_Failure_Swat_Team Key

Re: [OE-core] [OE-Core][PATCHv2 0/1] wic: argparse now used for help functionality.

2018-02-13 Thread Stephano Cetola
On 1/30/18 10:24 AM, anelliot wrote: > The wic help output formally consisted of manually created strings mixed with > argparse, > which was unformatted and unusable. This fix cleans up the help messages, > rewrites help > functionality to use argparse, and adds functionality to show information

Re: [OE-core] [rocko][PATCH 0/4] Backport masterimage fixes

2018-01-09 Thread Stephano Cetola
These missed the 2.4.1 boat. Could we get these applied so they are sure to make it into 2.4.2? Thanks! https://patchwork.openembedded.org/series/9690/ On 11/9/17 8:58 AM, Stephano Cetola wrote: > Backporting fixes since I never got these into 2.4. > > Requested by erik.b...@pelag

[OE-core] [rocko][PATCH 4/4] sshcontrol.py: in copy_to() always use scp

2017-11-09 Thread Stephano Cetola
From: Erik Botö <erik.b...@pelagicore.com> The current implementation is broken when the localpath is a link. Then only a symlink would be created on the target, instead of copying the actual file. [YOCTO #11524] Signed-off-by: Erik Botö <erik.b...@pelagicore.com> Signed-off-by: Ste

[OE-core] [rocko][PATCH 3/4] masterimage.py: rename parameter "params" in start() to "extra_bootparams"

2017-11-09 Thread Stephano Cetola
From: Erik Botö <erik.b...@pelagicore.com> This matches how it is called, and how it is named in qmeu target. [YOCTO #11524] Signed-off-by: Erik Botö <erik.b...@pelagicore.com> Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/lib/oeqa/controllers/

[OE-core] [rocko][PATCH 1/4] masterimage.py: fix issue with calling reboot on masterimage/DUT

2017-11-09 Thread Stephano Cetola
s part of the reboot and could take a very long time and testimage will fail to access the machine. This issue was observed consistently with one of our rootfs at Pelagicore. [YOCTO #11524] Signed-off-by: Erik Botö <erik.b...@pelagicore.com> Signed-off-by: Stephano Cetola &

[OE-core] [rocko][PATCH 0/4] Backport masterimage fixes

2017-11-09 Thread Stephano Cetola
Backporting fixes since I never got these into 2.4. Requested by erik.b...@pelagicore.com. This fixes [YOCTO #11524] Erik Botö (4): masterimage.py: fix issue with calling reboot on masterimage/DUT masterimage.py: fix stop() masterimage.py: rename parameter "params" in start() to

[OE-core] [rocko][PATCH 2/4] masterimage.py: fix stop()

2017-11-09 Thread Stephano Cetola
From: Erik Botö <erik.b...@pelagicore.com> The stop() function is called in the context of the masterimage, so self.master should be used instead of self.connection which is undefined at that time. [YOCTO #11524] Signed-off-by: Erik Botö <erik.b...@pelagicore.com> Signed-off-by: Ste

[OE-core] [PATCH 2/4] masterimage.py: fix stop()

2017-11-06 Thread Stephano Cetola
From: Erik Botö <erik.b...@pelagicore.com> The stop() function is called in the context of the masterimage, so self.master should be used instead of self.connection which is undefined at that time. [YOCTO #11524] Signed-off-by: Erik Botö <erik.b...@pelagicore.com> Signed-off-by: Ste

[OE-core] [PATCH 3/4] masterimage.py: rename parameter "params" in start() to "extra_bootparams"

2017-11-06 Thread Stephano Cetola
From: Erik Botö <erik.b...@pelagicore.com> This matches how it is called, and how it is named in qmeu target. [YOCTO #11524] Signed-off-by: Erik Botö <erik.b...@pelagicore.com> Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/lib/oeqa/controllers/

[OE-core] [PATCH 0/4] fix runtime tests with systemd-boot

2017-11-06 Thread Stephano Cetola
These patches fell through the cracks of 2.4. I've checked with the QA team and they are still relevant. Ping me with any questions. I will be backporting these to rocko as well. This fixes [YOCTO #11524] Erik Botö (4): masterimage.py: fix issue with calling reboot on masterimage/DUT

[OE-core] [PATCH 4/4] sshcontrol.py: in copy_to() always use scp

2017-11-06 Thread Stephano Cetola
From: Erik Botö <erik.b...@pelagicore.com> The current implementation is broken when the localpath is a link. Then only a symlink would be created on the target, instead of copying the actual file. [YOCTO #11524] Signed-off-by: Erik Botö <erik.b...@pelagicore.com> Signed-off-by: Ste

[OE-core] [PATCH 1/4] masterimage.py: fix issue with calling reboot on masterimage/DUT

2017-11-06 Thread Stephano Cetola
s part of the reboot and could take a very long time and testimage will fail to access the machine. This issue was observed consistently with one of our rootfs at Pelagicore. [YOCTO #11524] Signed-off-by: Erik Botö <erik.b...@pelagicore.com> Signed-off-by: Stephano Cetola &

Re: [OE-core] [PATCH v4 0/4] fix runtime testing using systemd-boot targe

2017-07-10 Thread Stephano Cetola
Any updates on this? I can setup a test for this if one doesn't already exist. Cheers, Stephano On 06/20, Stephano Cetola wrote: > I'm sending this on behalf of Zeeshan Ali <zee...@gmail.com>. One of > the patches required a rebase and he is on vacation. This fixes > [YOCTO #11

[OE-core] [PATCH v4 2/4] masterimage.py: fix stop()

2017-06-20 Thread Stephano Cetola
From: Erik Botö The stop() function is called in the context of the masterimage, so self.master should be used instead of self.connection which is undefined at that time. [YOCTO #11524] Signed-off-by: Erik Botö ---

[OE-core] [PATCH v4 4/4] masterimage.py: fix issue with calling reboot on masterimage/DUT

2017-06-20 Thread Stephano Cetola
From: Erik Botö On systemd systems calling reboot over an ssh connection doesn't return as expected causing an exception, therefore wrap the call to reboot in order to avoid this issue. Also sync the filesystems before rebooting cause otherwise, it will be done as part

[OE-core] [PATCH v4 3/4] masterimage.py: rename parameter "params" in start() to "extra_bootparams"

2017-06-20 Thread Stephano Cetola
From: Erik Botö This matches how it is called, and how it is named in qmeu target. [YOCTO #11524] Signed-off-by: Erik Botö --- meta/lib/oeqa/controllers/masterimage.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[OE-core] [PATCH v4 1/4] sshcontrol.py: in copy_to() always use scp

2017-06-20 Thread Stephano Cetola
From: Erik Botö The current implementation is broken when the localpath is a link. Then only a symlink would be created on the target, instead of copying the actual file. [YOCTO #11524] Signed-off-by: Erik Botö ---

[OE-core] [PATCH v4 0/4] fix runtime testing using systemd-boot targe

2017-06-20 Thread Stephano Cetola
I'm sending this on behalf of Zeeshan Ali . One of the patches required a rebase and he is on vacation. This fixes [YOCTO #11524]. Erik Botö (4): sshcontrol.py: in copy_to() always use scp masterimage.py: fix stop() masterimage.py: rename parameter "params" in start() to

Re: [OE-core] ✗ patchtest: failure for oeqa: fix runtime testing using systemd-boot target (rev2)

2017-06-20 Thread Stephano Cetola
Hey Zeeshan, Let me know if you need any help with this. This probably requires that you do a rebase with master and resubmit a V4. Cheers, Stephano On 06/19, Patchwork wrote: > == Series Details == > > Series: oeqa: fix runtime testing using systemd-boot target (rev2) > Revision: 2 > URL :

Re: [OE-core] [PATCH v2 1/4] masterimage.py: fix issue with calling reboot on masterimage/DUT

2017-06-15 Thread Stephano Cetola
On 06/15, Zeeshan Ali (Khattak) wrote: > Hi, > > Has this been already merged? I ask cause there is a small change > needed for large-enough rootfs: > This has not been merged yet. Erik, could you please send a V3? When you do so, please reference the bug in the commit messages like this:

Re: [OE-core] [PATCH] net-tools: enable native and nativesdk variant

2017-03-30 Thread Stephano Cetola
This won't work, and it is failing on the ab. https://autobuilder.yoctoproject.org/main/builders/nightly-no-x11/builds/449/steps/BuildImages/logs/stdio Just add a 'return' to your empty bash function and it'll work fine. Cheers, Stephano On 03/30, Patrick Ohly wrote: > net-tools-native is

[OE-core] [PATCH] insane.bbclass: error if file-rdeps not met

2017-03-27 Thread Stephano Cetola
will be thrown. [YOCTO #10949] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/classes/insane.bbclass | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass index 7a869977e1..dc1c2f8d41

[OE-core] [PATCH] devtool: open kconfig fragment in binary mode

2017-03-27 Thread Stephano Cetola
When devtool writes to the kconfig fragment, it writes the output of the diff command returned from pipe.communicate(). This function returns binary objects. We should open the kconfig fragment file in binary mode if we expect to write binary objects to it. [YOCTO #11171] Signed-off-by: Stephano

[OE-core] [PATCH] uninative: remove unneeded call to sparse copy

2017-03-10 Thread Stephano Cetola
-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/classes/uninative.bbclass | 3 --- 1 file changed, 3 deletions(-) diff --git a/meta/classes/uninative.bbclass b/meta/classes/uninative.bbclass index 0d1063a13c..ee4ab20c17 100644 --- a/meta/classes/uninative.bbclass +++ b/meta/c

[OE-core] [PATCH V2 0/1] remove call to non-existent method

2017-02-10 Thread Stephano Cetola
Changed since V1: reworded commit message Stephano Cetola (1): insane.bbclass: print QA message correctly meta/classes/insane.bbclass | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.11.1 -- ___ Openembedded-core mailing list

[OE-core] [PATCH V2 1/1] insane.bbclass: print QA message correctly

2017-02-10 Thread Stephano Cetola
Replace call to non-existent method "messages" with call to correct QA logging method. Fixes [YOCTO #11014] introduced in OE-core revision 1dcb38ca9d6 Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/classes/insane.bbclass | 2 +- 1 file changed, 1

[OE-core] [PATCH] insane.bbclass: print QA message correctly

2017-02-09 Thread Stephano Cetola
When checking for ".debug" files, print QA message correctly. Fixes [YOCTO #11014] introduced in OE-core revision 1dcb38ca9d6 Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/classes/insane.bbclass | 2 +- 1 file changed, 1 insertion(+), 1 deletio

[OE-core] [PATCH V6] package_manager: remove strings and migrate to direct arrays

2016-12-06 Thread Stephano Cetola
When using subprocess call and check_output, it is better to use arrays rather than strings when possible to avoid whitespace and quoting problems. [ YOCTO #9342 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/lib/oe/package.py | 13 +-- meta/

[OE-core] [PATCH V6] cleanup subprocess

2016-12-06 Thread Stephano Cetola
Changed since V5: Smart needs to dump the installed packages to a file, as this file is used to detect changes in the install. This was breaking: test_incremental_image_generation Stephano Cetola (1): package_manager: remove strings and migrate to direct arrays meta/lib/oe/package.py

[OE-core] [PATCH V5] subprocess cleanup

2016-11-29 Thread Stephano Cetola
changed since V4: Removed single quotes from the --show-format option on smart query. The quote was causing the arch search to fail in the following for loop. Stephano Cetola (1): package_manager: remove strings and migrate to direct arrays meta/lib/oe/package.py | 13 +-- meta/lib

[OE-core] [PATCH V5] package_manager: remove strings and migrate to direct arrays

2016-11-29 Thread Stephano Cetola
When using subprocess call and check_output, it is better to use arrays rather than strings when possible to avoid whitespace and quoting problems. [ YOCTO #9342 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/lib/oe/package.py | 13 +-- meta/

Re: [OE-core] [PATCH] devtool: fix handling of unicode characters from subprocess stdout

2016-11-16 Thread Stephano Cetola
On 11/11, Burton, Ross wrote: > A reader is definitely the right thing here, but I'm wondering why this > needs to loop on single characters. As I understand it doing a read() on a > reader wrapping stdout will read until it blocks (because the process > hasn't got anything to output) so result

[OE-core] [PATCH V4] Cleanup subprocess calls

2016-11-15 Thread Stephano Cetola
Changed since V3: Corrected commit summary by changing primary item worked on from subprocess to package_manager. Stephano Cetola (1): package_manager: remove strings and migrate to direct arrays meta/lib/oe/package.py | 13 +-- meta/lib/oe/package_manager.py | 229

[OE-core] [PATCH V4] package_manager: remove strings and migrate to direct arrays

2016-11-15 Thread Stephano Cetola
When using subprocess call and check_output, it is better to use arrays rather than strings when possible to avoid whitespace and quoting problems. [ YOCTO #9342 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/lib/oe/package.py | 13 +-- meta/

[OE-core] [PATCH V2] recipetool: add postinst to .deb import

2016-11-15 Thread Stephano Cetola
Changed since V1: Removed unused variable and cleaned up commit message. Stephano Cetola (1): recipetool: add postinst to .deb import scripts/lib/recipetool/create.py | 35 --- 1 file changed, 32 insertions(+), 3 deletions(-) -- 2.10.2

[OE-core] [PATCH V2] recipetool: add postinst to .deb import

2016-11-15 Thread Stephano Cetola
The .deb import feature did not import postinst, postrm, preinst, or prerm functions. This change checks to see if those files exist, and if so, adds the appropriate functions. [ YOCTO #10421 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- scripts/lib/recipetool/cre

[OE-core] [PATCH V3] Cleanup subprocess calls

2016-11-15 Thread Stephano Cetola
Changed since V2: Fixed rpm "remove" function, changing strings to arrays. This fixes the error when using "read-only-rootfs" image feature. Stephano Cetola (1): subprocess: remove strings and migrate to direct arrays meta/lib/oe/package.py | 13 +-- meta/lib/o

[OE-core] [PATCH V3] subprocess: remove strings and migrate to direct arrays

2016-11-15 Thread Stephano Cetola
When using subprocess call and check_output, it is better to use arrays rather than strings when possible to avoid whitespace and quoting problems. [ YOCTO #9342 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/lib/oe/package.py | 13 +-- meta/

[OE-core] [PATCH V3] devshell: list commands when throwing NoSupportedTerminals

2016-11-14 Thread Stephano Cetola
Changes since V2: add yocto bug number Stephano Cetola (1): devshell: list commands when throwing NoSupportedTerminals meta/classes/terminal.bbclass | 8 ++-- meta/lib/oe/terminal.py | 13 +++-- 2 files changed, 17 insertions(+), 4 deletions(-) -- 2.10.2

[OE-core] [PATCH V3] devshell: list commands when throwing NoSupportedTerminals

2016-11-14 Thread Stephano Cetola
When attempting to run devshell, if no terminal is available, the error being thrown was not very specific. This adds a list of commands that failed, informing the user of what they can install to fix the error. [ YOCTO #10472] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.

[OE-core] [PATCH] recipetool: add postinst to .deb import

2016-11-14 Thread Stephano Cetola
The .deb import feature did not import postinst, postrm, preinst, or¬ prerm functions. This change checks to see if those files exist, and if so, adds the appropriate functions adding correct path variables if¬ required.¬ [ YOCTO #10421 ] Signed-off-by: Stephano Cetola <stephano.

[OE-core] [PATCH V2] devshell: list commands when throwing NoSupportedTerminals

2016-11-14 Thread Stephano Cetola
When attempting to run devshell, if no terminal is available, the error being thrown was not very specific. This adds a list of commands that failed, informing the user of what they can install to fix the error. Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/c

[OE-core] [PATCH V2] devshell: list commands when throwing NoSupportedTerminals

2016-11-14 Thread Stephano Cetola
Changes since V1: Remove the placeholder "false" for custom terminal rather than changing it to None. That change would break the custom terminal. Stephano Cetola (1): devshell: list commands when throwing NoSupportedTerminals meta/classes/terminal.bbclass | 8 ++-- m

[OE-core] [PATCH] devshell: list commands when throwing NoSupportedTerminals

2016-11-14 Thread Stephano Cetola
When attempting to run devshell, if no terminal is available, the error being thrown was not very specific. This adds a list of commands that failed, informing the user of what they can install to fix the error. Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/c

Re: [OE-core] [PATCH V2] subprocess: remove strings and migrate to direct arrays

2016-10-10 Thread Stephano Cetola
On 10/09, Richard Purdie wrote: > On Thu, 2016-10-06 at 20:09 -0700, Stephano Cetola wrote: > > > > When using subprocess call and check_output, it is better to use > > arrays > > rather than strings when possible to avoid whitespace and quoting > >

[OE-core] [PATCH V2] subprocess: remove strings and migrate to direct arrays

2016-10-06 Thread Stephano Cetola
When using subprocess call and check_output, it is better to use arrays rather than strings when possible to avoid whitespace and quoting problems. [ YOCTO #9342 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/lib/oe/distro_check.py| 2 +- meta/

[OE-core] [PATCH V2] Cleanup subprocess calls

2016-10-06 Thread Stephano Cetola
. Stephano Cetola (1): subprocess: remove strings and migrate to direct arrays meta/lib/oe/distro_check.py| 2 +- meta/lib/oe/package.py | 13 +-- meta/lib/oe/package_manager.py | 218 - 3 files changed, 114 insertions(+), 119 deletions

Re: [OE-core] [PATCH V3] add function to check for git config user

2016-10-05 Thread Stephano Cetola
On 10/03, Stephano Cetola wrote: > Chnages since V2: > moved functionality from sanity to utils > utilize existing name / email variables > > Stephano Cetola (1): > utils.bbclass: add function to check for git config user > > meta/classes/buildhistory.bbclass | 15

[OE-core] [PATCH V4] devtool: modify command fails to ignore source files

2016-10-05 Thread Stephano Cetola
files for oe-local-files directory. [YOCTO #10326] introduced in OE-Core revision 9069fef5dad5a873c8a8f720f7bcbc7625556309 Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- scripts/lib/devtool/standard.py | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff

[OE-core] [PATCH V4] devtool: modify command fails to ignore source

2016-10-05 Thread Stephano Cetola
changed since V3: added a check for the case where $S = $WORKDIR fixes [YOCTO #10379] Stephano Cetola (1): devtool: modify command fails to ignore source files scripts/lib/devtool/standard.py | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) -- 2.10.0

[OE-core] [PATCH V3] devtool: modify command fails to ignore source files

2016-10-03 Thread Stephano Cetola
files for oe-local-files directory. [YOCTO #10326] introduced in OE-Core revision 9069fef5dad5a873c8a8f720f7bcbc7625556309 Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- scripts/lib/devtool/standard.py | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff

[OE-core] [PATCH V3] devtool: modify command fails to ignore source files

2016-10-03 Thread Stephano Cetola
Changes since V2: Added os.sep to path name for decreased chance of false positive matches. Stephano Cetola (1): devtool: modify command fails to ignore source files scripts/lib/devtool/standard.py | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) -- 2.10.0

Re: [OE-core] [PATCH V2] devtool: modify command fails to ignore source files

2016-10-03 Thread Stephano Cetola
On 10/04, Paul Eggleton wrote: > On Sun, 02 Oct 2016 16:58:24 Stephano Cetola wrote: > > With recent changes to recipeutils, the list of local files returned > > by get_recipe_local_files could possibly include source files. This > > only happens when the recipe contains a

[OE-core] [PATCH V3] utils.bbclass: add function to check for git config user

2016-10-03 Thread Stephano Cetola
. This patch abstracts out functionality that existed in buildhistory for use in other classes. It also adds a call to this functionality to the kernel-yocto class. Fixes [YOCTO #10346] introduced in OE-core revision 0f698dfd1c8bbc0d53ae7977e26685a7a3df52a3 Signed-off-by: Stephano Cetola

[OE-core] [PATCH V3] add function to check for git config user

2016-10-03 Thread Stephano Cetola
Chnages since V2: moved functionality from sanity to utils utilize existing name / email variables Stephano Cetola (1): utils.bbclass: add function to check for git config user meta/classes/buildhistory.bbclass | 15 ++- meta/classes/kernel-yocto.bbclass | 3 +++ meta/classes

Re: [OE-core] [PATCH V2] sanity: add function to check for git config user

2016-10-03 Thread Stephano Cetola
On 10/03, Burton, Ross wrote: > On 3 October 2016 at 01:43, Stephano Cetola <stephano.cet...@linux.intel.com > > wrote: > > > +# If the user hasn't set up their name/email, set some defaults > > +check_git_config() { > > + if [ -z "$1" ]; then >

[OE-core] [PATCH V2] sanity: add function to check for git config user

2016-10-02 Thread Stephano Cetola
. This patch abstracts out functionality that existed in buildhistory for use in other classes. It also adds a call to this functionality to the kernel-yocto class. Fixes [YOCTO #10346] introduced in OE-core revision 0f698dfd1c8bbc0d53ae7977e26685a7a3df52a3 Signed-off-by: Stephano Cetola

[OE-core] [PATCH V2] add function to check for git config user

2016-10-02 Thread Stephano Cetola
Changed since V1: forgot to add Yocto bug #, also cleaned up commit message Stephano Cetola (1): sanity: add function to check for git config user meta/classes/buildhistory.bbclass | 13 - meta/classes/kernel-yocto.bbclass | 2 ++ meta/classes/sanity.bbclass | 15

[OE-core] [PATCH V2] devtool: modify command fails to ignore source files

2016-10-02 Thread Stephano Cetola
files for oe-local-files directory. [YOCTO #10326] introduced in OE-Core revision 9069fef5dad5a873c8a8f720f7bcbc7625556309 Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- scripts/lib/devtool/standard.py | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff

Re: [OE-core] [PATCH] devtool: modify command fails to ignore source files

2016-10-02 Thread Stephano Cetola
On 10/03, Paul Eggleton wrote: > Hi Stephano, > > Thanks for looking into this, a few things: > > On Sun, 02 Oct 2016 13:55:06 Stephano Cetola wrote: > > With recent changes to recipeutils, the list of local files returned > > by get_recipe_local_files could po

[OE-core] [PATCH] sanity: add function to check for git config user

2016-10-02 Thread Stephano Cetola
. This patch abstracts out functionality that existed in buildhistory for use in other classes. It also adds a call to this functionality to the kernel-yocto class. Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/classes/buildhistory.bbclass | 13 - meta/c

[OE-core] [PATCH v2 1/2] subprocess: remove strings and migrate to direct arrays

2016-09-29 Thread Stephano Cetola
When using subprocess call and check_output, it is better to use arrays rather than strings when possible to avoid whitespace and quoting problems. [ YOCTO #9342 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/lib/oe/distro_check.py| 2 +- meta/lib/oe/pack

[OE-core] [PATCH v2 2/2] subprocess: remove Popen in favor of check_output

2016-09-29 Thread Stephano Cetola
This begins moving away from the deprecated subprocess calls in an effort to eventually move to some more global abstraction using the run convenience method provided in python 3.5. [ YOCTO #9342 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/c

Re: [OE-core] [PATCH 2/3] utils.py: gut python 2 commands in favor of subprocess.run

2016-09-29 Thread Stephano Cetola
On 09/29, Christopher Larson wrote: > On Thu, Sep 29, 2016 at 2:34 PM, Stephano Cetola < > stephano.cet...@linux.intel.com> wrote: > > > getstatusoutput is a wrapper around subprocess.getstatusouput() in > > Py3, which is basically deprecated and behaves alm

[OE-core] [PATCH v2 0/2] subprocess: cleanup subprocess calls

2016-09-29 Thread Stephano Cetola
changing these subprocess calls. With more time, many of the Popen calls that still exist could be moved to either check_output or sub.run. Stephano Cetola (2): subprocess: remove strings and migrate to direct arrays subprocess: remove Popen in favor of check_output meta/classes

[OE-core] [PATCH 3/3] subprocess: remove Popen in favor of check_output

2016-09-29 Thread Stephano Cetola
This begins moving away from the deprecated subprocess calls in an effort to eventually move to some more global abstraction using the run convenience method provided in python 3.5. [ YOCTO #9342 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/c

[OE-core] [PATCH 1/3] subprocess: remove strings and migrate to direct arrays

2016-09-29 Thread Stephano Cetola
When using subprocess call and check_output, it is better to use arrays rather than strings when possible to avoid whitespace and quoting problems. [ YOCTO #9342 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/lib/oe/distro_check.py| 2 +- meta/lib/oe/pack

[OE-core] [PATCH 0/3] subprocess: cleanup subprocess calls

2016-09-29 Thread Stephano Cetola
changing these subprocess calls. With more time, many of the Popen calls that still exist could be moved to either check_output or sub.run. Stephano Cetola (3): subprocess: remove strings and migrate to direct arrays utils.py: gut python 2 commands in favor of subprocess.run subprocess: remove

[OE-core] [PATCH 2/3] utils.py: gut python 2 commands in favor of subprocess.run

2016-09-29 Thread Stephano Cetola
getstatusoutput is a wrapper around subprocess.getstatusouput() in Py3, which is basically deprecated and behaves almost entirely unlike run(). [ YOCTO #9342 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/lib/oe/utils.py | 11 --- 1 file chan

[OE-core] [PATCH] scripts/runqemu: provide better error message on runqemu ifup fail

2016-09-22 Thread Stephano Cetola
t; which should give them all the info they need. Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- scripts/runqemu | 3 +++ 1 file changed, 3 insertions(+) diff --git a/scripts/runqemu b/scripts/runqemu index 658f7c8..06bd5f3 100755 --- a/scripts/runqemu +++ b/script

[OE-core] [PATCH] provide better error message when runqemu-ifup fails

2016-08-29 Thread Stephano Cetola
t; which should give them all the info they need. Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- scripts/runqemu-internal | 1 + 1 file changed, 1 insertion(+) diff --git a/scripts/runqemu-internal b/scripts/runqemu-internal index d10466d..3d728c0 100755 --- a/scri

[OE-core] [PATCH V5] rootfs.py: allow removal of unneeded packages

2016-08-17 Thread Stephano Cetola
ng easily overridden, as this would have allowed users to remove packages on a whim, which may be giving them just enough rope to hang themselves. That being said, IMHO this patch adds some flexibility without adding serious complexity or avenues for confusion. Stephano Cetola (1): root

[OE-core] [PATCH V5] rootfs.py: allow removal of unneeded packages

2016-08-17 Thread Stephano Cetola
Current functionality allows for the removal of certain packages based on the read-only image feature. This patch extends this functionality by adding the FORCE_RO_REMOVE variable, which will remove these packages regardless of any image features. [ YOCTO #9491 ] Signed-off-by: Stephano Cetola

[OE-core] [PATCH V2] simultaneous do_rootfs tasks with testimage fix

2016-08-10 Thread Stephano Cetola
Changed since V1: I've added a fix to the smart tests which creates a fresh index of the RPM packages specifically for the tests. This decouples the tests and the rootfs processes, which also does some indexing for installation. Stephano Cetola (1): Allow for simultaneous do_rootfs tasks

[OE-core] [PATCH V2] Allow for simultaneous do_rootfs tasks with rpm

2016-08-10 Thread Stephano Cetola
. Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/classes/rootfs_rpm.bbclass | 5 - meta/lib/oe/package_manager.py | 17 ++--- meta/lib/oeqa/runtime/smart.py | 41 - 3 files changed, 54 insertions(+), 9 del

Re: [OE-core] [PATCH] Allow for simultaneous do_rootfs tasks with rpm

2016-07-31 Thread Stephano Cetola
On 07/27, Richard Purdie wrote: > On Tue, 2016-07-26 at 20:43 -0700, Stephano Cetola wrote: > > On 07/20, Richard Purdie wrote: > > > On Thu, 2016-07-14 at 14:20 -0700, Stephano Cetola wrote: > > > > Give each rootfs its own RPM channel to use. This

Re: [OE-core] [PATCH] Allow for simultaneous do_rootfs tasks with rpm

2016-07-26 Thread Stephano Cetola
On 07/20, Richard Purdie wrote: > On Thu, 2016-07-14 at 14:20 -0700, Stephano Cetola wrote: > > Give each rootfs its own RPM channel to use. This puts the RPM > > metadata > > in a private subdirectory of $WORKDIR, rather than living in > > DEPLOY_DIR >

Re: [OE-core] [PATCH V3] rootfs.py: allow removal of support packages

2016-07-18 Thread Stephano Cetola
On 07/18, Burton, Ross wrote: > On 11 July 2016 at 18:55, Stephano Cetola <stephano.cet...@linux.intel.com> > wrote: > > > +unneeded_pkgs.extend(self.d.getVar("ROOTFS_BOOTSTRAP_INSTALL", > > +True).split(",")) >

[OE-core] [PATCH] Allow for simultaneous do_rootfs tasks with rpm

2016-07-14 Thread Stephano Cetola
the majority of the rootfs operation to run in parallel. [ YOCTO #9255 ] Signed-Off-By: Steven Walter <stevenrwal...@gmail.com> Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/classes/rootfs_rpm.bbclass | 5 - meta/lib/oe/package_manager.py | 17

[OE-core] [PATCH] systemd: allow add users to groups in rootfs postprocess

2016-07-14 Thread Stephano Cetola
Currently the functionality checks for the "u" and "g" flags to create users and groups, but not the "m" flag to add users to groups. This change first checks to be sure that the users and groups are created, creates them if necessary, then adds the user to the g

Re: [OE-core] [PATCH V4] rootfs.py: allow removal of support packages

2016-07-12 Thread Stephano Cetola
On 07/12, Stephano Cetola wrote: > Changes from V3: > Split off of whitespace rather than comma. > > Stephano Cetola (1): > rootfs.py: allow removal of support packages > > meta/lib/oe/rootfs.py | 20 ++-- > 1 file changed, 18 insertions(+), 2 delet

[OE-core] [PATCH V4] rootfs.py: allow removal of support packages

2016-07-12 Thread Stephano Cetola
Changes from V3: Split off of whitespace rather than comma. Stephano Cetola (1): rootfs.py: allow removal of support packages meta/lib/oe/rootfs.py | 20 ++-- 1 file changed, 18 insertions(+), 2 deletions(-) -- 2.9.0

[OE-core] [PATCH V4] rootfs.py: allow removal of support packages

2016-07-12 Thread Stephano Cetola
For images not containing a package manager, this allows the removal of support packages based on the REMOVE_PKG_SUPPORT variable. Packages that will be uninstalled include: update-rc.d, base-passwd, shadow, run-postinsts, and update-alternatives. [ YOCTO #9491 ] Signed-off-by: Stephano Cetola

[OE-core] [PATCH V3] rootfs.py: allow removal of support packages

2016-07-11 Thread Stephano Cetola
I've updated the list to be extened by getVar().split() as requested. Stephano Cetola (1): rootfs.py: allow removal of support packages meta/lib/oe/rootfs.py | 20 ++-- 1 file changed, 18 insertions(+), 2 deletions(-) -- 2.9.0

[OE-core] [PATCH V3] rootfs.py: allow removal of support packages

2016-07-11 Thread Stephano Cetola
For images not containing a package manager, this allows the removal of support packages based on the REMOVE_PKG_SUPPORT variable. Packages that will be uninstalled include: update-rc.d, base-passwd, shadow, run-postinsts, and update-alternatives. Signed-off-by: Stephano Cetola <stephano.

[OE-core] [PATCH V2] rootfs.py: allow removal of support packages

2016-07-08 Thread Stephano Cetola
I missed run-postinsts from the above package list that needs to be removed. Stephano Cetola (1): rootfs.py: allow removal of support packages meta/lib/oe/rootfs.py | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) -- 2.9.0

[OE-core] [PATCH V2] rootfs.py: allow removal of support packages

2016-07-08 Thread Stephano Cetola
For images not containing a package manager, this allows the removal of support packages based on the REMOVE_PKG_SUPPORT variable. Packages that will be uninstalled include: update-rc.d, base-passwd, shadow, run-postinsts, and update-alternatives. [ YOCTO #9491 ] Signed-off-by: Stephano Cetola

[OE-core] [PATCH] rootfs.py: allow removal of support packages

2016-07-06 Thread Stephano Cetola
For images not containing a package manager, this allows the removal of support packages based on the REMOVE_PKG_SUPPORT variable. Packages that will be uninstalled include: update-rc.d, base-passwd, shadow and update-alternatives. [ YOCTO #9491 ] Signed-off-by: Stephano Cetola <stephano.

[OE-core] [PATCH] useradd-staticids.bbclass: Restore failure on missing UIDs/GIDs

2016-06-22 Thread Stephano Cetola
From: Peter Kjellerstedt A regression was introduced with commit 3149319a whereby setting USERADD_ERROR_DYNAMIC no longer resulted in an error for users and groups that were missing numeric UIDs and GIDs but were not mentioned at all in any passwd or groups file.

[OE-core] [PATCH][for-krogoth] backport staticid fix

2016-06-22 Thread Stephano Cetola
I've backported this from master and tested. [ YOCTO #9777 ] Peter Kjellerstedt (1): useradd-staticids.bbclass: Restore failure on missing UIDs/GIDs meta/classes/useradd-staticids.bbclass | 19 +-- 1 file changed, 13 insertions(+), 6 deletions(-) -- 2.8.3 --

[OE-core] [PATCHv3] systemd: allow add users as a rootfs postprocess cmd

2016-05-31 Thread Stephano Cetola
ch user's add params. [ YOCTO #9497 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/classes/rootfs-postcommands.bbclass | 43 +++- 1 file changed, 20 insertions(+), 23 deletions(-) diff --git a/meta/classes/rootfs-postcommands.bbcl

[OE-core] [PATCHv3] systemd: allow add users as a rootfs postprocess cmd

2016-05-31 Thread Stephano Cetola
Changes since last revision: remove bbwarn Stephano Cetola (1): systemd: allow add users as a rootfs postprocess cmd meta/classes/rootfs-postcommands.bbclass | 43 +++- 1 file changed, 20 insertions(+), 23 deletions(-) -- 2.8.2

[OE-core] [PATCHv2] systemd: allow add users as a rootfs postprocess cmd

2016-05-31 Thread Stephano Cetola
ch user's add params. [ YOCTO #9497 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/classes/rootfs-postcommands.bbclass | 44 +++- 1 file changed, 21 insertions(+), 23 deletions(-) diff --git a/meta/classes/rootfs-postcommands.bbcl

[OE-core] [PATCHv2] systemd: allow add users as a rootfs postprocess cmd

2016-05-31 Thread Stephano Cetola
Changed since last version: Removed some unnecessary code. Stephano Cetola (1): systemd: allow add users as a rootfs postprocess cmd meta/classes/rootfs-postcommands.bbclass | 44 +++- 1 file changed, 21 insertions(+), 23 deletions(-) -- 2.8.2

[OE-core] [PATCH] systemd: allow add users as a rootfs postprocess cmd

2016-05-31 Thread Stephano Cetola
ch user's add params. [ YOCTO #9497 ] Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/classes/rootfs-postcommands.bbclass | 41 1 file changed, 21 insertions(+), 20 deletions(-) diff --git a/meta/classes/rootfs-postcommands.bbcl

[OE-core] [PATCH] package.bbclass: fix strip and split logic

2016-05-26 Thread Stephano Cetola
: no strip, debug split INHIBIT_PACKAGE_DEBUG_SPLIT: strip, no split BOTH: no strip, no split, DNP data structures Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> --- meta/classes/package.bbclass | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/meta/c

Re: [OE-core] [PATCH v2 0/2] Create systemd users at build time

2016-05-12 Thread Stephano Cetola
On 05/12, Puustinen, Ismo wrote: > On Wed, 2016-05-11 at 12:04 -0700, Stephano Cetola wrote: > > > I think this is a valuable feature. However, have you considered > > > adding > > > the "m" and "r" options too? I think that at least "m"

  1   2   >