From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Ed Bartosh <ed.bartosh@linux.intel.com>,
"Smith, Elliot" <elliot.smith@intel.com>
Cc: bitbake-devel@lists.openembedded.org
Subject: Re: [PATCH 1/2] toaster: Manually retrieve log file location from filesystem
Date: Sun, 11 Oct 2015 08:07:47 +0100 [thread overview]
Message-ID: <1444547267.14364.7.camel@linuxfoundation.org> (raw)
In-Reply-To: <e8ec343ddc6f66086080398f325e058d656b9516.1444213003.git.ed.bartosh@linux.intel.com>
On Wed, 2015-10-07 at 13:20 +0300, Ed Bartosh wrote:
> From: Elliot Smith <elliot.smith@intel.com>
>
> The log file location reported by the BB_CONSOLELOG variable
> does not point to the log location for the current build at
> the time when the BuildStarted event is fired. It
> actually points to the location where the next build will
> log to. This means that the log file paths associated with
> a build in the cooker_log_path field are incorrect, with
> the result that the "Download build log" button doesn't work.
>
> Instead, when a build starts, get the latest-dated log file
> and associate it with the build.
>
> An issue explaining why this is a problem, plus steps to
> demonstrate it:
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=8411
Rather than some of the proposed workarounds, I'd rather like to try and
fix this properly. I've put some quick analysis into the above bug and I
think it shouldn't be too difficult to fix. If anyone wants to test and
send a patch I'd be more than happy for that as I'm travelling at the
moment (doing this form the airport) but I'll try and take a look later
today if I get the change.
Cheers,
Richard
next prev parent reply other threads:[~2015-10-11 7:08 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-07 10:20 [PATCH 0/2] Fix cooker log download button Ed Bartosh
2015-10-07 10:20 ` [PATCH 1/2] toaster: Manually retrieve log file location from filesystem Ed Bartosh
2015-10-11 7:07 ` Richard Purdie [this message]
2015-10-07 10:20 ` [PATCH 2/2] toaster: Hide "Download build log" button if log doesn't exist Ed Bartosh
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=1444547267.14364.7.camel@linuxfoundation.org \
--to=richard.purdie@linuxfoundation.org \
--cc=bitbake-devel@lists.openembedded.org \
--cc=ed.bartosh@linux.intel.com \
--cc=elliot.smith@intel.com \
/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.