Skip to content
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

added support for ninja generator #229

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

RisingLiberty
Copy link

I added a generator to create Ninja files, and Ninja projects and solution, making it easy to build everything

@RisingLiberty RisingLiberty changed the title added support for ninja generator (executables only) added support for ninja generator Nov 17, 2022
@RisingLiberty
Copy link
Author

RisingLiberty commented Nov 18, 2022

I'm still developing this as I'd like to clean up the code while also fully support MSVC, Clang and GCC.
The compiler's standard library is not found when using the Ninja files (CMAKE has the same problem)

@RisingLiberty
Copy link
Author

support for gcc is added, support for static libraries and dynamic libraries (except dynamic libs for gcc) and dependency support as well.
I will use it now in practice, but my initial tests were successful

@RisingLiberty
Copy link
Author

Verified on a bigger project of mine that combines static library, executable and unit tests. All were successful

@jspelletier
Copy link
Collaborator

Hi,

Sorry for the lack of comments for your PR. We are in major repo ownership changes. We will take a look at your changes eventually.

Thanks for your understanding.

@jspelletier jspelletier changed the base branch from dev to main April 6, 2023 12:52
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.

2 participants