robinbrantley / floxmeta Goto Github PK

In our modules where we reference stability.channel.attrpath we are instead using pname. Examples

https://github.com/flox/floxEM/blob/main/floxEM/environments.py#L974

https://github.com/flox/floxEM/blob/main/floxEM/environments.py#L1012

The problem is that we want to use this pname as an identifier to perform operations like pausing upgrades. However, the pname value is not consistent and changes over time

examples

https://github.com/NixOS/nixpkgs/blob/81b0104d8512c7a4dfda701aaa613e4db064752a/pkgs/applications/graphics/hello-wayland/default.nix#L6C34-L6C34

https://github.com/NixOS/nixpkgs/blob/50ea1ba37d34de46c53f909c28f2da712d1bfe94/pkgs/applications/graphics/hello-wayland/default.nix#L7

This is causing the errors we are seeing in this issue

https://app.zenhub.com/workspaces/flox-engineering-6380b504813d442db8cea674/issues/gh/flox/floxem/164

To fix this issue, it is proposed to replace pname with attrPath data that is returned from flox list --json (extracting the the name from the system.stability tuple)

And where we are parsing and using catalog data, we propose to use the attPath name there, again extracting it from the list of items, or from the nested values in the json object.