Re: 52-factored-install.patch

2001-02-26 Thread Akim Demaille
I should have emphasized that there are no diffs for Automake's Makefile.ins. The test suite is now at libobj*.test, no errors so far.

Re: 52-factored-install.patch

2001-02-26 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > Me, too, but: > > * Automake is really slow > * We already have tons of other global constants > > Please change it back. No problem!

Re: 52-factored-install.patch

2001-02-26 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: >> How smart is Perl? >> I used to have this as a global so that it wasn't reevaluated each >> time through the loop. It never changes -- it is a constant >> structure. Akim> I privileged readability and modularity over efficiency. Me,

Re: 52-factored-install.patch

2001-02-26 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > Akim> + # Directories installed during 'install-exec' phase. > Akim> + my %exec_dir_p = > Akim> + ( > Akim> [ ... ] > > How smart is Perl? > I used to have this as a global so that it wasn't reevaluated each > time through

Re: 52-factored-install.patch

2001-02-26 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (&am_install_var): Recognize whether predefined Akim> directories are `data' or `exec'. Akim> * libs.am, ltlib.am, progs.am, python.am, scripts.am: Install Akim> ?EXEC?. You checked this in already, which is fi

Re: 52-factored-install.patch

2001-02-26 Thread Akim Demaille
Given that this time I think I understood, that the bug was really bad for users, I went ahead and applied it. Tom, feel free to strengthen instdata2! Here is what I committed. #! /bin/sh # Test that installing under $exec_prefix is handled by install-exec. # Testing with headers for instance.

Re: 52-factored-install.patch

2001-02-26 Thread Akim Demaille
I think I finally have it. I apologize for all the iterations I needed :( Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&am_install_var): Recognize whether predefined directories are `data' or `exec'. * libs.am, ltlib.am, progs.am, python.am, s

Re: 52-factored-install.patch

2001-02-26 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> So it is my (today's :) understanding that I did wrong to have Akim> some primaries independent from the dir name, right? Right. The primaries are irrelevant at this stage. Akim> But both CVS and 1.4 refuse sbin_DATA :( Sorry, ba

Re: 52-factored-install.patch

2001-02-26 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > > "Akim" == akim <[EMAIL PROTECTED]> writes: > > Akim> My bad, I forgot to remove `dir' from the _DATA var. > > Oops, I missed that too. > > Akim> mybindir = $(bindir)/my > Akim> mybin_DATA = doto > > User defined variables are handled by name. >

Re: 52-factored-install.patch

2001-02-25 Thread Tom Tromey
> "Akim" == akim <[EMAIL PROTECTED]> writes: Akim> My bad, I forgot to remove `dir' from the _DATA var. Oops, I missed that too. Akim> mybindir = $(bindir)/my Akim> mybin_DATA = doto User defined variables are handled by name. I agree that this case is handled correctly with the current c

Re: 52-factored-install.patch

2001-02-25 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> .am:19: invalid variable `mysysconfdir_DATA' This should read `Makefile.am:19:...'. Do you know why it broke? Tom

Re: 52-factored-install.patch

2001-02-25 Thread akim
On Sun, Feb 25, 2001 at 06:00:29PM -0700, Tom Tromey wrote: > > Akim> mysbindir = $(sbin)/my > Akim> mysbindir_DATA = data > > Akim> .am:11: invalid variable `mysbindir_DATA' > > This is a new failure, since this feature has worked for a long time. My bad, I forgot to remove `dir' from the _DA

Re: 52-factored-install.patch

2001-02-25 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> I think I have this right too, since not all the *.am file Akim> depend on ?EXEC?. See SCRIPTS and PROGRAMS below. The point is that it is the directory name and not the primary that determines which install-* target to use. For i

Re: 52-factored-install.patch

2001-02-25 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > > "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: > > Akim> . &transform_cond ('BASE' => $strip_subdir, > Akim> 'EXEC' => $X =~ /exec/) > > This is still wrong. > > U

Re: 52-factored-install.patch

2001-02-25 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> . &transform_cond ('BASE' => $strip_subdir, Akim>'EXEC' => $X =~ /exec/) This is still wrong. User-defined directory names follow the `exec'-name test. autoconf/au

Re: 52-factored-install.patch

2001-02-25 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > One example of how the new code can fail is a system-dependent header > file install: > > inclexecdir = $(exec_prefix)/include ## Just an example! > inclexec_HEADERS = my-config.h > > With the new code the header will be installed by `install-d

Re: 52-factored-install.patch

2001-02-24 Thread akim
On Sat, Feb 24, 2001 at 10:29:14AM -0700, Tom Tromey wrote: > > "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: > > Akim> Here is the updated version of the patch. > > This patch still has a problem. See below. > > Akim> Note that there is a new side effect to my patch: the > Akim> (un

Re: 52-factored-install.patch

2001-02-24 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> Here is the updated version of the patch. This patch still has a problem. See below. Akim> Note that there is a new side effect to my patch: the Akim> (un)?install-man targets appeared (since you said they were Akim> required) in

Re: 52-factored-install.patch

2001-02-23 Thread Tom Tromey
> "Tom" == Tom Tromey <[EMAIL PROTECTED]> writes: Tom> 2. Pushing the hook out means that if the user has a -local as Tom> well as a -hook then we have to rewrite the -local target to add Tom> a new dependency. We've never rewritten user targets, so this is Tom> a major(-ish) change to conte

Re: 52-factored-install.patch

2001-02-23 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: >> If it is a dependency then it won't work properly with a parallel >> make. That's because the -hook for must be run after all the other >> rules. Akim> Why don't we use dependencies to say that -hook must be run Akim> last, instead of

Re: 52-factored-install.patch

2001-02-23 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > Akim> Below are first presented the Makefiles of Automake. There is > Akim> obviously a problem, which I do not solve in this already too > Akim> long patch, but will address later: what do with empty targets. > > This patch is ok, but please don't check

Re: 52-factored-install.patch

2001-02-23 Thread Akim Demaille
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>

52-factored-install.patch

2001-02-22 Thread Akim Demaille
This patch applies the same transformation, factoring, to install-data, install-exec and uninstall. Below are first presented the Makefiles of Automake. There is obviously a problem, which I do not solve in this already too long patch, but will address later: what do with empty targets. For ins

Re: 52-factored-install.patch

2001-02-21 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> This patch applies the same transformation, factoring, to Akim> install-data, install-exec and uninstall. I haven't read this patch yet but I wanted to respond to some comments before I did. Akim> Below are first presented the Make

Re: 52-factored-install.patch

2001-02-21 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> This patch applies the same transformation, factoring, to Akim> install-data, install-exec and uninstall. Akim> Below are first presented the Makefiles of Automake. There is Akim> obviously a problem, which I do not solve in this a