meta.updateWalker: remove because it is not used

[Stunkymonkey]

Motivation for this change

#103997 removing the documentation about meta.updateWalker

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 22.05 Release Notes (or backporting 21.11 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
  • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
• Fits CONTRIBUTING.md.

[Stunkymonkey]

can someone help me in how to better document this? is running nixos/doc/manual/md-to-db.sh needed?

[ryantm]

Shouldn't this be in the same PR that removes update-walker.sh? (Assuming we actually do want to remove it, I don't know about it.)

is running nixos/doc/manual/md-to-db.sh needed?

No, that's only needed if you change the NixOS docs, not the nixpkgs ones like you have here.

[Stunkymonkey]

sure i can include the removal into a single PR. I thought more of splitting it up into multiple PRs

[Stunkymonkey]

i just notice, that all additional changes i did are not very helpful for reviewing the PR. Should I remove them?

[Stunkymonkey]

i removed all the changes, that were not necessary

[SuperSandro2000]

Can you also drop https://github.com/NixOS/nixpkgs/blob/master/pkgs/stdenv/generic/check-meta.nix#L239 and https://github.com/NixOS/nixpkgs/blob/master/pkgs/stdenv/generic/check-meta.nix#L247 ?

[Stunkymonkey]

is now removed in the last commit

[zowoq]

This seems to have missed a couple of files?

nixpkgs/pkgs/development/tools/misc/luarocks/default.upstream

Line 1 in bcd452a

url http://luarocks.org/releases/

nixpkgs/pkgs/servers/nosql/apache-jena/fuseki-binary.upstream

Line 1 in 30d4848

name apache-jena-fuseki

---

This is still used in one place, I'd suggest moving into the the same directory.

nixpkgs/pkgs/build-support/upstream-updater/urls-from-page.sh

Line 1 in bcd452a

#! /bin/sh

nixpkgs/pkgs/development/lisp-modules/from-quicklisp/barebones-quicklisp-expression.sh

Line 54 in 5a17134

version="$("$(dirname "$0")/../../../build-support/upstream-updater/urls-from-page.sh" "$url/releases/" | grep /tag/ | head -n 1 | xargs -l1 basename)"

---

Can also remove these from .editorconfig:

nixpkgs/.editorconfig

Lines 70 to 72 in 3540d00

	[pkgs/build-support/upstream-updater/**]
	indent_style = unset
	trim_trailing_whitespace = unset

[SuperSandro2000]

@Stunkymonkey can you please take a look at the above comment? I currently miss the time to do that.