* [U-Boot] [PATCH 1/2] travis-ci: set env vars to name jobs
@ 2016-10-24 22:41 Stephen Warren
2016-10-24 22:41 ` [U-Boot] [PATCH 2/2] travis-ci: remove duplicate build Stephen Warren
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Stephen Warren @ 2016-10-24 22:41 UTC (permalink / raw)
To: u-boot
From: Stephen Warren <swarren@nvidia.com>
Travis CI names sub-jobs after the first environment variable that is set
for a script. This doesn't produce meaningful results for any of the non-
buildman jobs. Add a dummy variable to give the jobs meaningful names.
Signed-off-by: Stephen Warren <swarren@nvidia.com>
---
.travis.yml | 24 +++++++++++++++++++-----
1 file changed, 19 insertions(+), 5 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index d65c5705875f..a172180a9953 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -160,30 +160,42 @@ matrix:
# QA jobs for code analytics
# static code analysis with cppcheck (we can add --enable=all later)
- - script:
+ - env:
+ - JOB="cppcheck"
+ script:
- cppcheck --force --quiet --inline-suppr .
# search for TODO within source tree
- - script:
+ - env:
+ - JOB="grep TODO"
+ script:
- grep -r TODO .
# search for FIXME within source tree
- - script:
+ - env:
+ - JOB="grep FIXME HACK"
+ script:
- grep -r FIXME .
# search for HACK within source tree and ignore HACKKIT board
script:
- grep -r HACK . | grep -v HACKKIT
# some statistics about the code base
- - script:
+ - env:
+ - JOB="sloccount"
+ script:
- sloccount .
# test/py
- - script:
+ - env:
+ - JOB="test.py sandbox"
+ script:
- ./test/py/test.py --bd sandbox --build
- env:
+ - JOB="test.py ARM"
- CROSS_COMPILE="/usr/bin/arm-linux-gnueabihf-"
script:
- ./test/py/test.py --bd vexpress_ca15_tc2 --id qemu --build;
./test/py/test.py --bd vexpress_ca9x4 --id qemu --build;
./test/py/test.py --bd integratorcp_cm926ejs --id qemu --build;
- env:
+ - JOB="test.py MIPS"
- TOOLCHAIN="mips"
CROSS_COMPILE="${HOME}/.buildman-toolchains/gcc-4.9.0-nolibc/mips-linux/bin/mips-linux-"
script:
@@ -192,10 +204,12 @@ matrix:
./test/py/test.py --bd qemu_mips64 --build -k 'not sleep';
./test/py/test.py --bd qemu_mips64el --build -k 'not sleep';
- env:
+ - JOB="test.py PowerPC"
- CROSS_COMPILE="/usr/bin/powerpc-linux-gnu-"
script:
- ./test/py/test.py --bd qemu-ppce500 --build -k 'not sleep'
- env:
+ - JOB="test.py x86-64"
- TOOLCHAIN="x86_64"
BUILD_ROM=yes
CROSS_COMPILE="${HOME}/.buildman-toolchains/gcc-4.9.0-nolibc/x86_64-linux/bin/x86_64-linux-"
--
2.10.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH 2/2] travis-ci: remove duplicate build
2016-10-24 22:41 [U-Boot] [PATCH 1/2] travis-ci: set env vars to name jobs Stephen Warren
@ 2016-10-24 22:41 ` Stephen Warren
2016-10-24 23:04 ` Tom Rini
2016-10-29 17:41 ` [U-Boot] [U-Boot,2/2] " Tom Rini
2016-10-24 23:03 ` [U-Boot] [PATCH 1/2] travis-ci: set env vars to name jobs Tom Rini
2016-10-29 17:40 ` [U-Boot] [U-Boot,1/2] " Tom Rini
2 siblings, 2 replies; 8+ messages in thread
From: Stephen Warren @ 2016-10-24 22:41 UTC (permalink / raw)
To: u-boot
From: Stephen Warren <swarren@nvidia.com>
There were two sub-jobs to build arm1136. Remove the duplicate.
Signed-off-by: Stephen Warren <swarren@nvidia.com>
---
.travis.yml | 2 --
1 file changed, 2 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index a172180a9953..00be5ff20396 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -85,8 +85,6 @@ matrix:
- env:
- BUILDMAN="arm1136"
- env:
- - BUILDMAN="arm1136"
- - env:
- BUILDMAN="arm1176"
- env:
- BUILDMAN="arm720t"
--
2.10.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH 1/2] travis-ci: set env vars to name jobs
2016-10-24 22:41 [U-Boot] [PATCH 1/2] travis-ci: set env vars to name jobs Stephen Warren
2016-10-24 22:41 ` [U-Boot] [PATCH 2/2] travis-ci: remove duplicate build Stephen Warren
@ 2016-10-24 23:03 ` Tom Rini
2016-10-24 23:18 ` Stephen Warren
2016-10-29 17:40 ` [U-Boot] [U-Boot,1/2] " Tom Rini
2 siblings, 1 reply; 8+ messages in thread
From: Tom Rini @ 2016-10-24 23:03 UTC (permalink / raw)
To: u-boot
On Mon, Oct 24, 2016 at 04:41:48PM -0600, Stephen Warren wrote:
> From: Stephen Warren <swarren@nvidia.com>
>
> Travis CI names sub-jobs after the first environment variable that is set
> for a script. This doesn't produce meaningful results for any of the non-
> buildman jobs. Add a dummy variable to give the jobs meaningful names.
>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
I'm not sure. This un-does in part killing off TEST_CMD as a hook. At
the end of the day, imho, what is important from travis-ci are the
emails it sends (which are starting and pass/fail/interrupted) and the
other notifications.
On the other hand it does seem to be common practice to do something
along these lines. So, in the end, I don't know.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161024/57d564c4/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH 2/2] travis-ci: remove duplicate build
2016-10-24 22:41 ` [U-Boot] [PATCH 2/2] travis-ci: remove duplicate build Stephen Warren
@ 2016-10-24 23:04 ` Tom Rini
2016-10-29 17:41 ` [U-Boot] [U-Boot,2/2] " Tom Rini
1 sibling, 0 replies; 8+ messages in thread
From: Tom Rini @ 2016-10-24 23:04 UTC (permalink / raw)
To: u-boot
On Mon, Oct 24, 2016 at 04:41:49PM -0600, Stephen Warren wrote:
> From: Stephen Warren <swarren@nvidia.com>
>
> There were two sub-jobs to build arm1136. Remove the duplicate.
>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161024/80affe77/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH 1/2] travis-ci: set env vars to name jobs
2016-10-24 23:03 ` [U-Boot] [PATCH 1/2] travis-ci: set env vars to name jobs Tom Rini
@ 2016-10-24 23:18 ` Stephen Warren
2016-10-25 0:20 ` Tom Rini
0 siblings, 1 reply; 8+ messages in thread
From: Stephen Warren @ 2016-10-24 23:18 UTC (permalink / raw)
To: u-boot
On 10/24/2016 05:03 PM, Tom Rini wrote:
> On Mon, Oct 24, 2016 at 04:41:48PM -0600, Stephen Warren wrote:
>
>> From: Stephen Warren <swarren@nvidia.com>
>>
>> Travis CI names sub-jobs after the first environment variable that is set
>> for a script. This doesn't produce meaningful results for any of the non-
>> buildman jobs. Add a dummy variable to give the jobs meaningful names.
>>
>> Signed-off-by: Stephen Warren <swarren@nvidia.com>
>
> I'm not sure. This un-does in part killing off TEST_CMD as a hook. At
> the end of the day, imho, what is important from travis-ci are the
> emails it sends (which are starting and pass/fail/interrupted) and the
> other notifications.
I don't believe I've ever received emails from Travis, although I
haven't used it in a while so perhaps it's a new feature.
Either way, I do like to look at the web UI to watch/check status
sometimes, and certainly would after a failure to track down the
problem, and this change makes the job list there possible to interpret,
whereas it isn't without it:
https://travis-ci.org/swarren/u-boot/builds/170278972
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH 1/2] travis-ci: set env vars to name jobs
2016-10-24 23:18 ` Stephen Warren
@ 2016-10-25 0:20 ` Tom Rini
0 siblings, 0 replies; 8+ messages in thread
From: Tom Rini @ 2016-10-25 0:20 UTC (permalink / raw)
To: u-boot
On Mon, Oct 24, 2016 at 05:18:24PM -0600, Stephen Warren wrote:
> On 10/24/2016 05:03 PM, Tom Rini wrote:
> >On Mon, Oct 24, 2016 at 04:41:48PM -0600, Stephen Warren wrote:
> >
> >>From: Stephen Warren <swarren@nvidia.com>
> >>
> >>Travis CI names sub-jobs after the first environment variable that is set
> >>for a script. This doesn't produce meaningful results for any of the non-
> >>buildman jobs. Add a dummy variable to give the jobs meaningful names.
> >>
> >>Signed-off-by: Stephen Warren <swarren@nvidia.com>
> >
> >I'm not sure. This un-does in part killing off TEST_CMD as a hook. At
> >the end of the day, imho, what is important from travis-ci are the
> >emails it sends (which are starting and pass/fail/interrupted) and the
> >other notifications.
>
> I don't believe I've ever received emails from Travis, although I
> haven't used it in a while so perhaps it's a new feature.
Ah, https://docs.travis-ci.com/user/notifications#Email-notifications
explains and it's setup by default to not spam uninterested parties.
Which I think in turn means it also doesn't email interested parties
easily, sadly.
> Either way, I do like to look at the web UI to watch/check status
> sometimes, and certainly would after a failure to track down the
> problem, and this change makes the job list there possible to
> interpret, whereas it isn't without it:
>
> https://travis-ci.org/swarren/u-boot/builds/170278972
Yeah, I spent a lot of time with the UI when I was doing the last
series. And since it is indeed harder to get email notifications than I
would like, this makes sense too.
Reviewed-by: Tom Rini <trini@konsulko.com>
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161024/c2c7fc46/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [U-Boot,1/2] travis-ci: set env vars to name jobs
2016-10-24 22:41 [U-Boot] [PATCH 1/2] travis-ci: set env vars to name jobs Stephen Warren
2016-10-24 22:41 ` [U-Boot] [PATCH 2/2] travis-ci: remove duplicate build Stephen Warren
2016-10-24 23:03 ` [U-Boot] [PATCH 1/2] travis-ci: set env vars to name jobs Tom Rini
@ 2016-10-29 17:40 ` Tom Rini
2 siblings, 0 replies; 8+ messages in thread
From: Tom Rini @ 2016-10-29 17:40 UTC (permalink / raw)
To: u-boot
On Mon, Oct 24, 2016 at 04:41:48PM -0600, Stephen Warren wrote:
> From: Stephen Warren <swarren@nvidia.com>
>
> Travis CI names sub-jobs after the first environment variable that is set
> for a script. This doesn't produce meaningful results for any of the non-
> buildman jobs. Add a dummy variable to give the jobs meaningful names.
>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161029/8064611c/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [U-Boot,2/2] travis-ci: remove duplicate build
2016-10-24 22:41 ` [U-Boot] [PATCH 2/2] travis-ci: remove duplicate build Stephen Warren
2016-10-24 23:04 ` Tom Rini
@ 2016-10-29 17:41 ` Tom Rini
1 sibling, 0 replies; 8+ messages in thread
From: Tom Rini @ 2016-10-29 17:41 UTC (permalink / raw)
To: u-boot
On Mon, Oct 24, 2016 at 04:41:49PM -0600, Stephen Warren wrote:
> From: Stephen Warren <swarren@nvidia.com>
>
> There were two sub-jobs to build arm1136. Remove the duplicate.
>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161029/e3a7f4f3/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-10-29 17:41 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-24 22:41 [U-Boot] [PATCH 1/2] travis-ci: set env vars to name jobs Stephen Warren
2016-10-24 22:41 ` [U-Boot] [PATCH 2/2] travis-ci: remove duplicate build Stephen Warren
2016-10-24 23:04 ` Tom Rini
2016-10-29 17:41 ` [U-Boot] [U-Boot,2/2] " Tom Rini
2016-10-24 23:03 ` [U-Boot] [PATCH 1/2] travis-ci: set env vars to name jobs Tom Rini
2016-10-24 23:18 ` Stephen Warren
2016-10-25 0:20 ` Tom Rini
2016-10-29 17:40 ` [U-Boot] [U-Boot,1/2] " Tom Rini
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.