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

readme: clarify flake instructions #388

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

TornaxO7
Copy link

@TornaxO7 TornaxO7 commented Oct 4, 2024

No description provided.

@lilyinstarlight lilyinstarlight changed the title Update installations instruction for flake readme: clarify flake instructions Oct 4, 2024
readme.md Outdated
Comment on lines 11 to 13
If you have an existing `configuration.nix`, you can use the `nixos-cosmic` flake with the following in an adjacent `flake.nix` (e.g. in `/etc/nixos`):

**Note:** If switching from traditional evaluation to flakes, `nix-channel` will no longer have any effect on the nixpkgs your system is built with, and therefore `nixos-rebuild --upgrade` will also no longer have any effect. You will need to use `nix flake update` from your flake directory to update nixpkgs and nixos-cosmic.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has been used by people coming to this repository before. Please do not remove the information

@@ -6,17 +6,25 @@ Dedicated development matrix room: <https://matrix.to/#/#cosmic:nixos.org>

## Usage

### With traditional evaluation

This isn't supported, you are required to use [nix flakes] if you want to use nixos-cosmic.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So uh this isn't true either. There is a valid entrypoint intended for that use-case, it's just undocumented and a bad idea unless you are using some other pinning/locking mechanism, e.g. npins

I'll provide more concrete guidance later on what this can be changed to, though. Thanks for working on these docs!

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.

2 participants