bug-gnulib@gnu.org mirror (unofficial)
 help / color / mirror / Atom feed
From: Akim Demaille <akim@lrde.epita.fr>
To: Bruno Haible <bruno@clisp.org>
Cc: Ben Pfaff <blp@gnu.org>, Gnulib bugs <bug-gnulib@gnu.org>
Subject: Re: relocatable-prog: avoid warnings from Automake
Date: Thu, 24 Jan 2019 07:38:31 +0100	[thread overview]
Message-ID: <093A1C18-8453-4CA0-8EB4-545E38DD808A@lrde.epita.fr> (raw)
In-Reply-To: <2550370.Q3CLR1mU43@omega>



> Le 24 janv. 2019 à 02:08, Bruno Haible <bruno@clisp.org> a écrit :
> 
> Hi Akim,

Hi Bruno,

>> This is what I get currently:
>> 
>> cd bison && /bin/sh build-aux/missing automake-1.16 --gnu Makefile
>> lib/gnulib.mk:1677: warning: .PHONY was already defined in condition TRUE, which includes condition RELOCATABLE_VIA_WRAPPER ...
> 
> I've seen this warning also with respect to normal dependencies like
> 
> foo.o : foo.c
> 	$(CC) $(CPPFLAGS) $(CFLAGS) -c foo.c
> if USE_XYZ
> foo.o : xyz.h
> endif
> 
> It would be better if this case was fixed in Automake.

Well, you mean adding the feature :)

> But in the absence of Automake maintenance, your proposed patch is good.
> Maybe add a comment:
> # Automake warns about conditionally added dependencies to unconditionally defined targets.

I've added that.  Thanks!

      reply	other threads:[~2019-01-24  6:50 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-23 21:48 relocatable-prog: avoid warnings from Automake Akim Demaille
2019-01-23 22:03 ` Ben Pfaff
2019-01-24  1:08 ` Bruno Haible
2019-01-24  6:38   ` Akim Demaille [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://lists.gnu.org/mailman/listinfo/bug-gnulib

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=093A1C18-8453-4CA0-8EB4-545E38DD808A@lrde.epita.fr \
    --to=akim@lrde.epita.fr \
    --cc=blp@gnu.org \
    --cc=bruno@clisp.org \
    --cc=bug-gnulib@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).