Open jakirkham opened 2 years ago
cc @beckermr @isuruf
gcc_linux-aarch64
and gcc_linux-64
are not meant to be installed at the same time. Maybe r-base
should depend only on the gcc_impl_
packages
Naive question, should r-base
have a run
time dependency on compilers at all? Or should that be up to others to add? Or perhaps be part of a split package (r-build
or similar?)?
Idk the full history of the compiler dependency of r-base
, but seems they were added in PR ( https://github.com/conda-forge/r-base-feedstock/pull/61 ).
Interestingly this isn't the first time there have been issues with r-base
's handling of compilers and downstream dependencies ( https://github.com/conda-forge/r-base-feedstock/pull/111 ).
cc @conda-forge/r-base (in case others have thoughts on moving to gcc_impl_*
or this issue generally)
This came up in the conda-forge meeting today.
Basically R users expect a compiler to be available at install time and R expects it to be the same one it was built against. So this is why it is requirement. Also packages installed there are installed from source so need to be compiled (there is not a prebuilt binary format).
That said, there isn't a need to have the compiler activated, which switching to the gcc_impl_*
packages should fix.
Another item that came up is whether r-base
is the right package to include these in or whether there should be a more minimal package (like r-base-base
or similar :) or a different split package to have it in (like r-compilers
).
Of these options sounds like we will go forward with gcc_impl_*
. We could discuss a different split package structure in the future
@jakirkham Any updates on this one? Let me know if there's anything I can help with.
@hcho3 the comment above outlines the path forward. Have not had time to pick this up. So if this is something you are interested in and have time to pursue, feel free to give that a go. Though no worries if not
Submitted PR ( https://github.com/conda-forge/r-base-feedstock/pull/213 ) based on the recommendations above
Can we close this issue now that https://github.com/conda-forge/r-base-feedstock/pull/213 is merged?
We can. It was actually suppose to close when that PR was merged. Not sure why it didn't.
That said, it looks like PR ( https://github.com/conda-forge/xgboost-feedstock/pull/88 ) is still failing. So we might want to keep this open given it looks like there may be more work to do here.
Solution to issue cannot be found in the documentation.
Issue
It appears that flags are being set by both
x86_64
&aarch64
Linux compiler variants, which is leading to issues buildingxgboost
R packages. More details in comment ( https://github.com/conda-forge/xgboost-feedstock/pull/88#issuecomment-1119533913 ). Appears related to PR ( https://github.com/conda-forge/ctng-compiler-activation-feedstock/pull/67 ).Here's the CI build and the attached CI log for posterity.
cc @hcho3 @xhochy
Installed packages
Environment info