Skip to content

Comments

Port nix.sh to the fish shell#626

Closed
tchajed wants to merge 1 commit intoNixOS:masterfrom
tchajed:fish-profile
Closed

Port nix.sh to the fish shell#626
tchajed wants to merge 1 commit intoNixOS:masterfrom
tchajed:fish-profile

Conversation

@tchajed
Copy link

@tchajed tchajed commented Aug 28, 2015

Sourcing nix.sh won't work on fish shell since it doesn't use the same syntax. This patch adds a new nix.fish that can be sourced from fish.

@domenkozar
Copy link
Member

I've seen requests for this, it's going to be pain to maintain (we'd need a test).

@Ericson2314
Copy link
Member

@domenkozar Hmm? I think it would be pretty popular and thus less likely to rot. But what tests were you thinking of?

@domenkozar
Copy link
Member

domenkozar commented Jun 3, 2016

@tchajed could you rebase and sync with recent changes? I don't see why we shouldn't support this as long as it's maintained.

@yacinehmito
Copy link

@tchajed What is the status on this please ? I would be happy to help.

@tchajed
Copy link
Author

tchajed commented Sep 25, 2016

It looks like the sh profile has become more complicated. While I fixed merge conflicts, the new profile needs to be translated to fish again. I'm not able to get nix to compile, and since I don't actually use it I'd rather pass this on to someone who will maintain it.

@gpyh: feel free to use this as a starting point and create your own pull request! I'd rather not submit code I haven't tested.

@nhooyr
Copy link

nhooyr commented Oct 12, 2016

I'm gonna try and work on this in a new PR.

@pickfire
Copy link

Any news on this?

@tchajed
Copy link
Author

tchajed commented May 11, 2020

I'm closing in favor of the latest attempt by @asymmetric in #1090, which hopefully gets resolved and merged.

@tchajed tchajed closed this May 11, 2020
@pickfire
Copy link

@tchajed @asymmetric issue was closed.

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.

8 participants