| 12 Nov 2024 |
 emily | which makes me wonder "are we absolutely sure it was tested that the fix actually fixed it?" | 13:49:51 |
| emily | but – Nixpkgs compilers are weird. it's also possible we're messing it up somehow and it's our fault | 13:50:02 |
 nbp | have the changes stick in LLVM, or did they got backed out of LLVM? | 13:56:14 |
 hexa | I can see the changes are still applied in 19.1.3 | 14:07:51 |
| emily | I didn't see any indication of revert | 14:12:07 |
| emily | the Mozilla bug does not directly link the LLVM change, but the circumstances, error message, and workaround are an exact match | 14:12:23 |
| emily | and it seems like it should directly fix the build error I saw | 14:12:33 |
| emily | so I'm wondering if upstream LLVM fix is broken and Mozilla didn't fully test it (understandable since they haven't yet upgraded their actual builders to LLVM 19 yet AFAIK), or if somehow we are breaking things with our weirdness | 14:12:57 |
| emily | it is no longer urgent since I reapplied the workaround on our LLVM 19 branch, but it is concerning | 14:13:11 |
| nbp | Mozilla maintains its own version of LLVM, maybe there is another fix that went in that has not been upstreamed? | 14:21:20 |
| nbp | * Mozilla maintains its pinned version of LLVM, maybe there is another fix that went in that has not been upstreamed? | 14:21:38 |
| emily | they said "Backed out as requested by owner because the upstream issue was fixed." | 14:30:41 |
| emily | and AFAIK the Mozilla builders are not yet on LLVM 19 (https://bugzilla.mozilla.org/show_bug.cgi?id=1925199) | 14:30:50 |
| emily | and the timing of the backout was very close to the LLVM PR I linked | 14:30:59 |
| emily | so I would be surprised were that the case | 14:31:03 |
| nbp | in this change, I see https://phabricator.services.mozilla.com/differential/changeset/?ref=8497643 , would that help? | 14:46:06 |
| emily | that's not relevant to us | 14:47:46 |
| emily | we do not use that build infra | 14:47:50 |
| nbp | would using the same revision help in identifying the issue? | 14:49:18 |
| emily | technically we went from 19.1.1 to 19.1.3, but I think I tested 19.1.2 between that too :) | 15:02:05 |
| emily | we do quite a bit of patching to LLVM so it's unlikely we'll exactly reproduce their build environment | 15:02:16 |
| emily | but, the patch was already in 19.1.1 | 15:02:23 |
| emily | it would be surprising if there was a follow-up in 19.1.2 that I completely failed to find that was then regressed in 19.1.3 | 15:02:34 |
| emily | not impossible, but surprising | 15:02:41 |
| emily | pretty sure the revert of the workaround predated 19.1.2 anyway | 15:03:06 |
| hexa | https://github.com/NixOS/nixpkgs/pull/355444 | 17:22:49 |
| 13 Nov 2024 |
|  Inayet joined the room. | 22:15:18 |
| 16 Nov 2024 |
| hexa | https://mozillafoundation.tfaforms.net/101 | 08:56:29 |
 K900 ⚡️ | 
Download {8F21A7B1-8EDB-4404-A8BB-C2F27F3E5097}.png | 08:57:46 |
| K900 ⚡️ | I love how "build a fucking browser" isn't even an option | 08:57:57 |
