Skip to content

PR: Add CYTHON environment variable to mesonbuild.envconfig.ENV_VAR_COMPILER_MAP attribute. - #13

Closed
thomasmansencal wants to merge 1 commit into
numpy:main-numpymesonfrom
thomasmansencal:patch-1
Closed

thomasmansencal wants to merge 1 commit into
numpy:main-numpymesonfrom
thomasmansencal:patch-1

Conversation

@thomasmansencal

Copy link
Copy Markdown

Hello,

The CYTHON key is required for custom build pipelines defining the build environment with variables.

Cheers,

Thomas

@rgommers

Copy link
Copy Markdown
Member

Thanks @kelsolaar. I see this was already done upstream: mesonbuild#12674. So I may take your PR as a ping to update to latest Meson in this repo, rather than only incorporate this fix.

@thomasmansencal

Copy link
Copy Markdown
Author

Sounds good with me!

@thomasmansencal

Copy link
Copy Markdown
Author

BTW, is there a reason for maintaining a fork and not pushing your changes to the main repo?

@rgommers

Copy link
Copy Markdown
Member

We have extensions for BLAS and SIMD detection that still need to be upstreamed. Once that's done we can drop our vendored copy.

@mattip

mattip commented May 17, 2024

Copy link
Copy Markdown
Member

This can be closed, right?

@rgommers

Copy link
Copy Markdown
Member

Yep, all good now. Thanks @kelsolaar and @mattip

@rgommers rgommers closed this May 17, 2024
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