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

loaOf deprecation warnings or errors #77189

Closed
worldofpeace opened this issue Jan 7, 2020 · 6 comments
Closed

loaOf deprecation warnings or errors #77189

worldofpeace opened this issue Jan 7, 2020 · 6 comments
Labels
0.kind: bug Something is broken 2.status: work-in-progress This PR isn't done 3.skill: sprintable A larger issue which is split into distinct actionable tasks 3.skill: trivial This is trivial to complete (typically find-and-replace) 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS
Milestone

Comments

@worldofpeace
Copy link
Contributor

Describe the bug
The type loaOf was deprecated and its use will spring up evaluation warnings like

trace: warning: In file <unknown-file>
a list is being assigned to the option config.fileSystems.
This will soon be an error as type loaOf is deprecated.
See https://git.io/fj2zm for more information.
Do
  fileSystems =
    { unnamed = {...}; ...}
instead of
  fileSystems =
    [ { name = "unnamed"; ...} ...]

This was done in #63103, but because the scope of the change
it wasn't feasible to get them all in one go and there was a few syntax related errors #77177.

rg "loaOf" returned this for me at the time of writing

nixos/modules/config/users-groups.nix:2
nixos/modules/security/pam.nix:1
nixos/modules/tasks/encrypted-devices.nix:1
nixos/modules/tasks/filesystems.nix:2
nixos/modules/tasks/network-interfaces.nix:1
nixos/modules/programs/tsm-client.nix:2
nixos/modules/programs/ssh.nix:1
nixos/modules/virtualisation/railcar.nix:2
nixos/modules/virtualisation/containers.nix:1
nixos/modules/services/backup/znapzend.nix:2
nixos/modules/system/etc/etc.nix:1
nixos/modules/services/desktops/geoclue2.nix:1
nixos/modules/system/boot/stage-1.nix:1
nixos/modules/system/boot/luksroot.nix:1
nixos/modules/services/networking/bitcoind.nix:1
nixos/modules/services/security/tor.nix:1
nixos/modules/services/networking/i2pd.nix:2
nixos/modules/services/x11/picom.nix:3
nixos/modules/services/networking/prosody.nix:1
nixos/modules/services/networking/nylon.nix:1
nixos/doc/manual/development/option-types.xml:1
nixos/modules/installer/tools/nixos-option/nixos-option.cc:1
nixos/modules/services/hardware/sane_extra_backends/brscan4.nix:1
nixos/modules/services/networking/hylafax/options.nix:2
nixos/modules/services/networking/ssh/sshd.nix:1

and it appears some modules were probably added while the PR was opened used the old format.

What to do if encountered
Duplicate such changes like in:

Here's the PR that added the release note, with details about the change #77109.

@worldofpeace worldofpeace added 0.kind: bug Something is broken 2.status: work-in-progress This PR isn't done 3.skill: sprintable A larger issue which is split into distinct actionable tasks 3.skill: trivial This is trivial to complete (typically find-and-replace) 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS labels Jan 7, 2020
@worldofpeace worldofpeace added this to the 20.03 milestone Jan 7, 2020
@Mindavi
Copy link
Contributor

Mindavi commented Jan 13, 2020

transmission is using the users.users and users.groups configuration, so also needs to be patched.

@worldofpeace
Copy link
Contributor Author

I'm seeing the last bit of warnings at https://hydra.nixos.org/jobset/nixos/trunk-combined#tabs-errors

@worldofpeace
Copy link
Contributor Author

worldofpeace commented Feb 5, 2020

@worldofpeace worldofpeace modified the milestones: 20.03, 20.09 Feb 10, 2020
@worldofpeace
Copy link
Contributor Author

The last warnings are just in tests.
Need to open an issue about actually removing it for 20.09 after this.

@stale
Copy link

stale bot commented Aug 8, 2020

Hello, I'm a bot and I thank you in the name of the community for opening this issue.

To help our human contributors focus on the most-relevant reports, I check up on old issues to see if they're still relevant. This issue has had no activity for 180 days, and so I marked it as stale, but you can rest assured it will never be closed by a non-human.

The community would appreciate your effort in checking if the issue is still valid. If it isn't, please close it.

If the issue persists, and you'd like to remove the stale label, you simply need to leave a comment. Your comment can be as simple as "still important to me". If you'd like it to get more attention, you can ask for help by searching for maintainers and people that previously touched related code and @ mention them in a comment. You can use Git blame or GitHub's web interface on the relevant files to find them.

Lastly, you can always ask for help at our Discourse Forum or at #nixos' IRC channel.

@stale stale bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Aug 8, 2020
@xaverdh
Copy link
Contributor

xaverdh commented Sep 2, 2020

can this be closed now?

@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Sep 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.kind: bug Something is broken 2.status: work-in-progress This PR isn't done 3.skill: sprintable A larger issue which is split into distinct actionable tasks 3.skill: trivial This is trivial to complete (typically find-and-replace) 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS
Projects
None yet
Development

No branches or pull requests

3 participants