* [U-Boot] [PATCH] .travis.yml: explicitly request sudo
@ 2016-02-03 17:41 Stephen Warren
2016-02-03 17:44 ` Andreas Färber
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Stephen Warren @ 2016-02-03 17:41 UTC (permalink / raw)
To: u-boot
From: Stephen Warren <swarren@nvidia.com>
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 <wd@denx.de>
Cc: Heiko Schocher <hs@denx.de>
Cc: Tom Rini <trini@ti.com>
Cc: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
Cc: Andreas F?rber <afaerber@suse.de>
Signed-off-by: Stephen Warren <swarren@nvidia.com>
---
.travis.yml | 2 ++
1 file changed, 2 insertions(+)
diff --git a/.travis.yml b/.travis.yml
index 67674e2f5878..8caaeb34553e 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -3,6 +3,8 @@
# build U-Boot on Travis CI - https://travis-ci.org/
+sudo: true
+
language: c
addons:
--
2.7.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* [U-Boot] [PATCH] .travis.yml: explicitly request sudo
2016-02-03 17:41 [U-Boot] [PATCH] .travis.yml: explicitly request sudo Stephen Warren
@ 2016-02-03 17:44 ` Andreas Färber
2016-02-04 9:00 ` Heiko Schocher
2016-02-08 20:49 ` [U-Boot] " Tom Rini
2 siblings, 0 replies; 6+ messages in thread
From: Andreas Färber @ 2016-02-03 17:44 UTC (permalink / raw)
To: u-boot
Am 03.02.2016 um 18:41 schrieb Stephen Warren:
> From: Stephen Warren <swarren@nvidia.com>
>
> 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 <wd@denx.de>
> Cc: Heiko Schocher <hs@denx.de>
> Cc: Tom Rini <trini@ti.com>
> Cc: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
> Cc: Andreas F?rber <afaerber@suse.de>
Actually I have no clue about Travis...
Cheers,
Andreas
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
> .travis.yml | 2 ++
> 1 file changed, 2 insertions(+)
[snip]
--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany
GF: Felix Imend?rffer, Jane Smithard, Graham Norton; HRB 21284 (AG N?rnberg)
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] .travis.yml: explicitly request sudo
2016-02-03 17:41 [U-Boot] [PATCH] .travis.yml: explicitly request sudo Stephen Warren
2016-02-03 17:44 ` Andreas Färber
@ 2016-02-04 9:00 ` Heiko Schocher
2016-02-04 16:33 ` Stephen Warren
2016-02-08 20:49 ` [U-Boot] " Tom Rini
2 siblings, 1 reply; 6+ messages in thread
From: Heiko Schocher @ 2016-02-04 9:00 UTC (permalink / raw)
To: u-boot
Hello Stephen,
Am 03.02.2016 um 18:41 schrieb Stephen Warren:
> From: Stephen Warren <swarren@nvidia.com>
>
> 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 <wd@denx.de>
> Cc: Heiko Schocher <hs@denx.de>
> Cc: Tom Rini <trini@ti.com>
> Cc: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
> Cc: Andreas F?rber <afaerber@suse.de>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
> .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
>
> diff --git a/.travis.yml b/.travis.yml
> index 67674e2f5878..8caaeb34553e 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -3,6 +3,8 @@
>
> # build U-Boot on Travis CI - https://travis-ci.org/
>
> +sudo: true
> +
> language: c
>
> addons:
>
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] .travis.yml: explicitly request sudo
2016-02-04 9:00 ` Heiko Schocher
@ 2016-02-04 16:33 ` Stephen Warren
2016-02-05 5:30 ` Heiko Schocher
0 siblings, 1 reply; 6+ messages in thread
From: Stephen Warren @ 2016-02-04 16:33 UTC (permalink / raw)
To: u-boot
On 02/04/2016 02:00 AM, Heiko Schocher wrote:
> Hello Stephen,
>
> Am 03.02.2016 um 18:41 schrieb Stephen Warren:
>> From: Stephen Warren <swarren@nvidia.com>
>>
>> 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.
>
> 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.
> 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.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] .travis.yml: explicitly request sudo
2016-02-04 16:33 ` Stephen Warren
@ 2016-02-05 5:30 ` Heiko Schocher
0 siblings, 0 replies; 6+ messages in thread
From: Heiko Schocher @ 2016-02-05 5:30 UTC (permalink / raw)
To: u-boot
Hello Stephen,
Am 04.02.2016 um 17:33 schrieb Stephen Warren:
> On 02/04/2016 02:00 AM, Heiko Schocher wrote:
>> Hello Stephen,
>>
>> Am 03.02.2016 um 18:41 schrieb Stephen Warren:
>>> From: Stephen Warren <swarren@nvidia.com>
>>>
>>> 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.
>>
>> 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.
>
>> 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:
Acked-by: Heiko Schocher <hs@denx.de>
bye,
Heiko
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] .travis.yml: explicitly request sudo
2016-02-03 17:41 [U-Boot] [PATCH] .travis.yml: explicitly request sudo Stephen Warren
2016-02-03 17:44 ` Andreas Färber
2016-02-04 9:00 ` Heiko Schocher
@ 2016-02-08 20:49 ` Tom Rini
2 siblings, 0 replies; 6+ messages in thread
From: Tom Rini @ 2016-02-08 20:49 UTC (permalink / raw)
To: u-boot
On Wed, Feb 03, 2016 at 10:41:34AM -0700, Stephen Warren wrote:
> From: Stephen Warren <swarren@nvidia.com>
>
> 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 <wd@denx.de>
> Cc: Heiko Schocher <hs@denx.de>
> Cc: Tom Rini <trini@ti.com>
> Cc: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
> Cc: Andreas F?rber <afaerber@suse.de>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> Acked-by: Heiko Schocher <hs@denx.de>
Applied to u-boot/master, thanks!
--
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>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-02-08 20:49 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-03 17:41 [U-Boot] [PATCH] .travis.yml: explicitly request sudo Stephen Warren
2016-02-03 17:44 ` Andreas Färber
2016-02-04 9:00 ` Heiko Schocher
2016-02-04 16:33 ` Stephen Warren
2016-02-05 5:30 ` Heiko Schocher
2016-02-08 20:49 ` [U-Boot] " Tom Rini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox