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

src/main.sh: Add --experimental-features and --extra-experimental-features support #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ShamrockLee
Copy link

@ShamrockLee ShamrockLee commented Feb 10, 2022

  • Default to using --extra-experimental-features
    instead of --experimental-features
    to show respect to the environment configuration.

  • Add flags --extra-experimental-features and --experimental-features
    to allow manual specification to the experimental-features setting
    (e.g. use builtins.getFlake when not globally enabled).

Fixes #35

@ShamrockLee ShamrockLee marked this pull request as draft February 10, 2022 04:40
@ShamrockLee ShamrockLee changed the title src/main.sh: Add --experimental-features and --extra-experimental-features support [WIP] src/main.sh: Add --experimental-features and --extra-experimental-features support Feb 10, 2022
@ShamrockLee ShamrockLee changed the title [WIP] src/main.sh: Add --experimental-features and --extra-experimental-features support src/main.sh: Add --experimental-features and --extra-experimental-features support Feb 10, 2022
@ShamrockLee ShamrockLee marked this pull request as ready for review February 10, 2022 05:37
@Kranzes
Copy link

Kranzes commented Mar 4, 2022

@msteen Can you please merge this and tag a new release?

@ShamrockLee
Copy link
Author

Describe the changes inside the CHANGELOG.md for version 0.4.2.

@ShamrockLee
Copy link
Author

@msteen Could you help take a look?

*   Default to using --extra-experimental-features
    instead of --experimental-features
    to show respect to the environment configuration.

*   Add flags --extra-experimental-features and --experimental-features
    to allow manual specification to the experimental-features setting
    (e.g. use `builtins.getFlake` when not globally enabled).
@ShamrockLee
Copy link
Author

I accidentally base the CHANGELOG.md work on a non-relevant commit. Just solve the problem.

@jakubgs
Copy link

jakubgs commented Nov 8, 2023

Any news on this? I'd love to be able to generate a new vendor SHA for packages built with buildGoModule in a flake.nix.

m15a added a commit to m15a/flake-awesome-neovim-plugins that referenced this pull request May 10, 2024
Hopefully fixes: #130

Even though nix-installer-action enables flakes by default,
it is overrided by nix-prefetch since it sets `--experimental-features`.

See msteen/nix-prefetch#34
@eclairevoyant
Copy link

This repo hasn't seen activity in 3 years, it's time for a fork.

@ShamrockLee
Copy link
Author

@eclairevoyant Considering that this project is mission-critical to Nixpkgs, it sounds reasonable to either merge it into Nixpkgs or move it under the nix-community organization. How should we proceed with the forking?

I opened an issue about this two years ago (#40). We could discuss it there.

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.

Add flake support
4 participants