Skip to content

Commit 09b6654

Browse files
yulyuginmjcharne
authored andcommitted
Move verbosities
Move verbosities to support silent build mode. Verbosity level 4 was the first unused. All verborsity levels prior to 4 are increased by 1. (cherry picked from commit 2ac1510)
1 parent 8e537a3 commit 09b6654

File tree

5 files changed

+33
-35
lines changed

5 files changed

+33
-35
lines changed

mbuild/dag.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -854,7 +854,7 @@ def _scan_headers(self, xinput, header_paths, assumed_directory=None):
854854
# headers is all the files that fn includes directly. One
855855
# level scan
856856
headers = scanner.mbuild_scan(fn, header_paths)
857-
if verbose(3):
857+
if verbose(4):
858858
for hr in headers:
859859
if hr.system:
860860
sys="System "
@@ -880,18 +880,18 @@ def _scan_headers(self, xinput, header_paths, assumed_directory=None):
880880
# we are expecting to build.
881881
ah = self._find_rec_for_missing_file(hr.file_name, assumed_directory)
882882
if ah:
883-
if verbose(3):
883+
if verbose(4):
884884
msgb("FOUND DEP REC FOR MISSING HEADER. WE WILL BUILD IT")
885885
hr.file_name = ah.file_name
886886
scanned_header = False
887887
elif not self._check_required_file(hr.file_name):
888-
if verbose(3):
888+
if verbose(4):
889889
msgb("MISSING HEADER NOT REQUIRED")
890890
continue
891891
elif assumed_directory:
892892
ofn = hr.file_name
893893
hr.file_name = util.join(assumed_directory, ofn)
894-
if verbose(3):
894+
if verbose(4):
895895
msgb("ASSUMING",
896896
"%s is in %s" % (ofn, assumed_directory))
897897

mbuild/doxygen.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -85,12 +85,12 @@ def _find_doxygen(env):
8585
base.msgb('DOXYGEN',"Could not find cygwin's doxygen," +
8686
"trying doxygen from PATH")
8787
elif env['build_os'] == 'lin':
88-
if base.verbose(1):
88+
if base.verbose(2):
8989
base.msgb("CHECKING FOR", doxygen_cmd_intel)
9090
if os.path.exists(doxygen_cmd_intel):
9191
doxygen_cmd = doxygen_cmd_intel
9292
elif env['build_os'] == 'mac':
93-
if base.verbose(1):
93+
if base.verbose(2):
9494
base.msgb("CHECKING FOR", doxygen_cmd_mac)
9595
if os.path.exists(doxygen_cmd_mac):
9696
doxygen_cmd = doxygen_cmd_mac
@@ -99,7 +99,7 @@ def _find_doxygen(env):
9999

100100
doxygen_cmd = env.escape_string(doxygen_cmd)
101101
doxygen_okay = False
102-
if base.verbose(1):
102+
if base.verbose(2):
103103
base.msgb('Checking doxygen version','...')
104104
if base.check_python_version(2,4):
105105
try:
@@ -108,7 +108,7 @@ def _find_doxygen(env):
108108
if retval==0:
109109
if len(output) > 0:
110110
first_line = output[0].strip()
111-
if base.verbose(1):
111+
if base.verbose(2):
112112
base.msgb("Doxygen version", first_line)
113113
doxygen_okay = _doxygen_version_okay(first_line, 1,4,6)
114114
else:
@@ -223,7 +223,7 @@ def _build_doxygen_main(args, env):
223223

224224
cmd = env['DOXYGEN'] + ' ' + \
225225
env.escape_string(env['doxygen_config_customized'])
226-
if base.verbose(1):
226+
if base.verbose(2):
227227
base.msgb("RUN DOXYGEN", cmd)
228228
(retval, output, error_output) = util.run_command(cmd)
229229

@@ -292,7 +292,7 @@ def _make_doxygen_reference_manual(env, doxygen_inputs, subs, work_queue,
292292
phase = "DOXYGEN"
293293
if not okay:
294294
base.die("[%s] failed. dying..." % phase)
295-
if base.verbose(1):
295+
if base.verbose(2):
296296
base.msgb(phase, "build succeeded")
297297

298298

mbuild/env.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -345,7 +345,7 @@ def verbose_startup(self):
345345
if self._emitted_startup_msg:
346346
return
347347
self._emitted_startup_msg = True
348-
if verbose(1):
348+
if verbose(2):
349349
msgb("INVOKED", " ".join(sys.argv))
350350
msgb("START TIME", self.env['start_time_str'])
351351
msgb("CURRENT DIRECTORY", os.getcwd())
@@ -1024,7 +1024,7 @@ def process_user_settings(self):
10241024
'Setting jobs to 1 because we could not detect' +
10251025
' the number of CPUs')
10261026

1027-
if verbose(1):
1027+
if verbose(2):
10281028
# print host_cpu here because it may be overridden for
10291029
# cross compilations
10301030
msgb("HOST_CPU", self.env['host_cpu'])
@@ -1317,7 +1317,7 @@ def set_compiler_env(self, compiler_family=None):
13171317
die("Compiler family not recognized. Need gnu or ms")
13181318

13191319
if self.env['use_yasm']:
1320-
if verbose(1):
1320+
if verbose(2):
13211321
msgb("USE YASM")
13221322
build_env.yasm_support(self)
13231323

mbuild/util.py

Lines changed: 12 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -62,48 +62,48 @@ def find_python(env):
6262
return pycmd
6363
def copy_file(src,tgt):
6464
"""Copy src to tgt."""
65-
if verbose(1):
65+
if verbose(2):
6666
msgb("COPY", tgt + " <- " + src)
6767
shutil.copy(src,tgt)
6868
def move_file(src,tgt):
6969
"""Move/Rename src to tgt."""
70-
if verbose(1):
70+
if verbose(2):
7171
msgb("MOVE", src + " -> " + tgt)
7272
shutil.move(src,tgt)
7373
def symlink(env,src,tgt):
7474
"""Make a symlink from src to target. Not available on windows."""
7575
if env.on_windows():
7676
die("symlink() not available on windows")
77-
if verbose(1):
77+
if verbose(2):
7878
msgb("SYMLINK", src + " -> " + tgt)
7979
os.symlink(src,tgt)
8080

8181
def copy_tree(src,tgt, ignore_patterns=None, symlinks=False):
8282
"""Copy the tree at src to tgt. This will first remove tgt if it
8383
already exists."""
84-
if verbose(1):
84+
if verbose(2):
8585
msgb("COPYTREE", tgt + " <- " + src)
8686
if not os.path.exists(src):
8787
error_msg("SRC TREE DOES NOT EXIST", src)
8888
raise Exception
8989
if os.path.exists(tgt):
90-
if verbose(1):
90+
if verbose(2):
9191
msgb("Removing existing target tree", tgt)
9292
shutil.rmtree(tgt, ignore_errors=True)
93-
if verbose(1):
93+
if verbose(2):
9494
msgb("Copying to tree", tgt)
9595
if ignore_patterns:
9696
sp = shutil.ignore_patterns(ignore_patterns)
9797
else:
9898
sp = None
9999
shutil.copytree(src,tgt,ignore=sp, symlinks=symlinks)
100-
if verbose(1):
100+
if verbose(2):
101101
msgb("Done copying tree", tgt)
102102

103103
def cmkdir(path_to_dir):
104104
"""Make a directory if it does not exist"""
105105
if not os.path.exists(path_to_dir):
106-
if verbose(1):
106+
if verbose(2):
107107
msgb("MKDIR", path_to_dir)
108108
os.makedirs(path_to_dir)
109109
def list2string(ls):
@@ -125,8 +125,7 @@ def remove_file(fn, env=None, quiet=True):
125125
make_writable(fn)
126126
if os.path.exists(fn) or os.path.lexists(fn):
127127
if not quiet:
128-
if verbose(1):
129-
vmsgb(1, "REMOVING", fn)
128+
vmsgb(2, "REMOVING", fn)
130129
os.unlink(fn)
131130
return (0, [])
132131
def remove_tree(dir_name, env=None, dangerous=False):
@@ -148,13 +147,13 @@ def _important_file(dir_name):
148147
return True
149148
return False
150149

151-
vmsgb(1, "CHECKING", dir_name)
150+
vmsgb(2, "CHECKING", dir_name)
152151
if os.path.exists(dir_name):
153152
if not dangerous and _important_file(dir_name):
154153
s = 'Did not remove directory {} because of a .svn/.git subdirectory'.format(dir_name)
155154
warn(s)
156155
return (1, [ s ])
157-
vmsgb(1, "REMOVING", dir_name)
156+
vmsgb(2, "REMOVING", dir_name)
158157
make_writable(dir_name)
159158
shutil.rmtree(dir_name, ignore_errors = True)
160159
return (0, [])
@@ -282,8 +281,7 @@ def qdip(fn):
282281

283282
def touch(fn):
284283
"""Open a file for append. Write nothing to it"""
285-
if verbose():
286-
vmsgb(1, "TOUCH", fn)
284+
vmsgb(1, "TOUCH", fn)
287285
f=open(fn,"a")
288286
f.close()
289287

mbuild/work_queue.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -339,7 +339,7 @@ def _pretty_cmd_str(self):
339339
def dump(self, tab_output=False, show_output=True):
340340
s = []
341341
nl = '\n'
342-
if verbose(1):
342+
if verbose(2):
343343
pass
344344
elif self.failed():
345345
pass
@@ -640,15 +640,15 @@ def _stop_daemons(self):
640640
for i in range(self.max_parallelism):
641641
t = command_t()
642642
t.terminator = True
643-
if verbose(3):
643+
if verbose(4):
644644
msgb("SENT TERMINATOR", str(i))
645645
self._start_a_job(t)
646646

647647
def _join_threads(self):
648648
"""Use this when not running threads in daemon-mode"""
649649
for t in self.threads:
650650
t.join()
651-
if verbose(3):
651+
if verbose(4):
652652
msgb("WORKER THREAD TERMINATED")
653653
self.threads = []
654654

@@ -779,7 +779,7 @@ def _start_more_jobs(self):
779779
cmd = ready.popleft()
780780
# FIXME: small concern that this could be slow
781781
self.pending_commands.remove(cmd)
782-
if verbose(2):
782+
if verbose(3):
783783
msgb("LAUNCHING", cmd.dump_cmd())
784784
self._start_a_job(cmd)
785785
self.pending -= 1
@@ -933,7 +933,7 @@ def _build_blind_threads(self,
933933
if started:
934934
c = self._wait_for_jobs()
935935
if c:
936-
if verbose(3):
936+
if verbose(4):
937937
msgb("JOB COMPLETED")
938938
if c.failed():
939939
self.errors += 1
@@ -952,7 +952,7 @@ def _build_blind_threads(self,
952952
c._complete()
953953
# Command objects can depend on each other
954954
# directly. Enable execution of dependent commands.
955-
if verbose(3):
955+
if verbose(4):
956956
msgb("ADD CMD-AFTERS")
957957
self.add(c._check_afters())
958958
# Or we might find new commands from the file DAG.
@@ -988,7 +988,7 @@ def _build_blind_no_threads(self, die_on_errors=True,
988988
_worker_one_task(self.out_queue, self.back_queue)
989989
c = self._wait_for_jobs()
990990
if c:
991-
if verbose(3):
991+
if verbose(4):
992992
msgb("JOB COMPLETED")
993993
if c.failed():
994994
okay = False
@@ -1002,7 +1002,7 @@ def _build_blind_no_threads(self, die_on_errors=True,
10021002
c._complete()
10031003
# Command objects can depende on each other
10041004
# directly. Enable execution of dependent commands.
1005-
if verbose(3):
1005+
if verbose(4):
10061006
msgb("ADD CMD-AFTERS")
10071007
self.add(c._check_afters())
10081008
# Or we might find new commands from the file DAG.

0 commit comments

Comments
 (0)