xgboost 0.90 (new formula)

[ankane]

• Have you followed the guidelines for contributing?
• Have you checked that there aren't other open pull requests for the same formula update/change?
• Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
• Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
• Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

---

[SMillerDev]

Please don't specify the amount of jobs for make. That might cause issues on other systems.

[ankane]

My bad, updated

[javian]

The docs say it requires gcc to enable multi-threading - do you think that would be necessary for this Formula?

[ankane]

Yeah, that'd be ideal. Just pushed an update.

[ankane]

It looks like brew linkage --test xgboost is failing for each build. However, I'm not sure how to reproduce it locally. Are you familiar with this error?

[ankane]

Sorry the noise. It was due to having gcc@8 marked as build when it's libgomp is needed during runtime. The formula now supports multi-threading.

[fxcoudert]

Sorry to disagree with @javian there, but unless multithreading is essential to the app's behaviour, we generally avoid it. GCC is a heavy dependency, so we want to avoid it if at all possible, and compile with the system compiler.

Using libomp maybe help provide OpenMP support, if that's what you need.

And in any case, compiling with an older version of GCC is a definite “no way”.

[ankane]

Hey @fxcoudert, thanks for the feedback on this as well. It sounds like @hcho3 has it working with libomp but it isn't merged yet, so will see what the maintainers want to do: dmlc/xgboost#4477

[ankane]

Hey @fxcoudert, I updated the formula to work with Apple Clang (disabled OpenMP). Also switched to make install (based on your feedback from the lightgbm formula). It looks like the build failed due to a credentials error.

[SMillerDev]

@BrewTestBot test this please

[SMillerDev]

Thanks @ankane! Without contributions like yours it'd be impossible to keep homebrew going with the high standards that users have come to expect from the project. You can feel good knowing that you've made the world a tiny bit better for homebrew users around the world! 👍 🎉

[ankane]

Thanks @SMillerDev and @fxcoudert 👍