[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Broken install-data-yes target
Re: Broken install-data-yes target
Thu, 6 Jun 2013 10:37:57 -0400
On 2013-06-05 14:37 -0700, Kip Warner wrote:
> My make install target for my autotool'd project's translations
> directory has a number of prerequisites. The one which seems to be
> causing trouble is the install-data-yes target. Yes, I am aware the path
> I am using contains spaces and yes I am aware that makefiles can choke
> on it if not properly dealt with (e.g. escaped). From the final
> generated Makefile...
> install-data-yes: all
> @catalogs='$(CATALOGS)'; \
> for cat in $$catalogs; do \
> cat=`basename $$cat`; \
> lang=`echo $$cat | sed -e 's/\.gmo$$//'`; \
> dir=$(localedir)/$$lang/LC_MESSAGES; \
> $(mkdir_p) $(DESTDIR)$$dir; \
This fragment appears incomplete. Moreover, the link provided below is
broken, so we cannot see the whole Makefile.am. It is unclear why the
above would choke on spaces in the working directory, since it does not
at a first glance appear to reference it at all. How are CATALOGS and
By using @ to suppress the command invocation you have rendered the
make output essentially useless, because we cannot see what shell
command is actually being run by make. I suspect the problem would be
obvious if you did not use @. I recommend avoiding @ for the most part,
especially for complex shell commands like the above!
Automake documents restrictions on filenames in the manual, but notably
does not mention (regular) spaces in build or install directories as
problematic. There are a couple spots where you can definitely run
into trouble, but for the most part I'd expect things to work (albeit
this configuration is probably not heavily used or tested).
However, you are missing some necessary quoting for the cases where
DESTDIR or localedir contain spaces. I suspect this is your main
> The second to last line seems to be causing problems.
> $ pwd
> ~/Projects/Avaneya/Extras/Viking Lander Remastered/Extractor/Translations
> $ make install --debug
> Must remake target `install-data-yes'.
> /bin/sh: 5: Lander: not found
> /bin/mkdir: missing operand
> Try '/bin/mkdir --help' for more information.
> /usr/bin/install: cannot create regular file
> ‘/viking-extractor.mo’: Permission denied
I can only guess that you have defined
localedir = Viking Lander Remastered
or similar. This will obviously fail in your rule since it lacks
the necessary quoting for the shell. Without the quoting, the shell
dir=Viking Lander Remastered/$lang/LC_MESSAGES
You can try running that manually in bash or similar. So that explains
the first error. The second error is explained by the fact that this
broken shell command now is not a normal variable assignment: it sets
dir only in the environment of the (non-existent) Lander command, so dir
is empty for the mkdir command, explaining the second error. The third
error does not appear to have come from your snippet.
Properly quoting your shell commands should fix it up.
Also, a little error handling goes a long way: your rule totally ignored
the (easily detectable!) errors and just happily proceeded as if nothing
was wrong. The make rule then proceeded to run installation commands
with garbage arguments, and appears to have subsequently attempted to
create files directly in /. Keep in mind that install rules are often
run as the superuser. It's no fun for anyone involved when a buggy
makefile trashes someone's root filesystem...
> The project's Makefile.am is here:
Hope that helps,
Nick Bowler, Elliptic Technologies (http://www.elliptictech.com/)