On Fri, Feb 09, 2018 at 10:53:44PM +0100, Daniel Schwierzeck wrote: > > > On 09.02.2018 22:43, Tom Rini wrote: > > 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.schwierz...@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. > > yes, that makes sense > > > > > 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! > > > > I've scheduled builds for 7.3.0 [1] and 6.4.0 [2] > > [1] https://travis-ci.org/danielschwierzeck/u-boot/builds/339663802 > [2] https://travis-ci.org/danielschwierzeck/u-boot/builds/339664291
x86 broke, but I submitted a patch for the new warning gcc-6.x and later shows yesterday. -- Tom _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot