Skip to contentSkip to conversation

.travis.yml: explicitly request sudo

PATCH In review Stephen Warren 4 participants 6 messages Feb 3, 2016 – Feb 8, 2016

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 .travis.yml: explicitly request sudo

From: Stephen Warren <[email protected]>

Without this, builds default to using new Travis CI infra-structure which does no allow sudo. The builds need sudo in order to install the ELDK compilers. Consequently, almost all builds fail without this.

I suspect that existing Travis CI users have not noticed this because their accounts or builds have been grand-fathered into backwards- compatible default settings, whereas I just set up a new build from scratch and received new default settings.

Cc Wolfgang Denk <[email protected]>
Cc Heiko Schocher <[email protected]>
Cc Tom Rini <[email protected]>
Cc Daniel Schwierzeck <[email protected]>
Cc Andreas F?rber <[email protected]>
---
 .travis.yml | 2 ++
 1 file changed, 2 insertions(+)
.travis.yml +2 -0
index 67674e2f5878..8caaeb34553e 100644
@@ -3,6 +3,8 @@
 
 # build U-Boot on Travis CI - https://travis-ci.org/
 
+sudo: true
+
 language: c
 
 addons:
Signature · 1 line
2.7.0
#1 lore raw

PATCH 1/1 .travis.yml: explicitly request sudo

Am 03.02.2016 um 18:41 schrieb Stephen Warren:

Quoted 11 lines

From: Stephen Warren <[email protected]>

Without this, builds default to using new Travis CI infra-structure which does no allow sudo. The builds need sudo in order to install the ELDK compilers. Consequently, almost all builds fail without this.

I suspect that existing Travis CI users have not noticed this because their accounts or builds have been grand-fathered into backwards- compatible default settings, whereas I just set up a new build from scratch and received new default settings.

Quoted 5 lines
Cc Wolfgang Denk <[email protected]>
Cc Heiko Schocher <[email protected]>
Cc Tom Rini <[email protected]>
Cc Daniel Schwierzeck <[email protected]>
Cc Andreas F?rber <[email protected]>

Actually I have no clue about Travis...

Cheers,
Andreas

Quoted 4 lines
---
 .travis.yml | 2 ++
 1 file changed, 2 insertions(+)

[snip]

Signature · 2 lines
SUSE Linux GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany
GF: Felix Imend?rffer, Jane Smithard, Graham Norton; HRB 21284 (AG N?rnberg)
#2 lore raw

PATCH 1/1 .travis.yml: explicitly request sudo

In reply to Stephen Warren

Hello Stephen,

Am 03.02.2016 um 18:41 schrieb Stephen Warren:

Quoted 17 lines

From: Stephen Warren <[email protected]>

Without this, builds default to using new Travis CI infra-structure which does no allow sudo. The builds need sudo in order to install the ELDK compilers. Consequently, almost all builds fail without this.

I suspect that existing Travis CI users have not noticed this because their accounts or builds have been grand-fathered into backwards- compatible default settings, whereas I just set up a new build from scratch and received new default settings.

Cc Wolfgang Denk <[email protected]>
Cc Heiko Schocher <[email protected]>
Cc Tom Rini <[email protected]>
Cc Daniel Schwierzeck <[email protected]>
Cc Andreas F?rber <[email protected]>
Quoted 3 lines
---
  .travis.yml | 2 ++
  1 file changed, 2 insertions(+)

Did you tried it with current ML? Or at last with the patch from Roger?

http://lists.denx.de/pipermail/u-boot/2016-January/243115.html
[U-Boot] [PATCH] .travis.yml: rework according to new travis-ci infra

If I look into:
https://travis-ci.org/u-boot/u-boot

There are only mips and m68k for which the build fail ...

bye,
Heiko

Quoted 15 lines
.travis.yml +2 -0
index 67674e2f5878..8caaeb34553e 100644
@@ -3,6 +3,8 @@
 
  # build U-Boot on Travis CI - https://travis-ci.org/
 
+sudo: true
+
  language: c
 
  addons:
Signature · 2 lines
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
#3 lore raw

PATCH 1/1 .travis.yml: explicitly request sudo

Quoted 14 lines · Heiko Schocher

Hello Stephen,

Am 03.02.2016 um 18:41 schrieb Stephen Warren:

Quoted 10 lines

From: Stephen Warren <[email protected]>

Without this, builds default to using new Travis CI infra-structure which does no allow sudo. The builds need sudo in order to install the ELDK compilers. Consequently, almost all builds fail without this.

I suspect that existing Travis CI users have not noticed this because their accounts or builds have been grand-fathered into backwards- compatible default settings, whereas I just set up a new build from scratch and received new default settings.

Quoted 5 lines

Did you tried it with current ML? Or at last with the patch from Roger?

http://lists.denx.de/pipermail/u-boot/2016-January/243115.html
[U-Boot] [PATCH] .travis.yml: rework according to new travis-ci infra

Both:-) That patch is already applied in mainline, and was included in the branch I pushed "to" Travis.

Quoted 4 lines

If I look into:
https://travis-ci.org/u-boot/u-boot

There are only mips and m68k for which the build fail ...

Yes. With the addition of the sudo tag, the same is now true for my branch.

#4 lore raw

PATCH 1/1 .travis.yml: explicitly request sudo

Hello Stephen,

Am 04.02.2016 um 17:33 schrieb Stephen Warren:

Quoted 21 lines · Heiko Schocher
Quoted 14 lines

Hello Stephen,

Am 03.02.2016 um 18:41 schrieb Stephen Warren:

Quoted 10 lines

From: Stephen Warren <[email protected]>

Without this, builds default to using new Travis CI infra-structure which does no allow sudo. The builds need sudo in order to install the ELDK compilers. Consequently, almost all builds fail without this.

I suspect that existing Travis CI users have not noticed this because their accounts or builds have been grand-fathered into backwards- compatible default settings, whereas I just set up a new build from scratch and received new default settings.

Quoted 5 lines

Did you tried it with current ML? Or at last with the patch from Roger?

http://lists.denx.de/pipermail/u-boot/2016-January/243115.html
[U-Boot] [PATCH] .travis.yml: rework according to new travis-ci infra

Quoted 8 lines

Both:-) That patch is already applied in mainline, and was included in the branch I pushed "to" Travis.

Quoted 4 lines

If I look into:
https://travis-ci.org/u-boot/u-boot

There are only mips and m68k for which the build fail ...

Yes. With the addition of the sudo tag, the same is now true for my branch.

Great! So I would say:

bye,
Heiko

Signature · 2 lines
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
#5 lore raw
In reply to Stephen Warren
Quoted 11 lines · Stephen Warren

From: Stephen Warren <[email protected]>

Without this, builds default to using new Travis CI infra-structure which does no allow sudo. The builds need sudo in order to install the ELDK compilers. Consequently, almost all builds fail without this.

I suspect that existing Travis CI users have not noticed this because their accounts or builds have been grand-fathered into backwards- compatible default settings, whereas I just set up a new build from scratch and received new default settings.

Quoted 7 lines
Cc Wolfgang Denk <[email protected]>
Cc Heiko Schocher <[email protected]>
Cc Tom Rini <[email protected]>
Cc Daniel Schwierzeck <[email protected]>
Cc Andreas F?rber <[email protected]>

Applied to u-boot/master, thanks!

Signature · 8 lines
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160208/92994415/attachment.sig>