Ian McInerney

Results 235 comments of Ian McInerney

Oh, thanks for catching that. Those are leftovers and should have been removed/adjusted (and actually, it looks like I can just delete the entire `osqp_api_utils.h` header now, since everything is...

> I'm getting a warning about the binaries using avx2 instructions on the w64 builds. I tried adding -mno-avx2 to CFLAGS but that didn't solve the problem, I'm not sure...

I see in the artifact that it includes `libpostal.a`, which is the static library. We generally don't like to include static libraries in the JLL packages (unless absolutely necessary) because...

It is building a static library because of this line: https://github.com/lballabio/QuantLib/blob/master/CMakeLists.txt#L151. If you add `-DBUILD_SHARED_LIBS=ON` to the CMake command, it will then build the shared library on Windows as well....

I don't think there is a use for having the testsuite built actually, is there? Boost test is basically a unit testing framework, so it isn't user-exposed, and since we...

Does that only compress the debug sections? I think a lot of the binaries here are built without debug information (e.g. the default recommended CMake configuration is `Release`, same with...

> edit: This release now points to the release candidate 1 release (as it requires no patches) to build. Is it ok to set the version number to v1.2.0 already...

> Ipopt_jll is already linked with LBT so we could remove the folder for the artifact IpoptMKL_jll. Has anyone tested running the LBT build with MKL selected? We should make...

> But we compile all COIN-OR packages with LP64 BLAS / LAPACK and not ILP64 BLAS / LAPACK so we can't have this problem. > > I also checked with...

The `libusb_fill_bulk_transfer` function is not actually contained inside the compiled library, it is shipped as an inline function inside the `libusb.h` header that the library consumers have to include. That...