public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] gdb/data-directory: silence output from mkinstalldirs script
@ 2024-04-12 16:15 Andrew Burgess
  0 siblings, 0 replies; only message in thread
From: Andrew Burgess @ 2024-04-12 16:15 UTC (permalink / raw)
  To: gdb-cvs

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=032e5e0c0c08977e8109e8482cd944bac8572d92

commit 032e5e0c0c08977e8109e8482cd944bac8572d92
Author: Andrew Burgess <aburgess@redhat.com>
Date:   Mon Apr 8 12:34:39 2024 +0100

    gdb/data-directory: silence output from mkinstalldirs script
    
    After my recent changes the data-directory build now uses
    silent-rules.mk to reduce the output.
    
    One problem that remains was the use of mkinstalldirs by stamp-python
    and stamp-guile for creating some directories, the mkinstalldirs
    prints some messages, so we're left with output like this:
    
        GEN    stamp-python
      mkdir -p -- ./python/gdb
      mkdir -p -- ./python/gdb/command
      mkdir -p -- ./python/gdb/dap
      mkdir -p -- ./python/gdb/function
      mkdir -p -- ./python/gdb/printer
    
    I was looking at adding a --silent option to the mkinstalldirs script,
    however, when I took a look at the automake package (which is where
    mkinstalldirs comes from) it turns out that mkinstalldirs is
    deprecated, at the advice is to use 'install-sh -d' instead.
    
    Just like we carry mkinstalldirs in the top-level directory, we also
    carry install-sh, and a version of install-sh which supports the -d
    flag.
    
    And best of all, 'install-sh -d' doesn't appear to print any of the
    information messages to stdout that mkinstalldirs does, so if we
    switch to use that, we get a quieter build.
    
    There should be no changes in what is built after this commit
    
    Approved-By: Tom Tromey <tom@tromey.com>

Diff:
---
 gdb/data-directory/Makefile.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gdb/data-directory/Makefile.in b/gdb/data-directory/Makefile.in
index c0419df86b3..98a43529b07 100644
--- a/gdb/data-directory/Makefile.in
+++ b/gdb/data-directory/Makefile.in
@@ -44,7 +44,7 @@ LN_S = @LN_S@
 
 INSTALL = @INSTALL@
 INSTALL_DATA = @INSTALL_DATA@
-INSTALL_DIR = $(SHELL) $(srcdir)/../../mkinstalldirs
+INSTALL_DIR = $(SHELL) $(srcdir)/../../install-sh -d
 
 GDB_DATADIR = @GDB_DATADIR@

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-04-12 16:15 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-04-12 16:15 [binutils-gdb] gdb/data-directory: silence output from mkinstalldirs script Andrew Burgess

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).