On Tue, Jul 23, 2019 at 08:08:03PM -0700, Simon Glass wrote: > Hi Tom, > > On Tue, 23 Jul 2019 at 11:16, Tom Rini <tr...@konsulko.com> wrote: > > > > On Mon, Jul 22, 2019 at 08:48:45AM -0600, Simon Glass wrote: > > > > > Hi Tom, > > > > > > Build is here: https://travis-ci.org/sglass68/u-boot/builds/561552377 > > > > > > The following changes since commit > > > 0de815356474912ef5bef9a69f0327a5a93bb2c2: > > > > > > Merge branch '2019-07-17-master-imports' (2019-07-18 11:31:37 -0400) > > > > > > are available in the Git repository at: > > > > > > git://git.denx.de/u-boot-dm.git tags/dm-pull-22jul19 > > > > > > for you to fetch changes up to 857ad7985ff63989c3c7feff56c2dc353d7d7c9a: > > > > > > dm: device: make power domain calls optional (2019-07-20 19:50:44 -0600) > > > > > > > First, in my LLVM-7 test: > > /home/trini/u-boot/u-boot/tools/ifwitool.c:584:23: warning: unused function > > 'read_le8' [-Wunused-function] > > static inline uint8_t read_le8(const void *src) > > ^ > > /home/trini/u-boot/u-boot/tools/ifwitool.c:695:20: warning: unused function > > 'zero_n' [-Wunused-function] > > static inline void zero_n(void *dest, size_t n) > > > > Which is simple to fix. But I don't see why it's not being caught in > > the travis test off-hand. > > I'm not sure either. Possible it depends on the compiler used?
I'm concerned the travis job isn't running LLVM like we think it is. > > And I further see I didn't get LLVM-7 > > included in GitLab at all yet. > > > > Next however I see: > > https://gitlab.denx.de/u-boot/u-boot/-/jobs/1222 > > which is the binman tests failing and needs to be fixed. > > Oh dear, I don't seem to have any idea about gitlab yet. Have updated > the gitlab config file. Should I stop using travis completely now? Long term I think GitLab CI is going to be better for custodians as we'll have more build resources available and thus the jobs will complete quicker. I don't know why the binman testsuite doesn't fail in travis like it does there. The good news is the gitlab job runs in a defined docker container you can pull and try locally :) -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
