All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bart Oldeman <bartoldeman@users.sourceforge.net>
To: linux-msdos@vger.kernel.org
Subject: Re: dosemu 1.1.5.7 (was Re: dosemu 1.1.5.6)
Date: Sat, 23 Aug 2003 23:17:14 +0100 (BST)	[thread overview]
Message-ID: <Pine.LNX.4.44.0308232306560.4390-100000@enm-bo-lt.localdomain> (raw)
In-Reply-To: <20030823215903.GO6464@dbz.icequake.net>

On Sat, 23 Aug 2003, Ryan Underwood wrote:

> On Sat, Aug 23, 2003 at 08:47:33PM +0100, Bart Oldeman wrote:
> > On Sat, 23 Aug 2003, Ryan Underwood wrote:
> >
> > > The problem is that when debian/rules install target calls the install
> > > target for DOSEMU, MAKELEVEL is 1, not zero, so abs_top_builddir is not
> > > set, and Makefile.main's install target tries to rm -rf /tmp (!).
> >
> > Looks like Debian fixed that:
> > http://cvs.debian.org/debian-installer/build/Makefile
> > http://cvs.debian.org/debian-installer/build/Makefile.diff?r1=1.154&r2=1.155
>
> Well, that particular patch looks like it is for the debian-installer
> package which is not related to DOSEMU.  However, I can use the same
> trick in my build script.  I'm mainly curious why this only started
> happening in 1.1.5.7 -- I built 1.1.5.6 and previous with no
> modifications or extra pain needed.

well the trick avoids using $(shell ... pwd) in Makefiles for each
recursive invocation of "make". $(shell) is fairly expensive; by reducing
the number of it to exactly one (instead of ~100 in 1.1.5.6) it was
possible to reduce the time to "make" a (nearly) built tree
significantly, on one PC by about 10 seconds from 15 to less than 5; and
Clarence Dang has seen even greater differences.

For a user who compiles her/himself this doesn't matter that much but for
a developer who often makes a small change and then recompiles it saves a
lot of waiting.

Bart


  reply	other threads:[~2003-08-23 22:17 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-07-20 16:33 dosemu 1.1.5.6 Bart Oldeman
2003-07-21  5:13 ` Grigory Batalov
2003-07-25 18:15 ` Eemeli Kantola
2003-08-22 23:10   ` dosemu 1.1.5.7 (was Re: dosemu 1.1.5.6) Bart Oldeman
2003-08-23  7:21     ` Eemeli Kantola
2003-08-23 17:58     ` Ryan Underwood
2003-08-23 19:47       ` Bart Oldeman
2003-08-23 21:59         ` Ryan Underwood
2003-08-23 22:17           ` Bart Oldeman [this message]
2003-08-23 22:36             ` Ryan Underwood

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Pine.LNX.4.44.0308232306560.4390-100000@enm-bo-lt.localdomain \
    --to=bartoldeman@users.sourceforge.net \
    --cc=linux-msdos@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.