Skip to content

flake: Fix devshell for darwin hosts - #5368

Merged
archseer merged 1 commit into
helix-editor:masterfrom
the-mikedavis:md-flake-darwin-devshell
Jan 2, 2023
Merged

flake: Fix devshell for darwin hosts#5368
archseer merged 1 commit into
helix-editor:masterfrom
the-mikedavis:md-flake-darwin-devshell

Conversation

@the-mikedavis

Copy link
Copy Markdown
Member

Runs of cargo build would fail with ld: framework not found CoreFoundation inside a devshell (nix develop) on a Darwin host.

This is fixed with two changes:

  • Including the CoreFoundation package within the devshell's derivation.
  • Switching from value to eval so that we append to the RUSTFLAGS environment variable rather than overwriting it. numtide/devshell already exposes the Library/Frameworks path within the devshell derivation but it was being overwritten by the manual setting of RUSTFLAGS in env.

Runs of `cargo build` would fail with `ld: framework not found
CoreFoundation` inside a devshell (`nix develop`) on a Darwin host.

This is fixed with two changes:

* Including the `CoreFoundation` package within the devshell's
  derivation.
* Switching from `value` to `eval` so that we append to the `RUSTFLAGS`
  environment variable rather than overwriting it. `numtide/devshell`
  already exposes the `Library/Frameworks` path within the devshell
  derivation but it was being overwritten by the manual setting of
  `RUSTFLAGS` in `env`.
@the-mikedavis the-mikedavis added C-bug Category: This is a bug A-packaging Area: Packaging and bundling S-waiting-on-review Status: Awaiting review from a maintainer. labels Jan 2, 2023
@archseer
archseer merged commit d6704ed into helix-editor:master Jan 2, 2023
@the-mikedavis
the-mikedavis deleted the md-flake-darwin-devshell branch January 2, 2023 16:28
semin-park pushed a commit to semin-park/helix that referenced this pull request Jan 4, 2023
Runs of `cargo build` would fail with `ld: framework not found
CoreFoundation` inside a devshell (`nix develop`) on a Darwin host.

This is fixed with two changes:

* Including the `CoreFoundation` package within the devshell's
  derivation.
* Switching from `value` to `eval` so that we append to the `RUSTFLAGS`
  environment variable rather than overwriting it. `numtide/devshell`
  already exposes the `Library/Frameworks` path within the devshell
  derivation but it was being overwritten by the manual setting of
  `RUSTFLAGS` in `env`.
semin-park pushed a commit to semin-park/helix that referenced this pull request Jan 4, 2023
Runs of `cargo build` would fail with `ld: framework not found
CoreFoundation` inside a devshell (`nix develop`) on a Darwin host.

This is fixed with two changes:

* Including the `CoreFoundation` package within the devshell's
  derivation.
* Switching from `value` to `eval` so that we append to the `RUSTFLAGS`
  environment variable rather than overwriting it. `numtide/devshell`
  already exposes the `Library/Frameworks` path within the devshell
  derivation but it was being overwritten by the manual setting of
  `RUSTFLAGS` in `env`.
hadronized pushed a commit to hadronized/helix that referenced this pull request Jan 4, 2023
Runs of `cargo build` would fail with `ld: framework not found
CoreFoundation` inside a devshell (`nix develop`) on a Darwin host.

This is fixed with two changes:

* Including the `CoreFoundation` package within the devshell's
  derivation.
* Switching from `value` to `eval` so that we append to the `RUSTFLAGS`
  environment variable rather than overwriting it. `numtide/devshell`
  already exposes the `Library/Frameworks` path within the devshell
  derivation but it was being overwritten by the manual setting of
  `RUSTFLAGS` in `env`.
seewishnew pushed a commit to seewishnew/helix that referenced this pull request Jan 6, 2023
Runs of `cargo build` would fail with `ld: framework not found
CoreFoundation` inside a devshell (`nix develop`) on a Darwin host.

This is fixed with two changes:

* Including the `CoreFoundation` package within the devshell's
  derivation.
* Switching from `value` to `eval` so that we append to the `RUSTFLAGS`
  environment variable rather than overwriting it. `numtide/devshell`
  already exposes the `Library/Frameworks` path within the devshell
  derivation but it was being overwritten by the manual setting of
  `RUSTFLAGS` in `env`.
freqmod pushed a commit to freqmod/helix that referenced this pull request Feb 8, 2023
Runs of `cargo build` would fail with `ld: framework not found
CoreFoundation` inside a devshell (`nix develop`) on a Darwin host.

This is fixed with two changes:

* Including the `CoreFoundation` package within the devshell's
  derivation.
* Switching from `value` to `eval` so that we append to the `RUSTFLAGS`
  environment variable rather than overwriting it. `numtide/devshell`
  already exposes the `Library/Frameworks` path within the devshell
  derivation but it was being overwritten by the manual setting of
  `RUSTFLAGS` in `env`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-packaging Area: Packaging and bundling C-bug Category: This is a bug S-waiting-on-review Status: Awaiting review from a maintainer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants