Gregory's Server@danderson that one is deliciously clever. I didn't see it when I looked at the diff despite having been primed to look for something evil.
|
6 comments
 @anymaw @dave_andersen Yeah, these feature checks usually work by compiling (and maybe running) a test program, to check that everything required is present. The original malicious commit that added this check explained that on some systems the header files for Landlock are present but Landlock doesn't actually work, so the configuration builds a test program to check if it actually works. And yes, any failure is interpreted as the feature being unavailable :/ @anymaw @dave_andersen And yes, different people. The joy of having a very common name :)  @danderson @anymaw @dave_andersen There was a Mrs. Smith in our church growing up. She got divorced and when she remarried became....Mrs Jones ;-) @pixelpusher220 @danderson @anymaw I probably should have taken my wife's name when we got married, but by then she and I both had extensive publication records under our original names.  @dave_andersen @pixelpusher220 @danderson @anymaw We always wanted to coax a bunch of Danish astronomer colleagues to write a joint paper. It could have been at least 6 Andersens, some even with the same initials... |
@dave_andersen @danderson
so how does it work? I guess CMake passes that chunk of C to the compiler, but then the build script assumes that if that compilation fails for any reason, including a syntax error, then the system doesn't actually support landlock?
and are you in fact two different people?