In reply to @asymmetric:matrix.dapp.org.uk
i'll be using the user agent "asymmetric", in case you want to track usage

In reply to @hexa:lossy.network
better yet, the url to the project repo 🙂

In reply to @hexa:lossy.network
better yet, the url to the project repo 🙂

Is there a way how to disable r-ryantm to update a specific package?

I was trying to disable vmagent for being updated (as we build it out of VictoriaMetrics package via override).
I was trying to add # nixpkgs-update: no auto update being added as comment.
https://github.com/r-ryantm/nixpkgs/blob/a9e94032782cc6f89c5fabcf5d9433b45578cad1/pkgs/by-name/vm/vmagent/package.nix#L5

Thus it did still create a PR ( https://github.com/NixOS/nixpkgs/pull/331673 ). I would have assumed that I would just create a PR for VictoriaMetrics now, does somebody know what is wrong?

Is there a way how to disable r-ryantm to update a specific package if it's build out of another?

I was trying to disable vmagent for being updated (as we build it out of VictoriaMetrics package via override).
I was trying to add # nixpkgs-update: no auto update being added as comment.
https://github.com/r-ryantm/nixpkgs/blob/a9e94032782cc6f89c5fabcf5d9433b45578cad1/pkgs/by-name/vm/vmagent/package.nix#L5

Thus it did still create a PR ( https://github.com/NixOS/nixpkgs/pull/331673 ). I would have assumed that I would just create a PR for VictoriaMetrics now, does somebody know what is wrong?

In reply to @asymmetric:matrix.dapp.org.uk
but naming is so hard 🙈 nixpkgs-update-notify-bot? nun (nixpkgs-update-notifier)? 😅

In reply to @hexa:lossy.network
better yet, the url to the project repo 🙂

In reply to @shawn8901:matrix.org

Is there a way how to disable r-ryantm to update a specific package if it's build out of another?

I was trying to disable vmagent for being updated (as we build it out of VictoriaMetrics package via override).
I was trying to add # nixpkgs-update: no auto update being added as comment.
https://github.com/r-ryantm/nixpkgs/blob/a9e94032782cc6f89c5fabcf5d9433b45578cad1/pkgs/by-name/vm/vmagent/package.nix#L5

Thus it did still create a PR ( https://github.com/NixOS/nixpkgs/pull/331673 ). I would have assumed that I would just create a PR for VictoriaMetrics now, does somebody know what is wrong?

If you want the comment to work need to fix meta.position:

❯ nix eval -f. vmagent.meta.position
"/Users/zowoq/src/nixpkgs/pkgs/by-name/vi/victoriametrics/package.nix:87"

diff --git a/pkgs/by-name/vm/vmagent/package.nix b/pkgs/by-name/vm/vmagent/package.nix
index f31526026319..612144762d6f 100644
--- a/pkgs/by-name/vm/vmagent/package.nix
+++ b/pkgs/by-name/vm/vmagent/package.nix
@@ -3,7 +3,7 @@
 # This package is build out of the victoriametrics package.
 # so no separat update prs are needed for vmagent
 # nixpkgs-update: no auto update
-lib.addMetaAttrs { mainProgram = "vmagent"; } (
+lib.addMetaAttrs { mainProgram = "vmagent"; position = toString ./package.nix + ":1"; } (
   victoriametrics.override {
     withServer = false;
     withVictoriaLogs = false;

❯ nix eval -f. vmagent.meta.position
"/Users/zowoq/src/nixpkgs/pkgs/by-name/vm/vmagent/package.nix:1"

Could add it to the skiplist instead:
https://github.com/nix-community/nixpkgs-update/blob/main/src/Skiplist.hs

In reply to @zowoq:matrix.org

If you want the comment to work need to fix meta.position:

❯ nix eval -f. vmagent.meta.position
"/Users/zowoq/src/nixpkgs/pkgs/by-name/vi/victoriametrics/package.nix:87"

diff --git a/pkgs/by-name/vm/vmagent/package.nix b/pkgs/by-name/vm/vmagent/package.nix
index f31526026319..612144762d6f 100644
--- a/pkgs/by-name/vm/vmagent/package.nix
+++ b/pkgs/by-name/vm/vmagent/package.nix
@@ -3,7 +3,7 @@
 # This package is build out of the victoriametrics package.
 # so no separat update prs are needed for vmagent
 # nixpkgs-update: no auto update
-lib.addMetaAttrs { mainProgram = "vmagent"; } (
+lib.addMetaAttrs { mainProgram = "vmagent"; position = toString ./package.nix + ":1"; } (
   victoriametrics.override {
     withServer = false;
     withVictoriaLogs = false;

❯ nix eval -f. vmagent.meta.position
"/Users/zowoq/src/nixpkgs/pkgs/by-name/vm/vmagent/package.nix:1"

Could add it to the skiplist instead:
https://github.com/nix-community/nixpkgs-update/blob/main/src/Skiplist.hs

In reply to @zowoq:matrix.org

If you want the comment to work need to fix meta.position:

❯ nix eval -f. vmagent.meta.position
"/Users/zowoq/src/nixpkgs/pkgs/by-name/vi/victoriametrics/package.nix:87"

diff --git a/pkgs/by-name/vm/vmagent/package.nix b/pkgs/by-name/vm/vmagent/package.nix
index f31526026319..612144762d6f 100644
--- a/pkgs/by-name/vm/vmagent/package.nix
+++ b/pkgs/by-name/vm/vmagent/package.nix
@@ -3,7 +3,7 @@
 # This package is build out of the victoriametrics package.
 # so no separat update prs are needed for vmagent
 # nixpkgs-update: no auto update
-lib.addMetaAttrs { mainProgram = "vmagent"; } (
+lib.addMetaAttrs { mainProgram = "vmagent"; position = toString ./package.nix + ":1"; } (
   victoriametrics.override {
     withServer = false;
     withVictoriaLogs = false;

❯ nix eval -f. vmagent.meta.position
"/Users/zowoq/src/nixpkgs/pkgs/by-name/vm/vmagent/package.nix:1"

Could add it to the skiplist instead:
https://github.com/nix-community/nixpkgs-update/blob/main/src/Skiplist.hs

it seems that setting the position fixes my overall issue. Tho not being on the ignored list but complaining before that the old version is not defined in the master derivation (as those are defined on the overriden ones).

I suspect that nixpkgs-update might should check for ignores first, and then for sanity [as it would at least in this case not result in a faulty error log].

Tho it still fine for my result.
ill keep the ignore comment still, so that the initial intention is clear why meta.position is added.

it seems that setting the position fixes my overall issue. Tho not being on the ignored list but complaining before that the old version is not defined in the master derivation (as those are defined on the overriden ones).

I suspect that nixpkgs-update might should check for ignores first, and then for sanity [as it would at least in this case not result in a faulty error log].

Tho it still fine for my result.
ill keep the ignore comment still, so that the initial intention is clear why meta.position is added.

Or do you have other suggestions for that? 🙂