[tbot] [DENX] tbot: board hangs if no autoload

Claudius Heine ch at denx.de
Fri Nov 16 11:50:29 UTC 2018


Hi Heiko,

Quoting Heiko Schocher (2018-11-16 11:18:00)
> Hello Claudius,
> 
> Am 16.11.2018 um 09:08 schrieb Claudius Heine:
> > Hi Heiko,
> > 
> > Quoting Heiko Schocher (2018-11-15 13:01:47)
> >>> But what developers surely need (and this is why I put functional tests
> >>> on top of priorities) is a way to validate what they did and to have
> >>> regression tests without a lot of effort. And in both of them, tbot excels.
> >>
> >> Isn;t it for example also a valid testcase to ensure, that u-boot for
> >> example compiles?
> >>
> >> Just yesterday I posted a patch on U-Boot ML, which compiled, but dropped
> >> warnings I did not check, because I build U-Boot with bitbake ... :-(
> > 
> > Maybe a bit off-topic, but shouldn't that be solved with `-Werror`?
> 
> Maybe there is an option in bitbake ? I do not know.

IMO that should be the default parameter upstream if that is the goal.

If the upstream project does not care about warnings, then bitbake has
to ignore them too, if the upstream cares about them and adds 'Werror'
to their build, then bitbake should abort the build as well.

From what I see u-boot does not have `Werror` in its default build, so
maybe add it?

In any case it doesn't make sense to check for compile warnings in tbot
itself IMO.

Cheers,
Claudius

--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch at denx.de

           PGP key: 6FF2 E59F 00C6 BC28 31D8 64C1 1173 CB19 9808 B153
                             Keyserver: hkp://pool.sks-keyservers.net
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: signature
URL: <http://lists.denx.de/pipermail/tbot/attachments/20181116/eb69752b/attachment-0001.sig>


More information about the tbot mailing list