[U-Boot] [PATCH] buildman: support newer gcc versions from kernel.org

Tom Rini trini at konsulko.com
Fri Feb 9 21:43:25 UTC 2018


On Fri, Feb 09, 2018 at 10:37:47PM +0100, Daniel Schwierzeck wrote:
> Add support for gcc versions 7.3.0, 6.4.0 and 4.9.4.
> 
> Also use a regex for matching the tarball names. Some gcc versions
> use '-ARCH-' instead of '_ARCH-'.
> 
> Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck at gmail.com>
> 
> ---
> 
>  tools/buildman/toolchain.py | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tools/buildman/toolchain.py b/tools/buildman/toolchain.py
> index 2076323d5d..76dcb67614 100644
> --- a/tools/buildman/toolchain.py
> +++ b/tools/buildman/toolchain.py
> @@ -33,7 +33,7 @@ class MyHTMLParser(HTMLParser):
>          HTMLParser.__init__(self)
>          self.arch_link = None
>          self.links = []
> -        self._match = '_%s-' % arch
> +        self.re_arch = re.compile('[-_]%s-' % arch)
>  
>      def handle_starttag(self, tag, attrs):
>          if tag == 'a':
> @@ -41,7 +41,7 @@ class MyHTMLParser(HTMLParser):
>                  if tag == 'href':
>                      if value and value.endswith('.xz'):
>                          self.links.append(value)
> -                        if self._match in value:
> +                        if self.re_arch.search(value):
>                              self.arch_link = value
>  
>  
> @@ -431,7 +431,7 @@ class Toolchains:
>          """
>          arch = command.OutputOneLine('uname', '-m')
>          base = 'https://www.kernel.org/pub/tools/crosstool/files/bin'
> -        versions = ['4.9.0', '4.6.3', '4.6.2', '4.5.1', '4.2.4']
> +        versions = ['7.3.0', '6.4.0', '5.5.0', '4.9.4', '4.9.0', '4.6.3', '4.6.2', '4.5.1', '4.2.4']

Thanks for doing this.  I think we should trim out the toolchain
versions a bit.  We shouldn't need anything older than 4.9.0, or maybe
even 4.9.4 now.

I also worry about making 7.x or 6.x be the default fetched versions
now.  I know gcc-6.x adds more warnings as I use the Debian toolchains
in my test builds.  So travis is going to go right to erroring out.
Since your patch is going to start with 7.3, can you please push this to
travis and see what fails, so we have an idea on how hard it will be to
move CI to 7.3?  Thanks!

-- 
Tom


More information about the U-Boot mailing list