All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
To: Alexander Kanavin <alexander.kanavin@linux.intel.com>,
	Paulo Neves <ptsneves@gmail.com>,
	Yocto list discussion <yocto@yoctoproject.org>
Subject: Re: ROOTFS_RPM_DEBUG undocumented
Date: Thu, 3 May 2018 13:37:03 +0000	[thread overview]
Message-ID: <50b98492ad0d40a0b8f32933c371125f@XBOX02.axis.com> (raw)
In-Reply-To: <70e7860b-5b9f-45c7-5436-73859b0d3706@linux.intel.com>

> -----Original Message-----
> From: yocto-bounces@yoctoproject.org [mailto:yocto-
> bounces@yoctoproject.org] On Behalf Of Alexander Kanavin
> Sent: den 3 maj 2018 15:04
> To: Paulo Neves <ptsneves@gmail.com>; Yocto list discussion
> <yocto@yoctoproject.org>
> Subject: Re: [yocto] ROOTFS_RPM_DEBUG undocumented
> 
> On 05/03/2018 01:42 PM, Paulo Neves wrote:
> > # Add ROOTFS_RPM_DEBUG to the documentation;
> 
> I'd rather get rid of it, the less variables the better :)
> 
> > # Detect if we are running with debug output and enable the debugging
> > output. This is the most elegant solution but I do not know how to
> > detect if debug log level is turned on;
> 
> I don't think you are meant to detect that; bitbake's debug level is
> not
> meant to affect what is being printed, but merely whether it's printed
> or discarded.
> 
> > # Have dnf always print in verbose mode and print the output to
> > bb.debug instead of bb.note.
> 
> I think this is the best solution actually.

I disagree with changing bb.note to bb.debug for this. It is very good 
to be able to look in log.do_rootfs to see what dnf has done (we have 
ROOTFS_RPM_DEBUG set by default). Changing the output to debug level 
would mean having to run bitbake with -D, which is anything but 
desirable.

> > I am happy to provide a patch upon decision or suggestions.
> 
> Thank you :) Do measure the performance before and after as we don't
> want to introduce a significant slowdown here.
> 
> Alex

//Peter



  reply	other threads:[~2018-05-03 13:37 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-03 10:42 ROOTFS_RPM_DEBUG undocumented Paulo Neves
2018-05-03 13:03 ` Alexander Kanavin
2018-05-03 13:37   ` Peter Kjellerstedt [this message]
2018-05-03 14:21     ` Alexander Kanavin
2018-05-04 12:42   ` Paulo Neves
2018-05-04 12:41     ` Alexander Kanavin
2018-05-04 12:43       ` Alexander Kanavin
2018-05-04 12:50       ` Scott Rifenbark
2018-05-04 12:45         ` Alexander Kanavin
2018-05-04 12:55       ` Paulo Neves
2018-05-07  9:03       ` Mirza Krak
2018-05-07 10:12         ` Alexander Kanavin
2018-05-08  9:33           ` Peter Kjellerstedt
2018-05-08 10:29             ` Alexander Kanavin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50b98492ad0d40a0b8f32933c371125f@XBOX02.axis.com \
    --to=peter.kjellerstedt@axis.com \
    --cc=alexander.kanavin@linux.intel.com \
    --cc=ptsneves@gmail.com \
    --cc=yocto@yoctoproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.