From: "Martin Hundebøll" <martin@geanix.com>
To: Alexander Kanavin <alex.kanavin@gmail.com>
Cc: openembedded-core@lists.openembedded.org,
Khem Raj <raj.khem@gmail.com>,
Randy MacLeod <randy.macleod@windriver.com>
Subject: Re: [PATCH 3/5] ninja: build modified version with GNU Make jobserver support
Date: Wed, 03 Apr 2024 21:08:52 +0200 [thread overview]
Message-ID: <af5f84b11560409a0cb7e1657341d91d50beb84d.camel@geanix.com> (raw)
In-Reply-To: <CANNYZj8x4kEP4GAuaei6Ct3bqcZoBnLk21xArzSgTGRmO71sXQ@mail.gmail.com>
On Wed, 2024-04-03 at 17:58 +0200, Alexander Kanavin wrote:
> We can't carry this mass of patches. It has to land upstream first.
Understandable.
> Are you able/willing to drive the upstream pull request to
> completion?
No, I don't think so. I'm not familiar with the ninja codebase, and the
pull requests has been lingering for too long.
Another option might be to propose adding the fifo feature to the
kitware fork of ninja:
https://github.com/Kitware/ninja
They have forked ninja to get the jobserver feature in, but haven't
taken the fifo part so far.
The kitware fork is also used by buildroot:
https://gitlab.com/buildroot.org/buildroot/-/blob/master/package/ninja/ninja.mk
// Martin
next prev parent reply other threads:[~2024-04-03 19:08 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-03 7:01 [PATCH 0/5] Jobserver support Martin Hundebøll
2024-04-03 7:02 ` [PATCH 1/5] classes: jobserver: support gnu make fifo jobserver Martin Hundebøll
2024-04-04 7:56 ` [OE-core] " Andreas Helbech Kleist
2024-04-04 7:58 ` Martin Hundebøll
2024-04-03 7:02 ` [PATCH 2/5] scripts: build-env: allow passing JOBSERVER_FIFO from environment Martin Hundebøll
2024-04-03 7:02 ` [PATCH 3/5] ninja: build modified version with GNU Make jobserver support Martin Hundebøll
2024-04-03 7:26 ` Patchtest results for " patchtest
2024-04-03 15:58 ` Alexander Kanavin
2024-04-03 19:08 ` Martin Hundebøll [this message]
2024-04-08 21:39 ` Randy MacLeod
2024-05-18 7:13 ` Martin Hundebøll
2024-04-03 7:02 ` [PATCH 4/5] qemu: enable parallel builds when using the jobserver class Martin Hundebøll
2024-04-03 7:02 ` [PATCH 5/5] contrib: add python service and systemd unit to run shared jobserver Martin Hundebøll
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=af5f84b11560409a0cb7e1657341d91d50beb84d.camel@geanix.com \
--to=martin@geanix.com \
--cc=alex.kanavin@gmail.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=raj.khem@gmail.com \
--cc=randy.macleod@windriver.com \
/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.