diff options
| author | Nirbheek Chauhan <nirbheek@centricular.com> | 2020-02-07 15:38:57 +0530 |
|---|---|---|
| committer | Nirbheek Chauhan <nirbheek@centricular.com> | 2020-02-11 16:04:28 +0530 |
| commit | a6d5fc19dd3c093af256b716f928f9dc266b13f1 (patch) | |
| tree | 203a561690b8cf11d154cb994d34f2254af5c9a6 | |
| parent | 8978717b6b51652bd455efbb6867f3f5799726c9 (diff) | |
| download | meson-nirbheek/less-noise-install-only-changed.tar.gz | |
minstall: Make --only-changed less verbosenirbheek/less-noise-install-only-changed
When `--only-changed` is passed, we only want to know about files that
were newly-installed. Everything else is noise. The full list can
always be found in `install-log.txt` anyway. Sample output:
```
ninja: Entering directory `.'
ninja: no work to do.
Preserved 667 unchanged files, see meson-logs\install-log.txt for the full list
```
| -rw-r--r-- | mesonbuild/minstall.py | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/mesonbuild/minstall.py b/mesonbuild/minstall.py index b1a55e5a7..cc6ea0aea 100644 --- a/mesonbuild/minstall.py +++ b/mesonbuild/minstall.py @@ -214,6 +214,7 @@ class Installer: self.did_install_something = False self.options = options self.lf = lf + self.preserved_file_count = 0 def should_preserve_existing_file(self, from_file, to_file): if not self.options.only_changed: @@ -239,7 +240,7 @@ class Installer: 'a file'.format(to_file)) if self.should_preserve_existing_file(from_file, to_file): append_to_log(self.lf, '# Preserving old file %s\n' % to_file) - print('Preserving existing file %s' % to_file) + self.preserved_file_count += 1 return False os.remove(to_file) print('Installing %s to %s' % (from_file, outdir)) @@ -347,6 +348,9 @@ class Installer: self.run_install_script(d) if not self.did_install_something: print('Nothing to install.') + if self.preserved_file_count > 0: + print('Preserved {} unchanged files, see {} for the full list' + .format(self.preserved_file_count, os.path.normpath(self.lf.name))) except PermissionError: if shutil.which('pkexec') is not None and 'PKEXEC_UID' not in os.environ: print('Installation failed due to insufficient permissions.') |