| 26 Oct 2021 |
das_j | yeah boy | 20:40:48 |
das_j | it might be this: https://lists.gnu.org/archive/html/coreutils/2021-10/msg00021.html | 20:40:50 |
das_j | And here's our go tools failing: https://github.com/golang/go/issues/48636 | 20:43:51 |
Ryan Burns | Interesting that someone else is seeing issues with bootstrapped go. Seems like you need a really specific sequence of events for this issue to manifest | 20:44:21 |
das_j | And somehow also something that happens on Darwin | 20:45:40 |
Ryan Burns | Ok yep bisected to the same commit
cp: use SEEK_DATA/SEEK_HOLE if available | 21:07:23 |
Ryan Burns | Also note this coreutils commit after the 9.0 release: https://git.savannah.gnu.org/cgit/coreutils.git/commit/?id=61c81ffaacb0194dec31297bc1aa51be72315858 | 21:08:18 |
Ryan Burns | Run the SEEK_DATA check on the
1TiB empty file to exclude both FreeBSD 9.1 which takes 35s,
and ZFS which requires a delay of about 5s between file creation
and use of SEEK_DATA to correctly determine it's empty (return ENXIO).
| 21:09:08 |
Ryan Burns | If you have to modify the unit test, isn't that a sign that something broke?? | 21:09:43 |
das_j | In reply to @r-burns:matrix.org If you have to modify the unit test, isn't that a sign that something broke?? Nono of course not :D | 21:09:55 |
das_j | we could manually unset SEEK_HOLE, maybe that would help? | 21:11:37 |
das_j | For reference, this is the commit: https://github.com/coreutils/coreutils/commit/a6eaee501f6ec0c152abe88640203a64c390993e | 21:12:12 |
das_j | Redacted or Malformed Event | 21:20:39 |
das_j | I'll open a bug report on coreutils-bugs | 21:21:21 |
das_j |  Download image.png | 21:46:13 |
das_j | Is there anything anyone wants to add/change here? | 21:46:19 |
Ryan Burns | Are we sure that peertube is the same problem? I wouldn't want to confuse them if it is not actually related | 21:47:07 |
Vladimír Čunát | Is there evidence that peertube is broken by the same coreutils commit? | 21:47:15 |
das_j | Good point | 21:47:23 |
das_j | And alacritty was Darwin, that might also be caused by another commit | 21:48:04 |
Vladimír Čunát | If the commit can be reverted on release without conflicts (after stripping NEWS), it shouldn't be that hard to verify both. | 21:49:24 |
| moritz.hedtke joined the room. | 21:49:24 |
das_j | I can try on Darwin | 21:49:46 |
Vladimír Čunát | (without messing with coreutils dependencies) | 21:49:50 |
Vladimír Čunát | I can check peertube tomorrow on the same Linux system; seems easy if it applies. | 21:51:19 |
Vladimír Čunát | Oh and I see a "coretuils" typo in the e-mail :-) | 21:52:17 |
moritz.hedtke | Is there already a diff then I could also test peertube with that commit reverted. Considering that I could build with (likely just) another -frandom-seed I don't believe that's the issue but I could still verify. | 21:54:09 |
moritz.hedtke | I can also create it myself doesn't seem that hard | 21:54:32 |
moritz.hedtke | Ohh I think we can't really verify whether its the revert then or a compiler bug as this also changes the -frandom-seed | 22:01:57 |
das_j | oh oof, right | 22:02:20 |