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

add meson build #130

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

add meson build #130

wants to merge 2 commits into from

Conversation

neheb
Copy link

@neheb neheb commented Dec 15, 2021

No description provided.

@neheb
Copy link
Author

neheb commented Dec 15, 2021

ERROR: Neither directory contains a build file meson.build.

Kinda stuck here...

@neheb neheb force-pushed the meson branch 2 times, most recently from ccaf1c8 to a046c77 Compare December 15, 2021 23:34
@serge-sans-paille
Copy link
Owner

@neheb thanks for working on this. Do you consider this PR ready for review?

@neheb neheb marked this pull request as draft February 2, 2022 00:20
@eli-schwartz
Copy link

Porting tests/CMakeLists.txt to meson would be nice. :) It's a fairly simple addition of a couple executables.

@neheb neheb force-pushed the meson branch 2 times, most recently from a349851 to 678ecba Compare May 24, 2022 22:26
neheb added 2 commits May 24, 2022 15:27
Allows seamless integration with meson projects.

Signed-off-by: Rosen Penev <[email protected]>
Signed-off-by: Rosen Penev <[email protected]>
- macOS-latest
steps:
- name : Fetch meson
run : sudo apt -y install meson

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably want to pip install here, because Ubuntu has an ancient version that doesn't support c++20 and the other OSes don't use apt anyway.

@neheb neheb marked this pull request as ready for review June 22, 2022 21:58
@neheb
Copy link
Author

neheb commented Jun 22, 2022

@serge-sans-paille this is ready. Only problem I see is:

../tests/test_unordered_set.cpp:68:42: error: modification of ‘ze_set’ is not a constant expression
   68 |   constexpr auto nocount = ze_set.count(4);

@risicle
Copy link

risicle commented Oct 16, 2022

I also get the above building 1.1.1 with gcc 11.3 on linux.

@risicle risicle mentioned this pull request Oct 16, 2022
13 tasks
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.

4 participants