Message ID | yddhawkx8zj.fsf@manam.CeBiTec.Uni-Bielefeld.DE |
---|---|
State | New |
Headers | show |
On 4/16/12 7:32 AM, Rainer Orth wrote: > Btw., it occured to me that it might be useful to add an option to > locate out-of-tree manifests. I often have several source trees > (unmodified sources, ones with local patches) and would like to share > manifests between them. While this can be achieved with symlinks, a > --manifest_dir or similar option might be an alternative. Thoughts? That would be fantastic. This is not the first time someone requests this, but I've never gotten around to implementing it. The only thing there is that multiple manifests means that versioning needs to be handled externally to the script. But that's not a big deal. > 2012-04-15 Rainer Orth<ro@CeBiTec.Uni-Bielefeld.DE> > > * testsuite-management/validate_failures.py (GetBuildData): Use > target_alias. OK. Diego.
Diego Novillo <dnovillo@google.com> writes: > On 4/16/12 7:32 AM, Rainer Orth wrote: > >> Btw., it occured to me that it might be useful to add an option to >> locate out-of-tree manifests. I often have several source trees >> (unmodified sources, ones with local patches) and would like to share >> manifests between them. While this can be achieved with symlinks, a >> --manifest_dir or similar option might be an alternative. Thoughts? > > That would be fantastic. This is not the first time someone requests this, > but I've never gotten around to implementing it. The only thing there is > that multiple manifests means that versioning needs to be handled > externally to the script. But that's not a big deal. Indeed, but the advantage is that people can choose whatever naming scheme they like for the different manifests instead of implementing some (probably limited) scheme inside validate_failures.py. I'll give it a whirl, but probably only in early May, once I return from a trip to California. Rainer
# HG changeset patch # Parent 84699f134dedb823eb7ecde316543a2870d1b47a Use target_alias in validate_failures.py diff --git a/contrib/testsuite-management/validate_failures.py b/contrib/testsuite-management/validate_failures.py --- a/contrib/testsuite-management/validate_failures.py +++ b/contrib/testsuite-management/validate_failures.py @@ -5,7 +5,7 @@ # Contributed by Diego Novillo <dnovillo@google.com> # -# Copyright (C) 2011 Free Software Foundation, Inc. +# Copyright (C) 2011, 2012 Free Software Foundation, Inc. # # This file is part of GCC. # @@ -241,7 +241,7 @@ def CompareResults(manifest, actual): def GetBuildData(options): - target = GetMakefileValue('%s/Makefile' % options.build_dir, 'target=') + target = GetMakefileValue('%s/Makefile' % options.build_dir, 'target_alias=') srcdir = GetMakefileValue('%s/Makefile' % options.build_dir, 'srcdir =') if not ValidBuildDirectory(options.build_dir, target): Error('%s is not a valid GCC top level build directory.' %