| 6 Sep 2023 |
Artturin | 1285 vs 495 damn | 18:29:13 |
infinisil | Not sure why we need that convention fwiw | 18:29:42 |
infinisil | I guess it's more consistent, easier to find it, less chance of making a mistake in the name | 18:30:15 |
infinisil | Fair enough | 18:30:18 |
K900 | What convention? | 18:30:20 |
infinisil | In reply to @infinisil:matrix.org
The pname and the version attribute must not contain uppercase letters — e.g., "mplayer" instead of "MPlayer"`.
Huh TIL, from https://github.com/NixOS/nixpkgs/tree/master/pkgs#package-naming
K900 ⚡️: | 18:30:58 |
K900 | Oh yeah I'm not sure about pname | 18:31:20 |
K900 | But attribute names should really be lower case | 18:31:24 |
infinisil | Maybe we should have pname match the attribute name too though | 18:32:05 |
K900 | I don't really care on that one | 18:35:53 |
davidak | that makes it much less confusing | 18:56:14 |
| 7 Sep 2023 |
DavHau | In reply to @infinisil:matrix.org Maybe we should have pname match the attribute name too though Yes please. Establishing that pattern is also important for downstream projects. Associating the log of a failed build with the correct package file can often be a nightmare, especially for team members with little nix experience. | 20:55:43 |
DavHau | Its also important if we want to better integrate package updates. We can attach an update script to a package via passthru, but that script needs to know the location of the package definition. Hardcoding the location breaks portability. A mechanism to compute the package location from the attribute path is needed. | 23:10:24 |
DavHau | * Its also important if we want to better integrate package updates. We can attach an update script to a package via passthru, but that script needs to know the location of the package definition. Hardcoding the location breaks portability. A mechanism to compute the package location from the package name is benefitial. | 23:12:44 |
| 8 Sep 2023 |
@piegames:matrix.org | I think currently update scripts are executed in the package's folder, and finding the Nixpkgs root can be done with a simple git command | 06:04:25 |