Bug 1260327 - Remove the --recheck option of config.status. r=chmanchester
authorMike Hommey <mh+mozilla@glandium.org>
Wed, 13 Apr 2016 11:51:16 +0900
changeset 351807 5864ce92b53344598fea07f25cabef72aa791514
parent 351806 f521d5b4368efcc3e163d601410b27152220d60a
child 351808 450647e36329144c9a3de6bc6642cad717ee0c30
push id15527
push userbmo:rail@mozilla.com
push dateFri, 15 Apr 2016 01:44:41 +0000
Bug 1260327 - Remove the --recheck option of config.status. r=chmanchester With mozconfigs injecting options, it never really worked as intended.
--- a/python/mozbuild/mozbuild/config_status.py
+++ b/python/mozbuild/mozbuild/config_status.py
@@ -69,19 +69,16 @@ def config_status(topobjdir='.', topsrcd
     Contrary to config.status, it doesn't use CONFIG_FILES or CONFIG_HEADERS
     Without the -n option, this program acts as config.status and considers
     the current directory as the top object directory, even when config.status
     is in a different directory. It will, however, treat the directory
     containing config.status as the top object directory with the -n option.
-    The --recheck option, like with the original config.status, runs configure
-    again, with the options given in the "ac_configure_args" subst.
     The options to this function are passed when creating the
     ConfigEnvironment. These lists, as well as the actual wrapper script
     around this function, are meant to be generated by configure.
     See build/autoconf/config.status.m4.
     if 'CONFIG_FILES' in os.environ:
         raise Exception('Using the CONFIG_FILES environment variable is not '
@@ -93,18 +90,16 @@ def config_status(topobjdir='.', topsrcd
     if not os.path.isabs(topsrcdir):
         raise Exception('topsrcdir must be defined as an absolute directory: '
             '%s' % topsrcdir)
     default_backends = ['RecursiveMake']
     default_backends = (substs or {}).get('BUILD_BACKENDS', ['RecursiveMake'])
     parser = ArgumentParser()
-    parser.add_argument('--recheck', dest='recheck', action='store_true',
-                        help='update config.status by reconfiguring in the same conditions')
     parser.add_argument('-v', '--verbose', dest='verbose', action='store_true',
                         help='display verbose output')
     parser.add_argument('-n', dest='not_topobjdir', action='store_true',
                         help='do not consider current directory as top object directory')
     parser.add_argument('-d', '--diff', action='store_true',
                         help='print diffs of changed files.')
     parser.add_argument('-b', '--backend', nargs='+', choices=sorted(backends),
@@ -137,21 +132,16 @@ def config_status(topobjdir='.', topsrcd
         for b in selected_backends:
             b.dry_run = True
     reader = BuildReader(env)
     emitter = TreeMetadataEmitter(env)
     # This won't actually do anything because of the magic of generators.
     definitions = emitter.emit(reader.read_topsrcdir())
-    if options.recheck:
-        # Execute configure from the top object directory
-        os.chdir(topobjdir)
-        os.execlp('sh', 'sh', '-c', ' '.join([os.path.join(topsrcdir, 'configure'), env.substs['ac_configure_args'], '--no-create', '--no-recursion']))
     log_level = logging.DEBUG if options.verbose else logging.INFO
     print('Reticulating splines...', file=sys.stderr)
     if len(selected_backends) > 1:
         definitions = list(definitions)