Skip to content

Compiling without Boehm works#7

Merged
p01arst0rm merged 1 commit intop01arst0rm:mesonfrom
Pamplemousse:disable_gc
Jul 12, 2021
Merged

Compiling without Boehm works#7
p01arst0rm merged 1 commit intop01arst0rm:mesonfrom
Pamplemousse:disable_gc

Conversation

@Pamplemousse
Copy link
Copy Markdown

Signed-off-by: Pamplemousse xav.maso@gmail.com

  * seems to have been fixed by NixOS#4947
  * change option name to match current `./configure` option
  * tested by succesfully running `meson compile nixexpr`

Signed-off-by: Pamplemousse <xav.maso@gmail.com>
@p01arst0rm
Copy link
Copy Markdown
Owner

you need to change the definition for the option you changed here :)

nix/meson_options.txt

Lines 43 to 47 in 9ea5634

option(
'with_gc',
type : 'boolean',
value : 'true',
description : 'Build nix with Boehm garbage collector')

@Pamplemousse
Copy link
Copy Markdown
Author

AFAICT, this file is indeed updated by this PR.
I don't understand which version of the file you are quoting... 🤔

@p01arst0rm
Copy link
Copy Markdown
Owner

nope, you changed with_gc to enable_gc. needs to be updated in the options file too :)

@Pamplemousse
Copy link
Copy Markdown
Author

needs to be updated in the options file too :)

Which options file are you talking about?
meson_options.txt IS updated by this PR...

@p01arst0rm
Copy link
Copy Markdown
Owner

ahh

for some reason when i went to check it wasnt updated :/ my mistake!

@p01arst0rm p01arst0rm merged commit 969ea76 into p01arst0rm:meson Jul 12, 2021
@Pamplemousse Pamplemousse deleted the disable_gc branch July 15, 2021 18:06
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