| 9 Jul 2021 |
|  tomcur changed their display name from Thom to tomcur. | 14:31:06 |
|  vika (she/her) 🏳️⚧️ set a profile picture. | 16:39:07 |
| 11 Jul 2021 |
|  rosariopulella joined the room. | 15:45:57 |
| 12 Jul 2021 |
|  softinio joined the room. | 06:06:35 |
|  Alex Zero joined the room. | 15:00:31 |
| 14 Jul 2021 |
|  Kočyš Mikajlo joined the room. | 08:32:40 |
|  paultrial joined the room. | 17:36:01 |
| 15 Jul 2021 |
 Las | Does anyone know if there's a way to verify commits in inputs? | 08:56:00 |
 matthewcroughan - nix.zone | Can somebody tell me if there's a way to do this better, or if we need to add it to the flake schema? | 22:04:03 |
| matthewcroughan - nix.zone | I need to use git describe on some sources. | 22:04:13 |
| matthewcroughan - nix.zone | The flake lock outputs lots of things, but I can't actually use git describe on a given source, because in order to do that I'd need to download the git index, which is highly impure! | 22:04:46 |
| matthewcroughan - nix.zone | * The flake lock describes lots of things, but I can't actually use git describe on a given source, because in order to do that I'd need to download the git index, which is highly impure! | 22:04:54 |
| matthewcroughan - nix.zone | https://discourse.nixos.org/t/git-describe-like-attributes-for-flakes/10805 | 22:27:03 |
| matthewcroughan - nix.zone | Looks like someone has asked already | 22:27:07 |
 mvnetbiz | Seems not possible unless the description itself is committed into the repo beforehand | 22:47:07 |
| mvnetbiz | I think multiple git commit histories could produce the same fixed-output with the way its currently implemented, so that can't be done purely | 22:50:41 |
| matthewcroughan - nix.zone | Well, it's an interesting case. | 22:50:57 |
| matthewcroughan - nix.zone | Maybe I can talk about it. | 22:51:05 |
| mvnetbiz | I didin't thikn about it, I was just thinking I could git describe ${self} in the outputs, but it doesn't have the git data itself, I think it was you were saying right? | 22:52:02 |
| mvnetbiz | * I didin't thikn about it, I was just thinking I could git describe ${self} in an output derivation, but it doesn't have the git data itself, I think that is what you were saying right? | 22:52:46 |
| matthewcroughan - nix.zone | https://github.com/dolphin-emu/dolphin uses CMake to find git, which in turn sets DOLPHIN_WC_DESCRIBE. This is used at runtime to determine compatibility between clients.
https://github.com/NixOS/nixpkgs/blob/nixos-21.05/pkgs/misc/emulators/dolphin-emu/master.nix#L51
The nixpkg currently requires the maintainer to find this information out by hand and then bind it to the version value on L24.
| 22:53:21 |
| matthewcroughan - nix.zone | In order to use git describe you need the .git index. Which is impure. | 22:54:00 |
| matthewcroughan - nix.zone | * In order to use git describe you need the .git index. Which is impure, because if someone adds a branch upstream, then you've mutated the .git index state. | 22:54:12 |
| matthewcroughan - nix.zone | so using leaveDotGit = true; in a package just to do this is a bad idea, as the sha256 will always be mutating. | 22:55:23 |
| matthewcroughan - nix.zone | * so using leaveDotGit = true; in a package just to do this is a bad idea, as the sha256 will always be changing. | 22:55:28 |
| mvnetbiz | how is that dolphinemu case different than other packages where the version is specified? just because you can go look for a tag easily without the local repo, vs cloning it to do describe? | 22:56:23 |
| mvnetbiz | Seems like a dolphin problem, couldn't it be a problem for the runtime behavior to change based on what is bsically a side effect of the particular VCS in use? | 22:58:33 |
| matthewcroughan - nix.zone | In reply to @mvtva:matrix.org
how is that dolphinemu case different than other packages where the version is specified? just because you can go look for a tag easily without the local repo, vs cloning it to do describe? It's supposed to happen as part of the build. | 22:58:45 |
| matthewcroughan - nix.zone | The cmake build is supposed to use git describe and use it in order to set this string. | 22:59:02 |
| matthewcroughan - nix.zone | I've honestly suggested this is a problem to Dolphin, but I also believe it would be possible to put some data in the flake schema that allows this same string to be resolved without the need for Dolphin to change. | 22:59:50 |
