Skip to content

Added note about ninja_syntax names being in the ninja namespace. - #359

Merged
henryiii merged 2 commits into
scikit-build:masterfrom
PythonCHB:update_readme
Aug 21, 2026
Merged

Added note about ninja_syntax names being in the ninja namespace.#359
henryiii merged 2 commits into
scikit-build:masterfrom
PythonCHB:update_readme

Conversation

@PythonCHB

Copy link
Copy Markdown
Contributor

Given all the confusion over on conda-forge, [*] I thought a little more docs would be helpful here.

If we had it to start all over again, I'd probably have simply installed ninja_syntax.py as a top level module, so it would work just like online docs suggest, e.g.:

import ninja_syntax
...
writer = ninja_syntax.Writer(sys.stdout)

from: https://bernsteinbear.com/blog/ninja-is-enough/

From a python-centric perspective this makes sense, too -- but should be documented :-).

The fact is that most (all?) folks using this package are simply getting it from a requirements file someone else set up -- I suspect there are very few people installing ninja from pypi so that they can use ninja_syntax anyway, but maybe some will?

I also added a note about:

python -m ninja

since it is there, it should be documented.

[*] conda-forge/ninja-feedstock#26

@henryiii
henryiii merged commit 59c7c31 into scikit-build:master Aug 21, 2026
21 checks passed
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

Successfully merging this pull request may close these issues.

3 participants