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

[Feature Request:] Overriding the stdenv not yet possible #567

Closed
573 opened this issue May 2, 2023 · 7 comments · Fixed by #1092
Closed

[Feature Request:] Overriding the stdenv not yet possible #567

573 opened this issue May 2, 2023 · 7 comments · Fixed by #1092
Assignees
Labels
bug Something isn't working
Milestone

Comments

@573
Copy link

573 commented May 2, 2023

Describe the bug
Currently it is not easily possible to override the stdenv, i. e. to compile to certain targets in rust / cargo.

To reproduce
See sscce for this issue.

Version

devenv: 0.6.2

@573 573 added the bug Something isn't working label May 2, 2023
@workflow
Copy link

workflow commented May 2, 2023

Related: #530

@domenkozar
Copy link
Member

We're getting closer to fixing this, I'm not sure if overriding makes sense for the whole devenv but rather than parts of it?

@domenkozar
Copy link
Member

For Rust see #900

Otherwise I'd suggest we add an option to devenv.yaml:

stdenv: pkgs.swift.stdenv

Would that work for your use case?

@domenkozar domenkozar modified the milestones: 1.0, 1.1 Mar 27, 2024
@573
Copy link
Author

573 commented Mar 27, 2024

#900 looks like what is missing, the devenv.yaml option could be feasible from flakes as well then further up the road, so yeah, this would be a nice addition.

@haraldh
Copy link

haraldh commented Mar 28, 2024

my use case would be stdenv = pkgs.stdenvAdapters.useMoldLinker pkgs.stdenv;

        rustVersion = pkgs.rust-bin.fromRustupToolchainFile ./rust-toolchain;

        stdenv = pkgs.stdenvAdapters.useMoldLinker pkgs.stdenv;

        rustPlatform = (pkgs.makeRustPlatform {
          cargo = rustVersion;
          rustc = rustVersion;
          inherit stdenv;
        });

@domenkozar
Copy link
Member

Can you try #1092 and tell me if that works for you as expected?

@573
Copy link
Author

573 commented Apr 2, 2024

Tried and worked. Thanks all for the work !

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

Successfully merging a pull request may close this issue.

4 participants