Project

General

Profile

Feature #9579

[PATCH] ext/extmk.rb: parallelize configure

Added by normalperson (Eric Wong) almost 6 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Target version:
-
[ruby-core:61142]

Description

This reduces "touch ext/extmk.rb && make -j" time from
roughly 38 to 16 seconds on my 8 core machine with
an otherwise fully-built source tree.

http://bogomips.org/ruby.git/patch?id=bc6acbaa40

git pull git://80x24.org/ruby.git extmk-parallel

History

Updated by nobu (Nobuyoshi Nakada) almost 6 years ago

Does it work with static-linked-ext or compiled? method?

Updated by normalperson (Eric Wong) almost 6 years ago

nobu@ruby-lang.org wrote:

Does it work with static-linked-ext or compiled? method?

Oops, I'll see if I can fix those.

Updated by normalperson (Eric Wong) almost 6 years ago

nobu@ruby-lang.org wrote:

Does it work with static-linked-ext or compiled? method?

OK, I should have both of them working with a lot of new code
and pushed out an addendum patch:

http://bogomips.org/ruby.git/patch?id=84ea2b48a
git://80x24.org/ruby.git extmk-parallel-v2

Even without these patches, I get a lot of
"warning: already initialized constant" warnings when running
`make check' and that seems to cause failures...

Updated by normalperson (Eric Wong) almost 6 years ago

Trivial addendum for v3 (pushed earlier, but I lost my Internet
connection before I could email).
http://bogomips.org/ruby.git/patch?id=5779e6040
git://80x24.org/ruby.git extmk-parallel-v3

Updated by normalperson (Eric Wong) almost 6 years ago

normalperson@yhbt.net wrote:

Trivial addendum for v3 (pushed earlier, but I lost my Internet
connection before I could email).
http://bogomips.org/ruby.git/patch?id=5779e6040
git://80x24.org/ruby.git extmk-parallel-v3

5779e60 ext/extmk.rb: uniq extlibs
195e511 ext/extmk.rb: fix for static-linked-ext and compiled? usage
3029a5e ext/extmk.rb: parallelize configure

I'll squash this series together and commit in a few days unless there's
objections/bugs found. It should help productivity for developers with
fork enabled systems and gracefully fallback for systems without fork.

Updated by nobu (Nobuyoshi Nakada) almost 6 years ago

EXT_DEPS doesn't look good to me.

Updated by normalperson (Eric Wong) almost 6 years ago

Thanks for taking a look. I not sure what to do about EXT_DEPS.
The compiled? check is strange.

dl and tk will be gone soon, we can wait (or remove dl+tk sooner).
win32 cannot fork, so the special case there is not needed, even.

#8

Updated by naruse (Yui NARUSE) almost 2 years ago

  • Target version deleted (2.2.0)
#9

Updated by normalperson (Eric Wong) almost 2 years ago

  • Status changed from Open to Closed

Also available in: Atom PDF