* [PATCH] buildman: Add gcc-11.1.0 to the directory list
@ 2021-10-05 16:20 Tom Rini
2021-10-15 1:05 ` Simon Glass
0 siblings, 1 reply; 6+ messages in thread
From: Tom Rini @ 2021-10-05 16:20 UTC (permalink / raw)
To: u-boot
While CI has been using gcc-11.1.0 for a long time, we have not updated
buildman to match. Correct this omission.
Signed-off-by: Tom Rini <trini@konsulko.com>
---
tools/buildman/toolchain.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/buildman/toolchain.py b/tools/buildman/toolchain.py
index fd137f7300e7..4e2471f3e370 100644
--- a/tools/buildman/toolchain.py
+++ b/tools/buildman/toolchain.py
@@ -498,7 +498,7 @@ class Toolchains:
if arch == 'aarch64':
arch = 'arm64'
base = 'https://www.kernel.org/pub/tools/crosstool/files/bin'
- versions = ['9.2.0', '7.3.0', '6.4.0', '4.9.4']
+ versions = ['11.1.0', '9.2.0', '7.3.0', '6.4.0', '4.9.4']
links = []
for version in versions:
url = '%s/%s/%s/' % (base, arch, version)
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] buildman: Add gcc-11.1.0 to the directory list
2021-10-05 16:20 [PATCH] buildman: Add gcc-11.1.0 to the directory list Tom Rini
@ 2021-10-15 1:05 ` Simon Glass
2021-10-15 1:12 ` Tom Rini
0 siblings, 1 reply; 6+ messages in thread
From: Simon Glass @ 2021-10-15 1:05 UTC (permalink / raw)
To: Tom Rini; +Cc: U-Boot Mailing List
Hi Tom,
On Tue, 5 Oct 2021 at 10:20, Tom Rini <trini@konsulko.com> wrote:
>
> While CI has been using gcc-11.1.0 for a long time, we have not updated
> buildman to match. Correct this omission.
>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
> tools/buildman/toolchain.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/buildman/toolchain.py b/tools/buildman/toolchain.py
> index fd137f7300e7..4e2471f3e370 100644
> --- a/tools/buildman/toolchain.py
> +++ b/tools/buildman/toolchain.py
> @@ -498,7 +498,7 @@ class Toolchains:
> if arch == 'aarch64':
> arch = 'arm64'
> base = 'https://www.kernel.org/pub/tools/crosstool/files/bin'
> - versions = ['9.2.0', '7.3.0', '6.4.0', '4.9.4']
> + versions = ['11.1.0', '9.2.0', '7.3.0', '6.4.0', '4.9.4']
> links = []
> for version in versions:
> url = '%s/%s/%s/' % (base, arch, version)
> --
> 2.17.1
>
This seems to break that n900 build:
https://source.denx.de/u-boot/custodians/u-boot-dm/-/jobs/336316
Regards,
Simon
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] buildman: Add gcc-11.1.0 to the directory list
2021-10-15 1:05 ` Simon Glass
@ 2021-10-15 1:12 ` Tom Rini
2021-10-15 1:25 ` Simon Glass
0 siblings, 1 reply; 6+ messages in thread
From: Tom Rini @ 2021-10-15 1:12 UTC (permalink / raw)
To: Simon Glass; +Cc: U-Boot Mailing List
[-- Attachment #1: Type: text/plain, Size: 1308 bytes --]
On Thu, Oct 14, 2021 at 07:05:33PM -0600, Simon Glass wrote:
> Hi Tom,
>
> On Tue, 5 Oct 2021 at 10:20, Tom Rini <trini@konsulko.com> wrote:
> >
> > While CI has been using gcc-11.1.0 for a long time, we have not updated
> > buildman to match. Correct this omission.
> >
> > Signed-off-by: Tom Rini <trini@konsulko.com>
> > ---
> > tools/buildman/toolchain.py | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/tools/buildman/toolchain.py b/tools/buildman/toolchain.py
> > index fd137f7300e7..4e2471f3e370 100644
> > --- a/tools/buildman/toolchain.py
> > +++ b/tools/buildman/toolchain.py
> > @@ -498,7 +498,7 @@ class Toolchains:
> > if arch == 'aarch64':
> > arch = 'arm64'
> > base = 'https://www.kernel.org/pub/tools/crosstool/files/bin'
> > - versions = ['9.2.0', '7.3.0', '6.4.0', '4.9.4']
> > + versions = ['11.1.0', '9.2.0', '7.3.0', '6.4.0', '4.9.4']
> > links = []
> > for version in versions:
> > url = '%s/%s/%s/' % (base, arch, version)
> > --
> > 2.17.1
> >
>
> This seems to break that n900 build:
>
> https://source.denx.de/u-boot/custodians/u-boot-dm/-/jobs/336316
*headdesk* the test hard-codes the toolchain version to add to PATH.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] buildman: Add gcc-11.1.0 to the directory list
2021-10-15 1:12 ` Tom Rini
@ 2021-10-15 1:25 ` Simon Glass
2021-10-15 1:28 ` Tom Rini
2021-10-21 18:49 ` Simon Glass
0 siblings, 2 replies; 6+ messages in thread
From: Simon Glass @ 2021-10-15 1:25 UTC (permalink / raw)
To: Tom Rini; +Cc: U-Boot Mailing List
Hi Tom,
On Thu, 14 Oct 2021 at 19:12, Tom Rini <trini@konsulko.com> wrote:
>
> On Thu, Oct 14, 2021 at 07:05:33PM -0600, Simon Glass wrote:
> > Hi Tom,
> >
> > On Tue, 5 Oct 2021 at 10:20, Tom Rini <trini@konsulko.com> wrote:
> > >
> > > While CI has been using gcc-11.1.0 for a long time, we have not updated
> > > buildman to match. Correct this omission.
> > >
> > > Signed-off-by: Tom Rini <trini@konsulko.com>
> > > ---
> > > tools/buildman/toolchain.py | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/tools/buildman/toolchain.py b/tools/buildman/toolchain.py
> > > index fd137f7300e7..4e2471f3e370 100644
> > > --- a/tools/buildman/toolchain.py
> > > +++ b/tools/buildman/toolchain.py
> > > @@ -498,7 +498,7 @@ class Toolchains:
> > > if arch == 'aarch64':
> > > arch = 'arm64'
> > > base = 'https://www.kernel.org/pub/tools/crosstool/files/bin'
> > > - versions = ['9.2.0', '7.3.0', '6.4.0', '4.9.4']
> > > + versions = ['11.1.0', '9.2.0', '7.3.0', '6.4.0', '4.9.4']
> > > links = []
> > > for version in versions:
> > > url = '%s/%s/%s/' % (base, arch, version)
> > > --
> > > 2.17.1
> > >
> >
> > This seems to break that n900 build:
> >
> > https://source.denx.de/u-boot/custodians/u-boot-dm/-/jobs/336316
>
> *headdesk* the test hard-codes the toolchain version to add to PATH.
It might be a good oppty to put some of its downloads into the docker image?
Regards,
Simon
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] buildman: Add gcc-11.1.0 to the directory list
2021-10-15 1:25 ` Simon Glass
@ 2021-10-15 1:28 ` Tom Rini
2021-10-21 18:49 ` Simon Glass
1 sibling, 0 replies; 6+ messages in thread
From: Tom Rini @ 2021-10-15 1:28 UTC (permalink / raw)
To: Simon Glass; +Cc: U-Boot Mailing List
[-- Attachment #1: Type: text/plain, Size: 1974 bytes --]
On Thu, Oct 14, 2021 at 07:25:24PM -0600, Simon Glass wrote:
> Hi Tom,
>
> On Thu, 14 Oct 2021 at 19:12, Tom Rini <trini@konsulko.com> wrote:
> >
> > On Thu, Oct 14, 2021 at 07:05:33PM -0600, Simon Glass wrote:
> > > Hi Tom,
> > >
> > > On Tue, 5 Oct 2021 at 10:20, Tom Rini <trini@konsulko.com> wrote:
> > > >
> > > > While CI has been using gcc-11.1.0 for a long time, we have not updated
> > > > buildman to match. Correct this omission.
> > > >
> > > > Signed-off-by: Tom Rini <trini@konsulko.com>
> > > > ---
> > > > tools/buildman/toolchain.py | 2 +-
> > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/tools/buildman/toolchain.py b/tools/buildman/toolchain.py
> > > > index fd137f7300e7..4e2471f3e370 100644
> > > > --- a/tools/buildman/toolchain.py
> > > > +++ b/tools/buildman/toolchain.py
> > > > @@ -498,7 +498,7 @@ class Toolchains:
> > > > if arch == 'aarch64':
> > > > arch = 'arm64'
> > > > base = 'https://www.kernel.org/pub/tools/crosstool/files/bin'
> > > > - versions = ['9.2.0', '7.3.0', '6.4.0', '4.9.4']
> > > > + versions = ['11.1.0', '9.2.0', '7.3.0', '6.4.0', '4.9.4']
> > > > links = []
> > > > for version in versions:
> > > > url = '%s/%s/%s/' % (base, arch, version)
> > > > --
> > > > 2.17.1
> > > >
> > >
> > > This seems to break that n900 build:
> > >
> > > https://source.denx.de/u-boot/custodians/u-boot-dm/-/jobs/336316
> >
> > *headdesk* the test hard-codes the toolchain version to add to PATH.
>
> It might be a good oppty to put some of its downloads into the docker image?
Well, it's the CI files that had been hard-coding the path (I was a bit
quick before), and should instead be using the one that's already in the
image, yes. So let me fire off something to fix that, and it just needs
to be in your tree before the other patch, to preserve bisectability.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] buildman: Add gcc-11.1.0 to the directory list
2021-10-15 1:25 ` Simon Glass
2021-10-15 1:28 ` Tom Rini
@ 2021-10-21 18:49 ` Simon Glass
1 sibling, 0 replies; 6+ messages in thread
From: Simon Glass @ 2021-10-21 18:49 UTC (permalink / raw)
To: Tom Rini; +Cc: U-Boot Mailing List, Simon Glass
On Thu, Oct 14, 2021 at 07:25:24PM -0600, Simon Glass wrote:
> Hi Tom,
>
> On Thu, 14 Oct 2021 at 19:12, Tom Rini <trini@konsulko.com> wrote:
> >
> > On Thu, Oct 14, 2021 at 07:05:33PM -0600, Simon Glass wrote:
> > > Hi Tom,
> > >
> > > On Tue, 5 Oct 2021 at 10:20, Tom Rini <trini@konsulko.com> wrote:
> > > >
> > > > While CI has been using gcc-11.1.0 for a long time, we have not updated
> > > > buildman to match. Correct this omission.
> > > >
> > > > Signed-off-by: Tom Rini <trini@konsulko.com>
> > > > ---
> > > > tools/buildman/toolchain.py | 2 +-
> > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
Applied to u-boot-dm, thanks!
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-10-21 18:51 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-05 16:20 [PATCH] buildman: Add gcc-11.1.0 to the directory list Tom Rini
2021-10-15 1:05 ` Simon Glass
2021-10-15 1:12 ` Tom Rini
2021-10-15 1:25 ` Simon Glass
2021-10-15 1:28 ` Tom Rini
2021-10-21 18:49 ` Simon Glass
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.