Re: [PATCH v5 10/16] buildman: Detect binman reporting missing blobs

2022-12-05 Thread Simon Glass
Hi Tom, On Tue, 6 Dec 2022 at 12:56, Tom Rini wrote: > > On Tue, Dec 06, 2022 at 12:55:08PM +1300, Simon Glass wrote: > > Hi Tom, > > > > On Tue, 6 Dec 2022 at 12:46, Tom Rini wrote: > > > > > > On Mon, Dec 05, 2022 at 11:43:24PM +, Peter Robinson wrote: > > > > On Mon, Dec 5, 2022 at 11:35

Re: [PATCH v5 10/16] buildman: Detect binman reporting missing blobs

2022-12-05 Thread Tom Rini
On Tue, Dec 06, 2022 at 12:55:08PM +1300, Simon Glass wrote: > Hi Tom, > > On Tue, 6 Dec 2022 at 12:46, Tom Rini wrote: > > > > On Mon, Dec 05, 2022 at 11:43:24PM +, Peter Robinson wrote: > > > On Mon, Dec 5, 2022 at 11:35 PM Tom Rini wrote: > > > > > > > > On Mon, Dec 05, 2022 at 11:29:30PM

Re: [PATCH v5 10/16] buildman: Detect binman reporting missing blobs

2022-12-05 Thread Simon Glass
Hi Tom, On Tue, 6 Dec 2022 at 12:46, Tom Rini wrote: > > On Mon, Dec 05, 2022 at 11:43:24PM +, Peter Robinson wrote: > > On Mon, Dec 5, 2022 at 11:35 PM Tom Rini wrote: > > > > > > On Mon, Dec 05, 2022 at 11:29:30PM +, Peter Robinson wrote: > > > > On Mon, Dec 5, 2022 at 11:23 PM Tom Rin

Re: [PATCH v5 10/16] buildman: Detect binman reporting missing blobs

2022-12-05 Thread Peter Robinson
On Mon, Dec 5, 2022 at 11:46 PM Tom Rini wrote: > > On Mon, Dec 05, 2022 at 11:43:24PM +, Peter Robinson wrote: > > On Mon, Dec 5, 2022 at 11:35 PM Tom Rini wrote: > > > > > > On Mon, Dec 05, 2022 at 11:29:30PM +, Peter Robinson wrote: > > > > On Mon, Dec 5, 2022 at 11:23 PM Tom Rini wro

Re: [PATCH v5 10/16] buildman: Detect binman reporting missing blobs

2022-12-05 Thread Tom Rini
On Mon, Dec 05, 2022 at 11:43:24PM +, Peter Robinson wrote: > On Mon, Dec 5, 2022 at 11:35 PM Tom Rini wrote: > > > > On Mon, Dec 05, 2022 at 11:29:30PM +, Peter Robinson wrote: > > > On Mon, Dec 5, 2022 at 11:23 PM Tom Rini wrote: > > > > > > > > On Mon, Dec 05, 2022 at 11:13:03PM +,

Re: [PATCH v5 10/16] buildman: Detect binman reporting missing blobs

2022-12-05 Thread Peter Robinson
On Mon, Dec 5, 2022 at 11:35 PM Tom Rini wrote: > > On Mon, Dec 05, 2022 at 11:29:30PM +, Peter Robinson wrote: > > On Mon, Dec 5, 2022 at 11:23 PM Tom Rini wrote: > > > > > > On Mon, Dec 05, 2022 at 11:13:03PM +, Peter Robinson wrote: > > > > On Thu, Nov 10, 2022 at 2:17 AM Simon Glass

Re: [PATCH v5 10/16] buildman: Detect binman reporting missing blobs

2022-12-05 Thread Tom Rini
On Mon, Dec 05, 2022 at 11:29:30PM +, Peter Robinson wrote: > On Mon, Dec 5, 2022 at 11:23 PM Tom Rini wrote: > > > > On Mon, Dec 05, 2022 at 11:13:03PM +, Peter Robinson wrote: > > > On Thu, Nov 10, 2022 at 2:17 AM Simon Glass wrote: > > > > > > > > Buildman should consider a build as a

Re: [PATCH v5 10/16] buildman: Detect binman reporting missing blobs

2022-12-05 Thread Peter Robinson
On Mon, Dec 5, 2022 at 11:23 PM Tom Rini wrote: > > On Mon, Dec 05, 2022 at 11:13:03PM +, Peter Robinson wrote: > > On Thu, Nov 10, 2022 at 2:17 AM Simon Glass wrote: > > > > > > Buildman should consider a build as a success (with warnings) if missing > > > blobs have been dealt with by binma

Re: [PATCH v5 10/16] buildman: Detect binman reporting missing blobs

2022-12-05 Thread Tom Rini
On Mon, Dec 05, 2022 at 11:13:03PM +, Peter Robinson wrote: > On Thu, Nov 10, 2022 at 2:17 AM Simon Glass wrote: > > > > Buildman should consider a build as a success (with warnings) if missing > > blobs have been dealt with by binman, even though buildman itself returns > > and error code ove

Re: [PATCH v5 10/16] buildman: Detect binman reporting missing blobs

2022-12-05 Thread Peter Robinson
On Thu, Nov 10, 2022 at 2:17 AM Simon Glass wrote: > > Buildman should consider a build as a success (with warnings) if missing > blobs have been dealt with by binman, even though buildman itself returns > and error code overall. This is how other warnings are dealt with. > > We cannot easily acce

Re: [PATCH v5 10/16] buildman: Detect binman reporting missing blobs

2022-11-22 Thread Simon Glass
Buildman should consider a build as a success (with warnings) if missing blobs have been dealt with by binman, even though buildman itself returns and error code overall. This is how other warnings are dealt with. We cannot easily access the 103 exit code, so detect the problem in the output. Wit

[PATCH v5 10/16] buildman: Detect binman reporting missing blobs

2022-11-09 Thread Simon Glass
Buildman should consider a build as a success (with warnings) if missing blobs have been dealt with by binman, even though buildman itself returns and error code overall. This is how other warnings are dealt with. We cannot easily access the 103 exit code, so detect the problem in the output. Wit