Tom Tromey <[EMAIL PROTECTED]> writes:
> At some point I'd like to fix the "adding a directory" problem. Right
> now if you add a new directory (edit AC_OUTPUT and SUBDIRS), automake
> won't notice correctly and a simple "make" won't rebuild the way you'd
> like.
>
> Likewise automake isn't ver
"Lars J. Aas" <[EMAIL PROTECTED]> writes:
> First a simple organize/cleanup patch.
Err, I must confess I'd like better &canonicalize or so...
Pavel Roskin <[EMAIL PROTECTED]> writes:
> Hello, Tom!
>
> I see that you are not using the latest and greatest Autoconf.
> Names beginning with AM_ are reserved for macros. Autoconf-2.49c will not
> allow such names in configure.
>
> That's why many tests are failing again - AM_INCLUDE is not
Tom Tromey <[EMAIL PROTECTED]> writes:
> Akim> I can't proceed without a full approval of this patch, it is
> Akim> required for the next ones.
>
> I thought I gave this, with a couple minor changes requested.
Correct, but the thing is these changes were not doable at that time.
I'll put it in
"Derek R. Price" <[EMAIL PROTECTED]> writes:
> Somebody checked in a bad quote recently. It breaks at least the
> stamph/header targets. Patch attached.
I already have this in the queue :) I just noticed it was wrong when
reading the code. The quote was needed when there was no &transform.
Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
>
> Akim> so naturally the question is: do we really want to keep this
> Akim> zillions of pseudo targets? The current automake, indeed,
n't determine the
relationship between the two. Aren't they the same?
----
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* subdirs.am: This file is the exception: clean recursive targets
are called
"Derek R. Price" <[EMAIL PROTECTED]> writes:
> Somebody checked in a bad quote recently. It breaks at least the
> stamph/header targets. Patch attached.
Well, I seem to have lost patches. Can't find the one named
`backname' either :( I'm applying yours, thanks!
Pavel Roskin <[EMAIL PROTECTED]> writes:
> > Tom, you should also know you can have a special exception, and tell
> > autoconf AM_INCLUDE is OK. But I agree with Pavel it looks way too
> > much like a macro name, in itself it is confusing.
>
> As far as I know, it cannot be done without breakin
Rob van der Leek <[EMAIL PROTECTED]> writes:
> Hi list,
>
> I want to define a path to a configuration file in `config.h`, therefore
> I call AC_DEFINE_UNQUOTED(CONFIG_FILE, $prefix/etc/config.xml, [Path to
> config file])
> The problem is that $prefix defaults to NONE when no --prefix flag is
>
Raja R Harinath <[EMAIL PROTECTED]> writes:
> Akim Demaille <[EMAIL PROTECTED]> writes:
> [snip]
> > For instance, I find it quite annoying to have to know that Automake
> > knows the -local targets: the code of 25 makes it possible for users
> > to have
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in: Various formatting changes, and modernization of
Perl constructs.
(&backname): New.
(&handle_configure, define_standard_variables): Use it.
Index: automake.in
-
44)
+++ 0.40(w)/ChangeLog Wed, 07 Feb 2001 00:30:10 +0100 akim (am/f/47_ChangeLog 1.35 644)
@@ -1,3 +1,13 @@
+2001-02-07 Akim Demaille <[EMAIL PROTECTED]>
+
+ * automake.in (&transform): Use `my', not `local'.
+ (&transform_cond): New.
+ (&am
Alexandre Oliva <[EMAIL PROTECTED]> writes:
> On Feb 6, 2001, Akim Demaille <[EMAIL PROTECTED]> wrote:
>
> > Don't go that way! AC_PREREQ.
>
> It won't let you define fallbacks for older releases, which is exactly
> the point. ifdef, as you propose
Alexandre Oliva <[EMAIL PROTECTED]> writes:
> On Feb 7, 2001, Akim Demaille <[EMAIL PROTECTED]> wrote:
>
> > I am referring to other applications depending upon Autoconf such as
> > Automake, for which it is perfectly reasonable to require a version of
> >
Tom Tromey <[EMAIL PROTECTED]> writes:
> Akim> - print "Copyright 2000 Free Software Foundation, Inc.\n";
> Akim> + print "Copyright 2000, 2001 Free Software Foundation, Inc.\n";
>
> Hmm... shouldn't we print all the copyright years here?
> And in aclocal?
No idea, you are t
Tom Tromey <[EMAIL PROTECTED]> writes:
> Akim> This patch includes the Makefile.ins' diffs, i.e., no changes.
>
> Umm, there are plenty of Makefile.in changes here.
Right, and a lot of thanks for reviewing them into the details!
> Some are whitespace changes, which I'd prefer not to change:
>
'm applying. The changes Tom noted were due to
my writing $pair instead of $pairs. I hope some day we really use
strict... And prototypes, which will require a lot of massage, since
it means functions should be defined before being used.
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTE
Sorry, I lost track from Tom's message about this.
No changes at all.
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&file_contents): Remove a dead branch.
Index: automake.in
--- automake.in Fri, 09 Feb 2001 04:37:00 +
Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
>
> >> Hmm... shouldn't we print all the copyright years here?
> >> And in aclocal?
>
> Akim> No idea, you are th
Seems simpler like this.
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&file_contents): Transform the global options (such
as CYGNUS).
(&handle_texinfo, &handle_dist_worker, &handle_tests_dejagnu): Don't.
Index: auto
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* distdir.am (distdir): Handle the DIST-TARGETS.
* automake.in (&handle_dist_worker): Don't.
Index: automake.in
--- automake.in Sat, 17 Feb 2001 18:43:49 +0100 akim (am/f/39_auto
by the fact the file name was a lie (it's actually
distcheck.am), and (ii), although I agree the long list of ?TOPDIR? is
gross, I do plan to implement some multilline if/else/fi support for
Automake files, which will make this much simpler, and more pleasant.
Index: ChangeLog
from Akim Dem
But I also introduced a bug, due to my misunderstanding the
functioning of the @dist_targets handling. Patch 35 fixes it. The
bug can be observed in ./Makefile.in: *after* tarball was made, the
dist-info hook is run.
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* distdir.
Last (latest?) clean ups for the dist chain.
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&dist_cmp): New.
(&handle_dist): Use it.
Promote `my' over `local'.
Index: automake.in
--- automake.in Sat, 17 Feb 2001 2
It is on purpose my uses of uniq have no &: it disables prototype
checking.
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* distdir.am (GZIP_ENV): Define.
* automake.in (&handle_dist): Don't.
(&uniq): New.
(&handle_dist_worke
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in ($top_builddir): Remove, used in one place.
(&define_standard_variables): Adjust.
Move the definition of triplet variables into...
* header-vars.am: here.
Index: a
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&handle_tests_dejagnu): Move the definition of a
default `site.exp' into...
* dejagnu.am (site.exp): here.
Use DOS compliant file names.
Index: automake.in
some day).
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* dist-vars.am: Remove, merge into...
* distdir.am: this.
* automake.in (dirname, basename, backname): Move at the top of
the file so that prototypes are checked,
U
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&initialize_global_constants): Don't define $USAGE.
(&usage): Handle the former content of $USAGE.
Don't pretend autoconf cares about Makefile.in.
Classify the options
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&conditional_true_when): Modernize, simplify.
Warning: `$comp' is now private (my), while it used to be
`inherited' from a `local' elsewhere in the code. AFAICT it was
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&check_ambiguous_conditional, &read_main_am_file):
Use a uniform naming scheme.
("e_cond_val, &unquote_cond_val): Delete, dead code.
(&read_am_file): Fix conditio
ndling of conditionalized/unconditinal variables.
This patch leaves scories of previously needed subs, which another
patch will clean up. As Jacques Brel, the great francophone Belgian
singer, would say, ``Too much c'est too much''.
Index: ChangeLog
from Akim Demaille <[EM
"Lars J. Aas" <[EMAIL PROTECTED]> writes:
> On Fri, Feb 09, 2001 at 11:53:08AM +0100, Akim Demaille wrote:
> : /tmp % ace --version nostromo 11:52
> : autoconf (GNU Autoconf) 2.49d
> : Written by David J. MacKenzie.
&g
Tom Tromey <[EMAIL PROTECTED]> writes:
> > "Derek" == Derek R Price <[EMAIL PROTECTED]> writes:
>
> Derek> Is there a good reason that Automake renames the three
> Derek> variables set by AC_CANONICAL_* ('build', 'host', & 'target')
> Derek> to 'build_triplet', 'host_triplet', & 'target_trip
"Derek R. Price" <[EMAIL PROTECTED]> writes:
> I've removed the references to $seen_canonical, re one of the FIXME
> comments on the way to enabling traces. I know this patch is kinda
> largish, but even if it doesn't get checked in yet, I'd like some
> feedback. This is somewhat integral to my
"Tim Van Holder" <[EMAIL PROTECTED]> writes:
> - /* | [A-Za-z]:*) ;;
> - */*) prog="`pwd`/$prog" ;;
> + [/\\]* | [A-Za-z]:[/\\]*) ;;
> + *[\\/]*) prog="`pwd`/$prog" ;;
> @@ -11,7 +11,7 @@
> # Always use an absolute srcdir. Otherwise symlinks made in subdirs
> # of the test dir just won't work
"Tim Van Holder" <[EMAIL PROTECTED]> writes:
> > Tim> 2001-02-10 Tim Van Holder <[EMAIL PROTECTED]>
> > Tim>* m4/sanity.m4: Quote AC_MSG_RESULT's argument.
> >
> > Why?
> As I understand it, the current convention is that ALL arguments
> to autoconf macros should be quoted (even trivia
Alexandre Oliva <[EMAIL PROTECTED]> writes:
> On Feb 13, 2001, Tom Tromey <[EMAIL PROTECTED]> wrote:
>
> > I don't want to remove anything from autoconf's blacklist. Doing that
> > means we have to update autoconf whenever we change automake.
>
> Nope. CVS Autoconf has a macro to remove strin
Tom Tromey <[EMAIL PROTECTED]> writes:
> > "Pavel" == Pavel Roskin <[EMAIL PROTECTED]> writes:
>
> Pavel> AC_CHECK_TOOL([STRIP],[strip])
>
> Yes, I wasn't thinking. I've removed this.
>
> Ideally we'd have some way to tell autoconf, "if this package can be
> cross-compiled, then check for
Pavel Roskin <[EMAIL PROTECTED]> writes:
> Hello, Tom!
>
> > Pavel> AC_CHECK_TOOL([STRIP],[strip])
> >
> > Yes, I wasn't thinking. I've removed this.
> >
> > Ideally we'd have some way to tell autoconf, "if this package can be
> > cross-compiled, then check for a cross `strip' as well". Maybe
Pavel Roskin <[EMAIL PROTECTED]> writes:
> Hello, Derek!
>
> On Fri, 9 Feb 2001, Derek R. Price wrote:
>
> > Why do the AC_CANONICAL_* functions no longer set *_alias? There's a
>
> "cvs annotate" and "cvs log" are your friends. Akim did it. But the log
> message is silent about "FIXME". Akim
"Tim Van Holder" <[EMAIL PROTECTED]> writes:
> > suggest that your patch makes them uniform. And of course, the same
> > as promoted by autoconf.texi :)
> How do you mean, Akim? autoconf.texi doesn't say anything
> about this... Or do you want me to add such an entry?
Arg, sorry, I really thoug
Stephen Torri <[EMAIL PROTECTED]> writes:
> Downloaded the latest automake from CVS. Running "make" went fine. Ran
> "make check" and got five errors:
>
> ansi3.test
> install2.test
> pr87.test
> subobj3.test
> target-cfalgs.test
>
> New to automake so I cannot point out the exact problem. Will
Lars Hecking <[EMAIL PROTECTED]> writes:
> target-cflags (note spelling) fails for me on OpenBSD as well. I
> have already sent verbose output to Tom.
I've found the related thread, but not your answer.
Lars Hecking <[EMAIL PROTECTED]> writes:
> === Running test ./target-cflags.test
> automake: Makefile.am: installing `./compile'
> creating cache ./config.cache
[...]
> creating ./config.status
> creating Makefile
> gcc -DPACKAGE=\"target-cflags\" -DVERSION=\"0.0\" -I. -I..-DFOO -g -O2 -c
Stephen Torri <[EMAIL PROTECTED]> writes:
> === Running test ./ansi3.test
> Makefile.am:1: installing `./ansi2knr.c'
> Makefile.am:1: installing `./ansi2knr.1'
> checking for a BSD compatible install... /usr/bin/install -c
> checking whether build environment is sane... yes
> checking for mawk...
Stephen Torri <[EMAIL PROTECTED]> writes:
> gcc -g -O2 @LDFLAGS@ -o foo foo.o
> gcc: @LDFLAGS@: No such file or directory
You are most probably using a version of Autoconf before this patch:
2001-01-26 Akim Demaille <[EMAIL PROTECTED]>
* aclang.m4: dnl AC
Lars Hecking <[EMAIL PROTECTED]> writes:
> > I'm surprised Make doesn't speak here. Reading the test files, GNU
> > Make is required, and GNU Make is usually verbose. In my case I have:
>
> I'm using BSD make here. That's the whole point of the exercise.
> Requiring GNU make doesn't make muc
Stephen Torri <[EMAIL PROTECTED]> writes:
> On 20 Feb 2001, Akim Demaille wrote:
>
> > Huh??? What version of autoconf are you using? Could you try this?
>
> autoconf 2.94d
You missed the second part of my request methinks, with the grep.
Update your cvs copy of Aut
How frustrating not to see what failed...
Could you try this:
/tmp % touch foo nostromo 16:38
/tmp % chmod 000 foo nostromo 16:38
/tmp % cp -pR foo barnostromo
Stephen Torri <[EMAIL PROTECTED]> writes:
> > cp: cannot open `foo' for reading: Permission non accordée
>
> Worked fine. I was able to create "bar". (logged in as root)
This test is precisely trying to check that automake does not perform
invalid operations, but the user is not expected to be
Stephen Torri <[EMAIL PROTECTED]> writes:
> No problem. I will move the code into a user directory and work as my
> normal user. In regards to install the "questionable" tarball, how do you
> handle it?
There is nothing but trust here. If someone wants to hurt you, given
that `install' is very
Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
>
> Akim> This patch includes all the diffs of Makefile.ins, i.e., subdirs'
> Akim> Makefile have not changed. The big change in the to
Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
>
> Akim> It is on purpose my uses of uniq have no &: it disables
> Akim> prototype checking.
>
> Do we really want protot
Tom Tromey <[EMAIL PROTECTED]> writes:
> BTW it is hard to read big rearrangement patches.
Agreed.
> It is made worse because sometimes they include actual code changes
> as well. For instance this patch wasn't just a rearrangement, it
> included semantic differences in file_contents.
That'
Tom Tromey <[EMAIL PROTECTED]> writes:
> Beware: I believe there is one place in automake where local() is
> required. I know that sucks. I think it is in the variable expansion
> code, but I'm afraid I don't remember for certain.
Yes, I seem to recall I found one. I forgot my patches today :
Tom Tromey <[EMAIL PROTECTED]> writes:
> Akim> Ideally I'd say we need objects for variables, and have a
> Akim> uniformized handling of conditionalized/unconditinal variables.
>
> This patch needed a lot less defense than you posted -- the only
> reason conditionals were implemented in that ugl
Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
>
> Akim> * dist-vars.am: Remove, merge into...
> Akim> * distdir.am: this.
> Akim> * automake.in (dirname, bas
Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
>
> Akim> As a specific advice, please, never test again any of the
> Akim> Autotools as root, as they are not meant for it, and this
&g
I don't understand the following line in automake.in. It seems very
wrong to me, but...
# Read Makefile.am and set up %contents. Simultaneously copy lines
# from Makefile.am into $output_trailer or $output_vars as
# appropriate. NOTE we put rules in the trailer section. We want
# user rules
I don't understand this:
~/src/am % make -s check TESTS=distdir.test nostromo Err 2
Making check in .
Making check in m4
Making check in tests
PASS: distdir.test
==
All 1 tests passed
==
~/src/am % grep -C10 'mkinstall.*foo' tests/testSubDir/M
"Tim Van Holder" <[EMAIL PROTECTED]> writes:
> OK - here goes. I've listed '[\\/]* | ?:[\\/]*' as the canonical
> absolute path test; I'll submit patches to use this throughout autoconf
> later.
Plenty of thanks!
> +@itemize @bullet
@bullets are atrocious! We use @minus.
> +@item No multipl
What is the general policy wrt `optimizations' in automake vs leaving
some job to make? For instance there are many places with code like:
if ($relative_dir eq '.')
{
push (@files, 'acconfig.h');
}
else
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in: Promote local `my' over `local'.
Index: automake.in
--- automake.in Wed, 21 Feb 2001 00:28:50 +0100 akim (am/f/39_automake.i 1.58 755)
+++ automake.in Wed, 21 Feb 2001 18:51:21
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in: Formatting and mying changes.
Index: automake.in
--- automake.in Tue, 20 Feb 2001 19:41:10 +0100 akim (am/f/39_automake.i 1.56 755)
+++ automake.in Tue, 20 Feb 2001 20:26:36 +0100 akim
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in: Formatting and mying changes.
Index: automake.in
--- automake.in Tue, 20 Feb 2001 20:28:16 +0100 akim (am/f/39_automake.i 1.57 755)
+++ automake.in Tue, 20 Feb 2001 20:44:05 +0100 akim
In two chunks 'coz I first went too far :)
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in: Formatting changes.
Index: automake.in
--- automake.in Mon, 19 Feb 2001 02:58:32 +0100 akim (am/f/39_automake.i 1.53 755)
+++ automake.in Tue, 20 Feb 2001 18:
t the first time I meet this desire, and actually most
explicit &depend in automake could be removed. Maybe introducing a
new syntax for conditional dependencies?
.PHONY:? install-man uninstall-man
or whatever the syntax, something saying `add me if not already
included'?
Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
>
> Akim> -install-man: $(MANS)
> Akim> - @$(NORMAL_INSTALL)
> Akim> - $(MAKE) $(AM_MAKEFLAGS) install-man1
>
> I&#
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in: Promote local `my' over `local'.
Index: automake.in
--- automake.in Wed, 21 Feb 2001 00:28:50 +0100 akim (am/f/39_automake.i 1.58 755)
+++ automake.in Wed, 21 Feb 2001 18:51:21
c-am
-
-install-data-am:
install-data: install-data-am
-
install-am: all-am
@$(MAKE) $(AM_MAKEFLAGS) install-exec-am install-data-am
install: install-am
-uninstall-am:
uninstall: uninstall-am
all-am: Makefile
install-strip:
Index: ChangeLog
from Akim Demaill
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&am_install_var): Don't hook on the clean targets
and .PHONY.
* data-clean.am, java-clean.am, libs-clean.am, lisp-clean.am,
* ltlib-clean.am, progs-clean.am, python-clean.am,
Tom Tromey <[EMAIL PROTECTED]> writes:
> Akim> -local ($sect, %sections, %vlist);
> Akim> +local (%sections, %vlist);
>
> If you need to keep local here for now that is fine. But if so please
> add a big comment explaining why.
>
> Ignore this if this spot is fixed in a later patch. I
"Tim Van Holder" <[EMAIL PROTECTED]> writes:
> > @bullets are atrocious! We use @minus.
> Fine by me. Note that the only two other @itemizes in the manual
> both use @bullet.
Arg, to be fixed ;)
> > I removed the @*, AFAIK it's useless here, and anyway it is
> > inconsistent with the rest of t
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in: Mying changes.
Index: automake.in
--- automake.in Thu, 22 Feb 2001 20:44:50 +0100 akim (am/f/39_automake.i 1.62 755)
+++ automake.in Thu, 22 Feb 2001 20:53:57 +0100 akim (am/f/39_automake.i 1.62 755)
@@ -
Index: mans.am
--- mans.am Wed, 21 Feb 2001 21:04:28 +0100 akim (am/g/24_mans.am 1.4 644)
+++ mans.am Thu, 22 Feb 2001 20:28:43 +0100 akim (am/g/24_mans.am 1.4 644)
@@ -17,7 +17,7 @@
## 02111-1307, USA.
.PHONY install-man: install-man@SECTION@
-install-man@SECTION@:
+install-man@SECTION@: $(ma
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in: Mying changes.
(&handle_tags): Fix a bug uncovered by the previous mying changes:
transform CONFIG into $config, not $xform.
Index: automake.in
--- automake.in Wed, 21 Feb
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in: Mying changes.
Index: automake.in
--- automake.in Thu, 22 Feb 2001 20:55:47 +0100 akim (am/f/39_automake.i 1.63 755)
+++ automake.in Thu, 22 Feb 2001 21:06:06 +0100 akim (am/f/39_automake.i 1.63 755)
@@ -
Six, six for them! (I'm not counting those for file handles, which
perl refuses as my, not sure to understand why).
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in: Mying changes.
* Makefile.am (maintainer-check): Check the stabi
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&initialize_per_input): %targets_conditionals is
new.
(&read_am_file): When slurping a rule, set the target related
variable, and no variable related variable.
(
o exist as a
target?
Because the reader inserts the targets in %contents, which holds the
variables...
The next patch addresses this issue.
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&variable_defined, &variable_value_as_list_worker):
If $V
Tom Tromey <[EMAIL PROTECTED]> writes:
> Akim> * automake.in: Promote local `my' over `local'.
>
> Ok.
>
> Thanks for doing this. I'm sure it is a pain.
Well, I've seen more exciting tasks :)
But since I quickly pollute my actual patches with such changes, I
should first get rid of what temp
Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
>
> Akim> * automake.in: Formatting and mying changes.
>
> Ok.
>
> Akim> + # FIXME: This code is mad,
Tom Tromey <[EMAIL PROTECTED]> writes:
> Targets which are presented to the user must be defined regardless of
> whether they have any effect. So for instance `install-exec' must
> always exist.
I understand this. Then I'll have a list of required targets and
output them in any case.
> Akim>
Pavel Roskin <[EMAIL PROTECTED]> writes:
> Hello, Akim!
>
> > Is this only GNU Make, or a general property?
> >
> > /tmp % cat Makefile 21:37 remo
> > all: foo bar
> > .PHONY: all foo
> > /tmp % make
Pavel Roskin <[EMAIL PROTECTED]> writes:
> Hello, Akim!
>
> You recently applied a patch to tests/target-cflags.test without
> mentioning it in ChangeLog. Maybe you didn't mean to commit that change?
>
> If you did, then two small fixes are needed:
>
> 1) You should run ./foo instead /foo
> 2)
Tom Tromey <[EMAIL PROTECTED]> writes:
> Akim --
>
> One problem with target/variable separation is that &am_line_error
> accepts a variable name as its first argument. If they are separate
> then some calls to this may become ambiguous. This can probably be
> fixed, but it is something to be
a definition closer to its uses, patch chunks are not as self
contained as one could think (i.e., a reject patch of a my change is a
bad thing).
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&handle_texinfo, &handle_man_page
Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
>
> Akim> * automake.in: Mying changes.
> Akim> (&handle_tags): Fix a bug uncovered by the previous mying changes:
> A
Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
>
> Akim> * automake.in: Mying changes.
> Akim> (&handle_tags): Fix a bug uncovered by the previous mying changes:
> A
Tom Tromey <[EMAIL PROTECTED]> writes:
> Akim> -local ($in_ac_output, $in_ac_replace) = (0, 0);
> Akim> +# Watchout: these guys need dynamic scope!
>
> Why remove the other local() though?
> Those variables also have dynamic scope :-(
Hm, do they?
I moved them inside the routines which
Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
>
> Akim> In fact, I've had to finish the job, because the following test
> Akim> was falling:
>
> Akim> cat > Makefil
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in: `my' the globals.
Index: automake.in
--- automake.in Fri, 23 Feb 2001 00:01:11 +0100 akim (am/f/39_automake.i 1.67 755)
+++ automake.in Sat, 24 Feb 2001 01:48:00 +0100 akim (am/f/39_automake.i 1.67 755
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&handle_man_pages): $found is dead.
Index: automake.in
--- automake.in Sat, 24 Feb 2001 13:30:31 +0100 akim (am/f/39_automake.i 1.69 755)
+++ automake.in Sat, 24 Feb 2001 13:35:08 +0100
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
Internally just store the conditionals as space separated lists of
CONDITIONS (instead of @CONDITIONS@).
* automake.in (&conditional_true_when, &variable_conditions_sub):
Split condit
return;
}
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&handle_gettext): Simplify redundant `if'.
Index: automake.in
--- automake.in Sat, 24 Feb 2001 14:51:56 +0100 akim (am/f/39_automake.i 1.71 755)
+++ automake.in Sat, 24 Feb 2001 14:59:50
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (@installdirs): Remove.
(&file_contents): Be able to grow Automake macros with `+='.
(&handle_texinfo, &handle_man_pages, &am_install_var): Let your
all-recursive' : 'all-am';
+ $all_target = (&variable_defined ('SUBDIRS')
+ ? 'all-recursive' : 'all-am');
) will anyway disapear from automake some day, moved into the right
*.am file.
Index: ChangeLog
from Aki
Index: ChangeLog
from Akim Demaille <[EMAIL PROTECTED]>
* automake.in (&handle_merge_targets): Move `install-strip' from
here...
* install.am: to here.
Index: automake.in
--- automake.in Sat, 24 Feb 2001 15:40:19 +0100 akim (am/f/39_automake.i 1.73 755)
301 - 400 of 929 matches
Mail list logo
