Skip to content

flake: add packages.<system>.default#2835

Merged
teto merged 1 commit intonix-community:masterfrom
otavio:topic/add-packages-default
Apr 2, 2022
Merged

flake: add packages.<system>.default#2835
teto merged 1 commit intonix-community:masterfrom
otavio:topic/add-packages-default

Conversation

@otavio
Copy link
Copy Markdown
Contributor

@otavio otavio commented Mar 24, 2022

Description

Since Nix 2.7.0, defaultPackage is deprecated and instead we should
use packages.<system>.default. This commit adds the new field but
keep the old for backward compatibility.

Checklist

  • Change is backwards compatible.

  • Code formatted with ./format.

  • Code tested through nix-shell --pure tests -A run.all.

  • Test cases updated/added. See example.

  • Commit messages are formatted like

    {component}: {description}
    
    {long description}
    

    See CONTRIBUTING for more information and recent commit messages for examples.

  • If this PR adds a new module

    • Added myself as module maintainer. See example.

    • Added myself and the module files to .github/CODEOWNERS.

Comment thread flake.nix Outdated
@teto
Copy link
Copy Markdown
Collaborator

teto commented Apr 1, 2022

could you amend/rebase to trigger CI please, I would like to merge but can't with the stuck CI.

Since Nix 2.7.0, `defaultPackage` is deprecated and instead we should
use `packages.<system>.default`.  This commit adds the new field but
keep the old for backward compatibility.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
@otavio
Copy link
Copy Markdown
Contributor Author

otavio commented Apr 2, 2022

@teto done.

@teto teto merged commit cfab869 into nix-community:master Apr 2, 2022
jficz pushed a commit to jficz/home-manager that referenced this pull request Apr 7, 2022
@teto teto mentioned this pull request Aug 22, 2022
7 tasks
teto pushed a commit to teto/home-manager that referenced this pull request Aug 22, 2022
spacekookie pushed a commit to spacekookie/home-manager that referenced this pull request Feb 10, 2023
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.

3 participants