Re: [PATCH] "make dist" did not depend on $(BUILT_SOURCES)

2017-11-28 Thread Jim Meyering
On Tue, Nov 28, 2017 at 8:35 PM, Nick Bowler wrote: > On 2017-11-28 18:13 -0800, Jim Meyering wrote: >> On Tue, Nov 28, 2017 at 12:45 PM, Nick Bowler wrote: >> > The Automake manual unequivocally states that BUILT_SOURCES files are >> > generated only when running 'make all', 'make check' or 'mak

Re: [PATCH] "make dist" did not depend on $(BUILT_SOURCES)

2017-11-28 Thread Nick Bowler
On 2017-11-28 18:13 -0800, Jim Meyering wrote: > On Tue, Nov 28, 2017 at 12:45 PM, Nick Bowler wrote: > > The Automake manual unequivocally states that BUILT_SOURCES files are > > generated only when running 'make all', 'make check' or 'make install'. > > > > So if they are going to be generated o

Re: [PATCH] "make dist" did not depend on $(BUILT_SOURCES)

2017-11-28 Thread Jim Meyering
On Tue, Nov 28, 2017 at 12:01 PM, Mathieu Lirzin wrote: > Hello Jim, ... >> >> Here's the patch I expect to push to master: ... > > OK to push. Thanks. Pushed. I have also removed the micro branch.

Re: [PATCH] port elisp-compilation support to emacs-23.1 and newer

2017-11-28 Thread Jim Meyering
On Tue, Nov 28, 2017 at 2:23 PM, Glenn Morris wrote: > Jim Meyering wrote: > >> Remember: this arises only in a non-srcdir build. That means build >> artifacts end up being written into the mostly-empty current directory >> hierarchy, which does not have copies of the sources. Installation >> proc

Re: [PATCH] "make dist" did not depend on $(BUILT_SOURCES)

2017-11-28 Thread Jim Meyering
On Tue, Nov 28, 2017 at 12:45 PM, Nick Bowler wrote: > Hi Jim, > > On 2017-11-28 11:21 -0800, Jim Meyering wrote: >> Date: Thu, 20 Mar 2014 12:31:32 -0700 >> Subject: [PATCH] "make dist" did not depend on $(BUILT_SOURCES) >> >> * lib/am/distdir.am (distdir-am): New intermediate target. >> Interpos

Re: [PATCH] port elisp-compilation support to emacs-23.1 and newer

2017-11-28 Thread Glenn Morris
Jim Meyering wrote: > Remember: this arises only in a non-srcdir build. That means build > artifacts end up being written into the mostly-empty current directory > hierarchy, which does not have copies of the sources. Installation > processes will continue to copy both .el and .elc files into plac

Re: [PATCH] port elisp-compilation support to emacs-23.1 and newer

2017-11-28 Thread Mathieu Lirzin
Jim Meyering writes: > On Mon, Nov 27, 2017 at 8:12 PM, Jim Meyering wrote: >> On Mon, Nov 27, 2017 at 12:52 PM, Jim Meyering wrote: >>> On Mon, Nov 27, 2017 at 10:27 AM, Glenn Morris wrote: In general, Emacs expects .el and .elc to be found in the same directory. Not adhering t

Re: [PATCH] "make dist" did not depend on $(BUILT_SOURCES)

2017-11-28 Thread Nick Bowler
Hi Jim, On 2017-11-28 11:21 -0800, Jim Meyering wrote: > Date: Thu, 20 Mar 2014 12:31:32 -0700 > Subject: [PATCH] "make dist" did not depend on $(BUILT_SOURCES) > > * lib/am/distdir.am (distdir-am): New intermediate target. > Interpose this target between $(distdir) and its dependency > on $(DIST

Re: [PATCH] "make dist" did not depend on $(BUILT_SOURCES)

2017-11-28 Thread Mathieu Lirzin
Hello Jim, Jim Meyering writes: > I was dismayed to discover this patch I wrote over three years ago. > Today I confirmed that the problem still remains by running these commands: > > git clone git://git.sv.gnu.org/hello.git \ > cd hello && ./bootstrap && ./configure && env make dist > > I

[PATCH] "make dist" did not depend on $(BUILT_SOURCES)

2017-11-28 Thread Jim Meyering
I was dismayed to discover this patch I wrote over three years ago. Today I confirmed that the problem still remains by running these commands: git clone git://git.sv.gnu.org/hello.git \ cd hello && ./bootstrap && ./configure && env make dist It failed like this: src/hello.c:27:10: fatal