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

removeConfigureFlags appears to be broken (asks for list of strings, but throws a type error when receiving a list of strings) #296

Closed
ghost opened this issue Apr 3, 2024 · 9 comments
Assignees
Labels
bug Something isn't working case for tests Things we must automate testing of

Comments

@ghost
Copy link

ghost commented Apr 3, 2024

This bit throws an error: error: expected a set but found a string: "--with-gcc=gcc".

settings = {
    foo.removeConfigureFlags = [ "--with-gcc=gcc" ];
};

If I change the list to a set or change the string to a set, it says error: A definition for option 'removeConfigureFlags' is not of type 'null or (list of string)'. I've tried just about everything I can think of... is this attribute bugged?

@srid srid self-assigned this Apr 3, 2024
@srid srid added bug Something isn't working case for tests Things we must automate testing of labels Apr 3, 2024
@srid
Copy link
Owner

srid commented Apr 3, 2024

Can repro; looks like this was always broken. Fix incoming.

Aside: we need property tests for Nix.

@srid srid closed this as completed in cac0cff Apr 3, 2024
@ghost
Copy link
Author

ghost commented Apr 3, 2024

Is it actually removing the flag for you? I still see the flag I'm trying to remove.

@srid
Copy link
Owner

srid commented Apr 3, 2024

@srid
Copy link
Owner

srid commented Apr 3, 2024

Does setting buildFromSdist = false; make it work? If so, please a new bug report linking to #293

@srid
Copy link
Owner

srid commented Apr 3, 2024

Also make sure to try with this fix first #298

@ghost
Copy link
Author

ghost commented Apr 3, 2024

I might be doing something wrong... buildFromSdist doesn't appear to make a difference in the build. Putting settings into default doesn't seem to have an effect on my package either.

Out of curiosity, where are the default configure flags coming from? I see some that correspond to settings, but others like enable-library-vanilla and extra-lib-dirs repeated a few times with ncurses, libffi, elfutils, gmp-with-cxx.

I'm using horizon-platform for basePackages and devShell.tools if that matters at all.

@ghost
Copy link
Author

ghost commented Apr 3, 2024

I was trying to use the LLVM backend

@srid
Copy link
Owner

srid commented Apr 3, 2024

At this point, I'd suggest continuing this discussion in Zulip or Github Discussions.

@ghost
Copy link
Author

ghost commented Apr 3, 2024

Okay I'll pop in there. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working case for tests Things we must automate testing of
Projects
None yet
Development

No branches or pull requests

1 participant