User Details
- User Since
- Jan 7 2014, 10:13 PM (597 w, 1 d)
Tue, Jun 17
Tue, Jun 10
I think this makes sense, thanks. Proves how often I run cleanworld, which is almost never :)
Sat, May 31
Okay, let's change the default then. It would be nice if you can put a reminder in the commit message that says you have to turn on LLVM_ASSERTIONS whenever you want to report an issue with the toolchain itself.
Fri, May 30
The idea behind this being on in -CURRENT is to catch more errors. The same really holds for WITNESS and other assertions, but you could argue that those are more for the core of the system, while llvm-project is contributed software. Note that we caught many assertions since they are enabled by default, so they definitely have value.
Wed, May 28
Sun, May 25
Thu, May 22
Maybe a silly question, but when make unwraps these command lines and then prints them for the user, will the output become longer due extra spaces? Could be bad for my OCD... :)
May 18 2025
May 9 2025
May 8 2025
May 5 2025
May 2 2025
Are you planning on MFC'ing this? But LGTM for the rest.
Apr 30 2025
It's a bit unfortunate, but better to do the same thing as most other systems.
Apr 26 2025
Apr 25 2025
Add OptionalObsoleteFiles.inc entries.
Add workaround for powerpc "relocation R_PPC_GOT16 out of range" errors.
Apr 24 2025
I tried this out, and it solved my issue of etcupdate leaving around a /usr/src/lib/libc/nsswitch.conf file.
Apr 23 2025
Apr 22 2025
Abandoning, arc diff totally messed this up.
Split off the bsd.suffixes-extra.mk part to D49965.
Note this requires D49951 to be committed first, since otherwise tests/sys/fs/fusefs falls over due to it picking sbin/mount.c over tests/sys/fs/fusefs/mount..cc for targeting mount.pieo. This has to do with some internal ordering in make that I haven't been able to fully figure out.
LGTM, as long as there are include guards in mntopts.h :)
Another minor remark, in D49958 I added include guards to mntopts.h. Since the header is now actually installed, I think it would be proper to do so here too.
Apart from the nits, LGTM!
Apr 19 2025
Apr 18 2025
Mar 30 2025
Great, thanks.
Mar 29 2025
Yes, this is fine too. At some point both gcc and clang switched to gnu++17 by default, but it's better to be explicit.
Mar 28 2025
Kludge approved :)
Mar 26 2025
Mar 3 2025
Yeah, go for it. :)
Feb 26 2025
Feb 16 2025
Feb 13 2025
Feb 10 2025
Thanks, I will commit this.
Feb 3 2025
Jan 31 2025
Jan 27 2025
LGTM