Skip to content

update cc code: tokenpool-client, tokenpool-master, jobserver-fifo #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
milahu opened this issue Aug 11, 2023 · 0 comments
Open

Comments

@milahu
Copy link
Owner

milahu commented Aug 11, 2023

currently, the cc folder has an old version of stefanb2's topic-tokenpool-master branch
see also https://github.com/milahu/ninja/tree/stefanb2/topic-tokenpool-master-old

the new versions should be merged:

merged commits

git log --format=$'%H\n%s\n' -- cc/ | tac

ninja-build/ninja/pull/1140

cc: init from upstream https://github.com/ninja-build/ninja
5ea78ff
add COPYING

cc: Add GNU make jobserver client support
0ab7545 from ninja-build/ninja@3e04d21

cc: Add TokenPool monitoring to SubprocessSet::DoWork()
21a6390 from ninja-build/ninja@5c8dc53

cc: Ignore jobserver when -jN is forced on command line
864c5d3 from ninja-build/ninja@a6eec5c

cc: Honor -lN from MAKEFLAGS
25cb1ad from ninja-build/ninja@3d15060

cc: Use LinePrinter for TokenPool messages
3331c4c from ninja-build/ninja@1157e43

cc: Prepare PR for merging
f9d9469 from ninja-build/ninja@6100863

cc: Add tests for TokenPool
1be7cb3 from ninja-build/ninja@008bc6e

Add tests for subprocess module
ninja-build/ninja@cf02adc
not merged

Add tests for build module
ninja-build/ninja@f99d747
not merged

cc: Add Win32 implementation for GNUmakeTokenPool
178bf97 from ninja-build/ninja@71ad7da

cc: Prepare PR for merging - part II
b92c1a2 from ninja-build/ninja@05d39cb

ninja-build/ninja/pull/2260

cc: Rename TokenPool::Setup() to SetupClient()
343b7de from ninja-build/ninja@6ec8fb0

cc: Add TokenPool::SetupMaster() method
2b498af from ninja-build/ninja@89e4aef

Add command line option -m/--tokenpool-master
ninja-build/ninja@0ec8c4a
not merged

cc: Implement GNUmakeTokenPool::SetupMaster()
5a21c33 from ninja-build/ninja@8d21910

cc: Implement GNUmakeTokenPool*::CreatePool()
1e6f188 from ninja-build/ninja@6602155

cc: Add tests for TokenPool::SetupMaster()
573c9bb from ninja-build/ninja@bee56b6

ninja-build/ninja/pull/2263

Add GNU make jobserver fifo style client support
ninja-build/ninja@e615e63
TODO merge

Add optional argument to -m/--tokenpool-master
ninja-build/ninja@66350c0
not merged

Add GNU make jobserver fifo style master support
ninja-build/ninja@c9e21db
TODO merge

other commits

cc: set -j in MAKEFLAGS
a97d5ad

cc: add test
a22030e

cc: add debug prints
4938d37

cc: use token value in acquire + release
0d1eb00

cc: fix test
a36c159

cc: mv COPYING license.txt
5d003bd

cc: rename ReturnToken -> ReleaseToken
fc83de9

cc: expose low-level interface
4c9900d

cc: test low-level interface
62c2c68

cc: add test/test.sh
9c06681

cc: test: explain low vs high level
82c5362

cc: test: tokens_ -> tokenpool_
83c0fca

cc: test: refactor
d119ba3

cc: test: add cleanup
4465d58

cc: test: %i -> %li
78e7ed8

cc: fix warnings
d598c4d

cc: test: job_count
30b573d

cc: add readme
4eb559a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant