unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* two patches for the Makefiles
@ 2004-11-06 21:00 Han Boetes
  2004-11-09 11:05 ` Jan D.
  0 siblings, 1 reply; 7+ messages in thread
From: Han Boetes @ 2004-11-06 21:00 UTC (permalink / raw)


Hi,

Here are two patches for the emacs Makefiles:

This patch aims to make sure the only file which is forgotten
after a make maintainer-clean is also removed.


--- orig/lisp/Makefile.in
+++ mod/lisp/Makefile.in
@@ -312,7 +312,7 @@
 	fi
 
 maintainer-clean:
-	cd $(lisp); rm -f *.elc */*.elc $(AUTOGENEL)
+	-rm -f ./Makefile; cd $(lisp); rm -f *.elc */*.elc $(AUTOGENEL)
 
 # Generate/update files for the bootstrap process.
 



This patch causes that make install doesn't relink emacs before
installing emacs.


--- orig/src/Makefile.in
+++ mod/src/Makefile.in
@@ -969,10 +969,8 @@
    to both ensure that lwlib gets remade based on its dependencies
    in its own makefile,
    and remake temacs if lwlib gets changed by this.  */
-stamp-oldxmenu: ${OLDXMENU} ../src/$(OLDXMENU)
+stamp-oldxmenu: ${OLDXMENU}
 	touch stamp-oldxmenu
-/* Supply an ordering for parallel make.  */
-../src/$(OLDXMENU): ${OLDXMENU}
 
 #ifdef USE_X_TOOLKIT
 $(OLDXMENU): really-lwlib




# Han

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: two patches for the Makefiles
  2004-11-06 21:00 two patches for the Makefiles Han Boetes
@ 2004-11-09 11:05 ` Jan D.
  2004-11-09 12:36   ` Han Boetes
  0 siblings, 1 reply; 7+ messages in thread
From: Jan D. @ 2004-11-09 11:05 UTC (permalink / raw)
  Cc: emacs-devel

Han Boetes wrote:
> Hi,
> 
> Here are two patches for the emacs Makefiles:
> 
> This patch aims to make sure the only file which is forgotten
> after a make maintainer-clean is also removed.

I've made maintainer-clean depend on distclean so the Makefile is removed.

> This patch causes that make install doesn't relink emacs before
> installing emacs.
> 

Does this happen on any other Emacs variant than GTK for you?  I've added a 
different fix for GTK.

	Jan D.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: two patches for the Makefiles
  2004-11-09 11:05 ` Jan D.
@ 2004-11-09 12:36   ` Han Boetes
  2004-11-09 12:55     ` Jan D.
  0 siblings, 1 reply; 7+ messages in thread
From: Han Boetes @ 2004-11-09 12:36 UTC (permalink / raw)


Jan D. wrote:
> Han Boetes wrote:
> > This patch causes that make install doesn't relink emacs
> > before installing emacs.
>
> Does this happen on any other Emacs variant than GTK for you?
> I've added a different fix for GTK.

Ehm this happens on GNU as well as on OpenBSD, with or without X.

Reproduction is easy: after a succesfull make run make install and
you'll see that emacs is linked again.

Lots of files are installed for a second time like the DOC file
and emacs-21.3.50.x and emacs-21.3.50.x+1



# Han

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: two patches for the Makefiles
  2004-11-09 12:36   ` Han Boetes
@ 2004-11-09 12:55     ` Jan D.
  2004-11-09 14:43       ` Han Boetes
  0 siblings, 1 reply; 7+ messages in thread
From: Jan D. @ 2004-11-09 12:55 UTC (permalink / raw)
  Cc: emacs-devel

Han Boetes wrote:
> Jan D. wrote:
> 
>>Han Boetes wrote:
>>
>>>This patch causes that make install doesn't relink emacs
>>>before installing emacs.
>>
>>Does this happen on any other Emacs variant than GTK for you?
>>I've added a different fix for GTK.
> 
> 
> Ehm this happens on GNU as well as on OpenBSD, with or without X.
> 
> Reproduction is easy: after a succesfull make run make install and
> you'll see that emacs is linked again.
> 
> Lots of files are installed for a second time like the DOC file
> and emacs-21.3.50.x and emacs-21.3.50.x+1

Sorry, this just doesn't happen for me with a current CVS.  I tried Lucid, GTK, 
no toolit, and no X.  Nothing gets relinked or regenerated on a make install 
after a successful make.

If you have GNU make you could check why make thinks it needs to relink Emacs 
with 'make --debug=b'.

	Jan D.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: two patches for the Makefiles
  2004-11-09 12:55     ` Jan D.
@ 2004-11-09 14:43       ` Han Boetes
  2004-11-09 14:51         ` Jan D.
  0 siblings, 1 reply; 7+ messages in thread
From: Han Boetes @ 2004-11-09 14:43 UTC (permalink / raw)


Jan D. wrote:
> Han Boetes wrote:
> > Jan D. wrote:
> > > Han Boetes wrote:
> > > > This patch causes that make install doesn't relink emacs
> > > > before installing emacs.
> > >
> > > Does this happen on any other Emacs variant than GTK for
> > > you? I've added a different fix for GTK.
> >
> > Ehm this happens on GNU as well as on OpenBSD, with or
> > without X.
> >
> > Reproduction is easy: after a succesfull make run make install
> > and you'll see that emacs is linked again.
> >
> > Lots of files are installed for a second time like the DOC
> > file and emacs-21.3.50.x and emacs-21.3.50.x+1
>
> Sorry, this just doesn't happen for me with a current CVS. I
> tried Lucid, GTK, no toolit, and no X. Nothing gets relinked or
> regenerated on a make install after a successful make.
>
> If you have GNU make you could check why make thinks it needs to
> relink Emacs with 'make --debug=b'.

I just tried reproducing it with current emacs, and indeed, it
doesn't happen anymore. About 6 month ago I spend a few days
hunting this bug, and I came up with that fix.

This behaviour must have changed recently. Sorry for the noise.



# Han

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: two patches for the Makefiles
  2004-11-09 14:43       ` Han Boetes
@ 2004-11-09 14:51         ` Jan D.
  2004-11-09 15:46           ` Han Boetes
  0 siblings, 1 reply; 7+ messages in thread
From: Jan D. @ 2004-11-09 14:51 UTC (permalink / raw)
  Cc: emacs-devel


> I just tried reproducing it with current emacs, and indeed, it
> doesn't happen anymore. About 6 month ago I spend a few days
> hunting this bug, and I came up with that fix.
>
> This behaviour must have changed recently. Sorry for the noise.

It actually did happen for GTK, so the noise was useful:-)

	Jan D.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: two patches for the Makefiles
  2004-11-09 14:51         ` Jan D.
@ 2004-11-09 15:46           ` Han Boetes
  0 siblings, 0 replies; 7+ messages in thread
From: Han Boetes @ 2004-11-09 15:46 UTC (permalink / raw)


Jan D. wrote:
> Han wrote:
> > I just tried reproducing it with current emacs, and indeed, it
> > doesn't happen anymore. About 6 month ago I spend a few days
> > hunting this bug, and I came up with that fix.
> >
> > This behaviour must have changed recently. Sorry for the noise.
>
> It actually did happen for GTK, so the noise was useful:-)

Well actually, that's how I build emacs all that time. I recently
changed to building emacs without X support. OK, that explains a
lot. :-)



# Han

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2004-11-09 15:46 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-11-06 21:00 two patches for the Makefiles Han Boetes
2004-11-09 11:05 ` Jan D.
2004-11-09 12:36   ` Han Boetes
2004-11-09 12:55     ` Jan D.
2004-11-09 14:43       ` Han Boetes
2004-11-09 14:51         ` Jan D.
2004-11-09 15:46           ` Han Boetes

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).