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

Brush is incompatible with MacOS #16

Open
JDRomano2 opened this issue Sep 12, 2022 · 3 comments
Open

Brush is incompatible with MacOS #16

JDRomano2 opened this issue Sep 12, 2022 · 3 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@JDRomano2
Copy link
Collaborator

Brush requires a modern version of GCC to compile, but modern versions of GCC are hard to come by on MacOS (e.g., the newest supported version on conda-forge is v4.8.5).

It would be ideal if we could add support for clang++ (LLVM). The version that comes installed on most Macs has support for almost all of the C++20 feature proposals (see https://clang.llvm.org/cxx_status.html#cxx20).

This is probably not highest priority at this point, but I can see it being an important feature down the road.

@JDRomano2 JDRomano2 added bug Something isn't working enhancement New feature or request labels Sep 12, 2022
@JDRomano2 JDRomano2 self-assigned this Sep 12, 2022
@JDRomano2
Copy link
Collaborator Author

I haven't ported anything major from gcc to clang, but suspect this will mostly involve mucking about with the CMake files and inserting preprocessor directives in the c++ code.

@lacava
Copy link
Member

lacava commented Sep 12, 2022

this is good because @msnliu spent a lot of time trying to get it to work with Mac. Supporting clang is definitely worthwhile. We tried instead to install gcc-11 to no avail.

@folivetti
Copy link
Collaborator

not really sure if it is related or if it is of any help, but talking to Bogdan he mentioned that
he had to use a meta-package cxx-compiler to build operon conda package.

name: pyoperon
channels:
  - conda-forge
dependencies:
  - cxx-compiler
  - clangxx
  - cmake
  - eigen
  - git
  - make
  - ninja
  - pkg-config
  - pybind11
  - scikit-build
  - taskflow
  - xxhash

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants