[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug #60201] gettext-runtime/po/Makefile.in.in should not use @mkdir_p@

From: Arfrever
Subject: [bug #60201] gettext-runtime/po/Makefile.in.in should not use @mkdir_p@
Date: Tue, 9 Mar 2021 18:12:55 -0500 (EST)
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:72.0) Gecko/20100101 Firefox/72.0


                 Summary: gettext-runtime/po/Makefile.in.in should not use
                 Project: GNU gettext
            Submitted by: arfrever
            Submitted on: Tue 09 Mar 2021 11:12:53 PM UTC
                Category: None
                Severity: 3 - Normal
              Item Group: None
                  Status: None
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
         Discussion Lock: Any



gettext provides /usr/share/gettext/po/Makefile.in.in (described as "Makefile
for PO directory in any package using GNU gettext.") which has this fragment

# We use $(mkdir_p).
# In automake <= 1.9.x, $(mkdir_p) is defined either as "mkdir -p --" or as
# "$(mkinstalldirs)" or as "$(install_sh) -d". For these automake versions,
# @install_sh@ does not start with $(SHELL), so we add it.
# In automake >= 1.10, @mkdir_p@ is derived from ${MKDIR_P}, which is defined
# either as "/path/to/mkdir -p" or ".../install-sh -c -d". For these automake
# versions, $(mkinstalldirs) and $(install_sh) are unused.
mkinstalldirs = $(SHELL) @install_sh@ -d
install_sh = $(SHELL) @install_sh@
mkdir_p = @mkdir_p@

With newer versions of Automake, @mkdir_p@ is no longer normally expanded.
@MKDIR_P@ is expanded, but MKDIR_P is actually not used anywhere else in this

@mkdir_p@ would be expanded by deprecated AM_PROG_MKDIR_P macro:

My suggestion is to use $(MKDIR_P) instead of $(mkdir_p) in this file, and
drop above mkdir_p assignment.

Original report: https://bugs.gentoo.org/457986


Reply to this item at:


  Message sent via Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]