public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
* [PATCH] linux-custom: Make tracing more consistent
@ 2022-07-10 20:51 Jan Kiszka
  2022-07-11 13:07 ` Henning Schild
  2022-07-19 18:55 ` Anton Mikanovich
  0 siblings, 2 replies; 6+ messages in thread
From: Jan Kiszka @ 2022-07-10 20:51 UTC (permalink / raw)
  To: isar-users; +Cc: Cedric Hombourger

From: Jan Kiszka <jan.kiszka@siemens.com>

Include more of the do_build and anything of do_configure into the trace
log.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
---
 meta/recipes-kernel/linux/files/debian/isar/build.tmpl     | 6 +++---
 meta/recipes-kernel/linux/files/debian/isar/configure.tmpl | 3 +++
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-kernel/linux/files/debian/isar/build.tmpl b/meta/recipes-kernel/linux/files/debian/isar/build.tmpl
index 550bdc67..65261e41 100644
--- a/meta/recipes-kernel/linux/files/debian/isar/build.tmpl
+++ b/meta/recipes-kernel/linux/files/debian/isar/build.tmpl
@@ -10,6 +10,9 @@ do_build() {
     # Print a few things that are of particular interest
     print_settings
 
+    # Trace what we do here
+    set -x
+
     # Process existing kernel configuration to make sure it is complete
     # (use defaults for options that were not specified)
     ${MAKE} O=${KERNEL_BUILD_DIR} olddefconfig prepare
@@ -18,9 +21,6 @@ do_build() {
     KR=$(${MAKE} O=${KERNEL_BUILD_DIR} -s --no-print-directory kernelrelease)
     sed -i "s/@KR@/${KR}/g" ${S}/debian/control ${S}/debian/linux-image-${KERNEL_NAME_PROVIDED}.*
 
-    # Trace what we do here
-    set -x
-
     # Build the Linux kernel
     ${MAKE} O=${KERNEL_BUILD_DIR} ${PARALLEL_MAKE}
 
diff --git a/meta/recipes-kernel/linux/files/debian/isar/configure.tmpl b/meta/recipes-kernel/linux/files/debian/isar/configure.tmpl
index 58f10326..389c9a85 100644
--- a/meta/recipes-kernel/linux/files/debian/isar/configure.tmpl
+++ b/meta/recipes-kernel/linux/files/debian/isar/configure.tmpl
@@ -7,6 +7,9 @@
 
 do_configure() {
 
+    # Trace what we do here
+    set -x
+
     # Process kernel config target and fragments
     ${MAKE} O=${KERNEL_BUILD_DIR} ${KERNEL_CONFIG_TARGET}
     ./scripts/kconfig/merge_config.sh -O ${KERNEL_BUILD_DIR}/ \
-- 
2.35.3

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] linux-custom: Make tracing more consistent
  2022-07-10 20:51 [PATCH] linux-custom: Make tracing more consistent Jan Kiszka
@ 2022-07-11 13:07 ` Henning Schild
  2022-07-11 17:02   ` Jan Kiszka
  2022-07-19 18:55 ` Anton Mikanovich
  1 sibling, 1 reply; 6+ messages in thread
From: Henning Schild @ 2022-07-11 13:07 UTC (permalink / raw)
  To: Jan Kiszka; +Cc: isar-users, Cedric Hombourger

Am Sun, 10 Jul 2022 22:51:31 +0200
schrieb Jan Kiszka <jan.kiszka@siemens.com>:

> From: Jan Kiszka <jan.kiszka@siemens.com>
> 
> Include more of the do_build and anything of do_configure into the
> trace log.
> 
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
> ---
>  meta/recipes-kernel/linux/files/debian/isar/build.tmpl     | 6 +++---
>  meta/recipes-kernel/linux/files/debian/isar/configure.tmpl | 3 +++
>  2 files changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/meta/recipes-kernel/linux/files/debian/isar/build.tmpl
> b/meta/recipes-kernel/linux/files/debian/isar/build.tmpl index
> 550bdc67..65261e41 100644 ---
> a/meta/recipes-kernel/linux/files/debian/isar/build.tmpl +++
> b/meta/recipes-kernel/linux/files/debian/isar/build.tmpl @@ -10,6
> +10,9 @@ do_build() { # Print a few things that are of particular
> interest print_settings
>  
> +    # Trace what we do here
> +    set -x

Maybe make that 
printenv | grep -q BB_VERBOSE_LOGS && set -x

as seen in the common.sh for other debian packages

Henning

> +
>      # Process existing kernel configuration to make sure it is
> complete # (use defaults for options that were not specified)
>      ${MAKE} O=${KERNEL_BUILD_DIR} olddefconfig prepare
> @@ -18,9 +21,6 @@ do_build() {
>      KR=$(${MAKE} O=${KERNEL_BUILD_DIR} -s --no-print-directory
> kernelrelease) sed -i "s/@KR@/${KR}/g" ${S}/debian/control
> ${S}/debian/linux-image-${KERNEL_NAME_PROVIDED}.* 
> -    # Trace what we do here
> -    set -x
> -
>      # Build the Linux kernel
>      ${MAKE} O=${KERNEL_BUILD_DIR} ${PARALLEL_MAKE}
>  
> diff --git
> a/meta/recipes-kernel/linux/files/debian/isar/configure.tmpl
> b/meta/recipes-kernel/linux/files/debian/isar/configure.tmpl index
> 58f10326..389c9a85 100644 ---
> a/meta/recipes-kernel/linux/files/debian/isar/configure.tmpl +++
> b/meta/recipes-kernel/linux/files/debian/isar/configure.tmpl @@ -7,6
> +7,9 @@ do_configure() {
>  
> +    # Trace what we do here
> +    set -x
> +
>      # Process kernel config target and fragments
>      ${MAKE} O=${KERNEL_BUILD_DIR} ${KERNEL_CONFIG_TARGET}
>      ./scripts/kconfig/merge_config.sh -O ${KERNEL_BUILD_DIR}/ \


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] linux-custom: Make tracing more consistent
  2022-07-11 13:07 ` Henning Schild
@ 2022-07-11 17:02   ` Jan Kiszka
  2022-07-11 17:19     ` Jan Kiszka
  0 siblings, 1 reply; 6+ messages in thread
From: Jan Kiszka @ 2022-07-11 17:02 UTC (permalink / raw)
  To: Henning Schild; +Cc: isar-users, Cedric Hombourger

On 11.07.22 15:07, Henning Schild wrote:
> Am Sun, 10 Jul 2022 22:51:31 +0200
> schrieb Jan Kiszka <jan.kiszka@siemens.com>:
> 
>> From: Jan Kiszka <jan.kiszka@siemens.com>
>>
>> Include more of the do_build and anything of do_configure into the
>> trace log.
>>
>> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
>> ---
>>  meta/recipes-kernel/linux/files/debian/isar/build.tmpl     | 6 +++---
>>  meta/recipes-kernel/linux/files/debian/isar/configure.tmpl | 3 +++
>>  2 files changed, 6 insertions(+), 3 deletions(-)
>>
>> diff --git a/meta/recipes-kernel/linux/files/debian/isar/build.tmpl
>> b/meta/recipes-kernel/linux/files/debian/isar/build.tmpl index
>> 550bdc67..65261e41 100644 ---
>> a/meta/recipes-kernel/linux/files/debian/isar/build.tmpl +++
>> b/meta/recipes-kernel/linux/files/debian/isar/build.tmpl @@ -10,6
>> +10,9 @@ do_build() { # Print a few things that are of particular
>> interest print_settings
>>  
>> +    # Trace what we do here
>> +    set -x
> 
> Maybe make that 
> printenv | grep -q BB_VERBOSE_LOGS && set -x
> 
> as seen in the common.sh for other debian packages

Will try that.

Jan

-- 
Siemens AG, Technology
Competence Center Embedded Linux

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] linux-custom: Make tracing more consistent
  2022-07-11 17:02   ` Jan Kiszka
@ 2022-07-11 17:19     ` Jan Kiszka
  2022-07-12  7:43       ` Henning Schild
  0 siblings, 1 reply; 6+ messages in thread
From: Jan Kiszka @ 2022-07-11 17:19 UTC (permalink / raw)
  To: Henning Schild; +Cc: isar-users, Cedric Hombourger

On 11.07.22 19:02, Jan Kiszka wrote:
> On 11.07.22 15:07, Henning Schild wrote:
>> Am Sun, 10 Jul 2022 22:51:31 +0200
>> schrieb Jan Kiszka <jan.kiszka@siemens.com>:
>>
>>> From: Jan Kiszka <jan.kiszka@siemens.com>
>>>
>>> Include more of the do_build and anything of do_configure into the
>>> trace log.
>>>
>>> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
>>> ---
>>>  meta/recipes-kernel/linux/files/debian/isar/build.tmpl     | 6 +++---
>>>  meta/recipes-kernel/linux/files/debian/isar/configure.tmpl | 3 +++
>>>  2 files changed, 6 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/meta/recipes-kernel/linux/files/debian/isar/build.tmpl
>>> b/meta/recipes-kernel/linux/files/debian/isar/build.tmpl index
>>> 550bdc67..65261e41 100644 ---
>>> a/meta/recipes-kernel/linux/files/debian/isar/build.tmpl +++
>>> b/meta/recipes-kernel/linux/files/debian/isar/build.tmpl @@ -10,6
>>> +10,9 @@ do_build() { # Print a few things that are of particular
>>> interest print_settings
>>>  
>>> +    # Trace what we do here
>>> +    set -x
>>
>> Maybe make that 
>> printenv | grep -q BB_VERBOSE_LOGS && set -x
>>
>> as seen in the common.sh for other debian packages
> 
> Will try that.
> 

Nope, BB_VERBOSE_LOGS does not make it into the sbuild environment.

Jan

-- 
Siemens AG, Technology
Competence Center Embedded Linux

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] linux-custom: Make tracing more consistent
  2022-07-11 17:19     ` Jan Kiszka
@ 2022-07-12  7:43       ` Henning Schild
  0 siblings, 0 replies; 6+ messages in thread
From: Henning Schild @ 2022-07-12  7:43 UTC (permalink / raw)
  To: Jan Kiszka; +Cc: isar-users, Cedric Hombourger

Am Mon, 11 Jul 2022 19:19:21 +0200
schrieb Jan Kiszka <jan.kiszka@siemens.com>:

> On 11.07.22 19:02, Jan Kiszka wrote:
> > On 11.07.22 15:07, Henning Schild wrote:  
> >> Am Sun, 10 Jul 2022 22:51:31 +0200
> >> schrieb Jan Kiszka <jan.kiszka@siemens.com>:
> >>  
> >>> From: Jan Kiszka <jan.kiszka@siemens.com>
> >>>
> >>> Include more of the do_build and anything of do_configure into the
> >>> trace log.
> >>>
> >>> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
> >>> ---
> >>>  meta/recipes-kernel/linux/files/debian/isar/build.tmpl     | 6
> >>> +++--- meta/recipes-kernel/linux/files/debian/isar/configure.tmpl
> >>> | 3 +++ 2 files changed, 6 insertions(+), 3 deletions(-)
> >>>
> >>> diff --git
> >>> a/meta/recipes-kernel/linux/files/debian/isar/build.tmpl
> >>> b/meta/recipes-kernel/linux/files/debian/isar/build.tmpl index
> >>> 550bdc67..65261e41 100644 ---
> >>> a/meta/recipes-kernel/linux/files/debian/isar/build.tmpl +++
> >>> b/meta/recipes-kernel/linux/files/debian/isar/build.tmpl @@ -10,6
> >>> +10,9 @@ do_build() { # Print a few things that are of particular
> >>> interest print_settings 
> >>> +    # Trace what we do here
> >>> +    set -x  
> >>
> >> Maybe make that 
> >> printenv | grep -q BB_VERBOSE_LOGS && set -x
> >>
> >> as seen in the common.sh for other debian packages  
> > 
> > Will try that.
> >   
> 
> Nope, BB_VERBOSE_LOGS does not make it into the sbuild environment.

Maybe that should be fixed.

Henning

> Jan
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] linux-custom: Make tracing more consistent
  2022-07-10 20:51 [PATCH] linux-custom: Make tracing more consistent Jan Kiszka
  2022-07-11 13:07 ` Henning Schild
@ 2022-07-19 18:55 ` Anton Mikanovich
  1 sibling, 0 replies; 6+ messages in thread
From: Anton Mikanovich @ 2022-07-19 18:55 UTC (permalink / raw)
  To: Jan Kiszka, isar-users; +Cc: Cedric Hombourger

10.07.2022 23:51, Jan Kiszka wrote:
> From: Jan Kiszka <jan.kiszka@siemens.com>
>
> Include more of the do_build and anything of do_configure into the trace
> log.
>
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>

Applied to next, thanks.


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-07-19 18:56 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-10 20:51 [PATCH] linux-custom: Make tracing more consistent Jan Kiszka
2022-07-11 13:07 ` Henning Schild
2022-07-11 17:02   ` Jan Kiszka
2022-07-11 17:19     ` Jan Kiszka
2022-07-12  7:43       ` Henning Schild
2022-07-19 18:55 ` Anton Mikanovich

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox