| 25 Dec 2024 |
 K900 ⚡️ | No, we should not be encouraging people to hardcode git revisions in random places | 18:39:47 |
 🐰 xiaoxiangmoe | How about this?
src = fetchFromGitHub {
owner = "konoui";
repo = "lipo";
tag = "v${version}";
rev = "4f0dadbf38ee4cf4cc38cbc232b7708fddf965bc";
hash = "sha256-lZgOoN+oibo2h6bw5KHXuiwQvQecTQiqu400sGfaMi0=";
}
| 18:41:27 |
| K900 ⚡️ | That is completely duplicate information | 18:41:58 |
| 🐰 xiaoxiangmoe | tag may change | 18:42:18 |
| 🐰 xiaoxiangmoe | rev is the only source of truth | 18:42:27 |
| K900 ⚡️ | Then you shouldn't provide the tag | 18:42:39 |
| 🐰 xiaoxiangmoe | But the tag is needed for version | 18:42:58 |
| 🐰 xiaoxiangmoe | nixpkg need version field | 18:43:06 |
| K900 ⚡️ | Look | 18:43:11 |
| K900 ⚡️ | We can't be making accommodations in nixpkgs for every stupid decision an upstream makes | 18:43:29 |
| 🐰 xiaoxiangmoe | Okay | 18:43:50 |
| 🐰 xiaoxiangmoe | Also, builtins.fetchGit { url= "https://github.com/konoui/lipo.git"; } can provide lastModified, lastModifiedDate, rev, shortRev attr | 18:47:52 |
| 🐰 xiaoxiangmoe | Should fetchFromGitHub also provide this? | 18:48:04 |
| 🐰 xiaoxiangmoe | nix-repl> builtins.fetchGit { url= "https://github.com/konoui/lipo.git"; }
copying '«git+https://github.com/konoui/lipo.git?exportIgnore=1&ref=refs/heads/main&rev=d405c7e62772120bf98ea326b26799bde8d5a128»«unkn{
lastModified = 1735092575;
lastModifiedDate = "20241225020935";
narHash = "sha256-T4V6VDjwMJ1/uopxbotrCgjNkG8lc3ecYKWRjqT3GnU=";
outPath = "/nix/store/na175bylllrcnn0yq8irfj9q7w3xhh2k-source";
rev = "d405c7e62772120bf98ea326b26799bde8d5a128";
revCount = 134;
shortRev = "d405c7e";
submodules = false;
}
| 18:48:28 |
 linj | In reply to @k900:0upti.me
Not guaranteed, technically I am curious why removing .git does not guarante reproducibility. Could you elaborate? | 18:52:23 |
| 🐰 xiaoxiangmoe | Also, I'm curious whether passthru.updateScript will work for unfree apps or not. I found passthru.tests will not run in nixpkgs ci for unfree apps. | 18:57:23 |
| 🐰 xiaoxiangmoe | * Also, I'm curious whether r-ryantm bot will run passthru.updateScript for unfree apps or not. I found passthru.tests will not run in nixpkgs ci for unfree apps. | 18:58:21 |
| K900 ⚡️ | In reply to@xiaoxiangmoe:matrix.org
Also, I'm curious whether r-ryantm bot will run passthru.updateScript for unfree apps or not. I found passthru.tests will not run in nixpkgs ci for unfree apps. It will | 19:04:28 |
| K900 ⚡️ | As long as those apps are listed on repology | 19:04:35 |
| K900 ⚡️ | In reply to@me:linj.tech
I am curious why removing .git does not guarante reproducibility. Could you elaborate? All kinds of possible filter weirdness | 19:04:45 |
 Perchun Pak | https://github.com/NixOS/nixpkgs/issues/8567 | 19:05:01 |
| Perchun Pak | you don't have to, just set it to an empty string or fetch it using an update script | 19:06:22 |
| linj | In reply to @perchun:matrix.org
https://github.com/NixOS/nixpkgs/issues/8567 I failed to find an answer for my question without clicking "load more" and there are too many comments. Could you provide a more specific link? | 19:07:19 |
| Perchun Pak | tldr is that hash sometimes changes without any reason, and it triggers a build failure | 19:08:07 |
| linj | In reply to @perchun:matrix.org
https://github.com/NixOS/nixpkgs/issues/8567 The last comment says removing .git is a valid use case. https://github.com/NixOS/nixpkgs/issues/8567#issuecomment-2398540893 | 19:08:56 |
| Perchun Pak | then ig it is indeed valid, but still I would avoid it if possible | 19:09:35 |
| Perchun Pak | here - it is possible | 19:09:46 |
| 26 Dec 2024 |
|  nevoyu left the room. | 01:50:48 |
| 27 Dec 2024 |
|  raitobezarius changed their display name from raitobezarius to raitobezarius (DECT: 3538 / EPVPN 2681). | 07:31:31 |
|  Dimitar set a profile picture. | 11:41:20 |
