automake-patches
[Top][All Lists]
Advanced

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

Re: FYI: Automake::Location (part of PR/360).


From: Akim Demaille
Subject: Re: FYI: Automake::Location (part of PR/360).
Date: 30 Sep 2002 10:26:51 +0200
User-agent: Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.4 (Honest Recruiter)

| I'm checking this in (on HEAD).

Magnifique!

| @@ -2068,16 +2076,16 @@
|                            # object is in this directory, so
|                            # $(DEPDIR) is the correct location for
|                            # dependencies.
| -                          'DEPBASE'   => '$(DEPDIR)/$*',
| -                          'BASE'      => '$*',
| -                          'SOURCE'    => '$<',
| -                          'OBJ'       => '$@',
| -                          'OBJOBJ'    => '$@',
| -                          'LTOBJ'     => '$@',
| -
| -                          'COMPILE'   => '$(' . $lang->compiler . ')',
| -                          'LTCOMPILE' => '$(LT' . $lang->compiler . ')',
| -                          '-o'        => $output_flag);
| +                          DEPBASE   => '$(DEPDIR)/$*',
| +                          BASE      => '$*',
| +                          SOURCE    => '$<',
| +                          OBJ       => '$@',
| +                          OBJOBJ    => '$@',
| +                          LTOBJ     => '$@',
| +
| +                          COMPILE   => '$(' . $lang->compiler . ')',
| +                          LTCOMPILE => '$(LT' . $lang->compiler . ')',
| +                          -o        => $output_flag);

Je ne savais pas que les '' étaient inutiles...  Par contre, je ne
comprends pas pq tu retires les () de la hash, ça me paraît être un
risque inutile de `porosité' entre arguments, non ?  Peut-être même
d'ailleurs faudrait-il prendre une réf de hash, aussi pour des
questions de perfs.

Dit en l'air, pas une demande.




reply via email to

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