[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: 52-factored-install.patch
From: |
akim |
Subject: |
Re: 52-factored-install.patch |
Date: |
Sat, 24 Feb 2001 20:29:03 +0100 |
User-agent: |
Mutt/1.3.12i |
On Sat, Feb 24, 2001 at 10:29:14AM -0700, Tom Tromey wrote:
> >>>>> "Akim" == Akim Demaille <address@hidden> 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 Automake.
>
> I misspoke. I don't think these targets aren't required. Under
> --cygnus mode, `install-info' is required, but that is different.
>
> Akim> - if (defined $exec_dir_p{$X} ? $exec_dir_p{$X} : ($X =~
> /exec/))
>
> You pushed half of this test out into the *.am files. That's fine.
> But the second half of this test is important, and I don't understand
> how it is handled in the revised code.
>
> 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-data'.
> However, that is wrong. `install-exec' ought to install this header.
Aaaaaaaaaaaah! :(
OK, I will write the test, and I will provide a fix for this. But again,
hm, my stack is stacking...
- Re: ##-xxx-xxx.patch, (continued)
- Re: 52-factored-install.patch, Tom Tromey, 2001/02/21
- Re: 52-factored-install.patch, Akim Demaille, 2001/02/23
- Re: 52-factored-install.patch, Tom Tromey, 2001/02/24
- Re: 52-factored-install.patch,
akim <=
- Re: 52-factored-install.patch, Akim Demaille, 2001/02/25
- Re: 52-factored-install.patch, Tom Tromey, 2001/02/25
- Re: 52-factored-install.patch, Akim Demaille, 2001/02/25
- Re: 52-factored-install.patch, Tom Tromey, 2001/02/25
- Re: 52-factored-install.patch, akim, 2001/02/25
- Re: 52-factored-install.patch, Tom Tromey, 2001/02/25
- Re: 52-factored-install.patch, Akim Demaille, 2001/02/26
- Re: 52-factored-install.patch, Tom Tromey, 2001/02/26
- Re: 52-factored-install.patch, Akim Demaille, 2001/02/26
- Re: 52-factored-install.patch, Tom Tromey, 2001/02/26