conda-forge / r-rcpparmadillo-feedstock

A conda-smithy repository for r-rcpparmadillo.
BSD 3-Clause "New" or "Revised" License
0 stars 6 forks source link

Adjust `llvm-openmp` dependency #48

Closed mfansler closed 1 year ago

mfansler commented 1 year ago

The last osx-* builds list two constraints on llvm-openmp:

- llvm-openmp >=14.0.6
- llvm-openmp >=15.0.4

but conda-forge-pinning is set at llvm-openmp=14. I'm hoping that moving the llvm-openmp dependency to be under host not run might resolve this.

Possibly related to: https://github.com/conda-forge/r-mlpack-feedstock/pull/5

Checklist

conda-forge-webservices[bot] commented 1 year ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

mfansler commented 1 year ago

@conda-forge-admin please rerender

mfansler commented 1 year ago

After looking at the rerendered output, I noticed that llvm-openmp was already pinned in the OSX build scripts. E.g.,

https://github.com/conda-forge/r-rcpparmadillo-feedstock/blob/456fa96e34782e261ed75437df11b02788fad65b/.ci_support/osx_64_r_base4.2.yaml#L21-L22

Hence, this PR currently doesn't do anything substantive. In fact, despite this pinning, we still see the test solve result in using llvm-openmp=15, e.g.,

Solved Test Environment Packages

``` _r-mutex: 1.0.1-anacondar_1 conda-forge bwidget: 1.9.14-h694c41f_1 conda-forge bzip2: 1.0.8-h0d85af4_4 conda-forge c-ares: 1.18.1-h0d85af4_0 conda-forge ca-certificates: 2022.12.7-h033912b_0 conda-forge cairo: 1.16.0-h904041c_1014 conda-forge cctools_osx-64: 973.0.1-hcc6d90d_11 conda-forge clang: 14.0.6-h694c41f_0 conda-forge clang-14: 14.0.6-default_h55ffa42_0 conda-forge clang_osx-64: 14.0.6-h3113cd8_4 conda-forge clangxx: 14.0.6-default_h55ffa42_0 conda-forge clangxx_osx-64: 14.0.6-h6f97653_4 conda-forge compiler-rt: 14.0.6-h613da45_0 conda-forge compiler-rt_osx-64: 14.0.6-hab78ec2_0 conda-forge curl: 7.87.0-h6df9250_0 conda-forge expat: 2.5.0-hf0c8a7f_0 conda-forge font-ttf-dejavu-sans-mono: 2.37-hab24e00_0 conda-forge font-ttf-inconsolata: 3.000-h77eed37_0 conda-forge font-ttf-source-code-pro: 2.038-h77eed37_0 conda-forge font-ttf-ubuntu: 0.83-hab24e00_0 conda-forge fontconfig: 2.14.1-h5bb23bf_0 conda-forge fonts-conda-ecosystem: 1-0 conda-forge fonts-conda-forge: 1-0 conda-forge freetype: 2.12.1-h3f81eb7_1 conda-forge fribidi: 1.0.10-hbcb3906_0 conda-forge gettext: 0.21.1-h8a4c099_0 conda-forge gfortran_impl_osx-64: 11.3.0-h1f927f5_27 conda-forge gfortran_osx-64: 11.3.0-h18f7dce_0 conda-forge gmp: 6.2.1-h2e338ed_0 conda-forge graphite2: 1.3.13-h2e338ed_1001 conda-forge gsl: 2.7-h93259b0_0 conda-forge harfbuzz: 6.0.0-h08f8713_0 conda-forge icu: 70.1-h96cf925_0 conda-forge isl: 0.25-hb486fe8_0 conda-forge jpeg: 9e-hac89ed1_2 conda-forge krb5: 1.20.1-h049b76e_0 conda-forge ld64_osx-64: 609-hfd63004_11 conda-forge lerc: 4.0.0-hb486fe8_0 conda-forge libblas: 3.9.0-16_osx64_openblas conda-forge libcblas: 3.9.0-16_osx64_openblas conda-forge libclang-cpp14: 14.0.6-default_h55ffa42_0 conda-forge libcurl: 7.87.0-h6df9250_0 conda-forge libcxx: 14.0.6-hccf4f1f_0 conda-forge libdeflate: 1.14-hb7f2c08_0 conda-forge libedit: 3.1.20191231-h0678c8f_2 conda-forge libev: 4.33-haf1e3a3_1 conda-forge libffi: 3.4.2-h0d85af4_5 conda-forge libgfortran: 5.0.0-11_3_0_h97931a8_27 conda-forge libgfortran-devel_osx-64: 11.3.0-h824d247_27 conda-forge libgfortran5: 11.3.0-h082f757_27 conda-forge libglib: 2.74.1-h4c723e1_1 conda-forge libiconv: 1.17-hac89ed1_0 conda-forge liblapack: 3.9.0-16_osx64_openblas conda-forge libllvm14: 14.0.6-h5b596cc_1 conda-forge libnghttp2: 1.51.0-he2ab024_0 conda-forge libopenblas: 0.3.21-openmp_h429af6e_3 conda-forge libpng: 1.6.39-ha978bb4_0 conda-forge libssh2: 1.10.0-h47af595_3 conda-forge libtiff: 4.5.0-h6268bbc_0 conda-forge libwebp-base: 1.2.4-h775f41a_0 conda-forge libxml2: 2.10.3-hb9e07b5_0 conda-forge libzlib: 1.2.13-hfd90126_4 conda-forge llvm-openmp: 15.0.7-h61d9ccf_0 conda-forge llvm-tools: 14.0.6-h5b596cc_1 conda-forge make: 4.3-h22f3db7_1 conda-forge mpc: 1.2.1-hbb51d92_0 conda-forge mpfr: 4.1.0-h0f52abe_1 conda-forge ncurses: 6.3-h96cf925_1 conda-forge openssl: 3.0.7-hfd90126_1 conda-forge pango: 1.50.12-hbd9bf65_1 conda-forge pcre2: 10.40-h1c4e4bc_0 conda-forge pixman: 0.40.0-hbcb3906_0 conda-forge r-base: 4.2.2-h841e2fe_3 conda-forge r-rcpp: 1.0.9-r42h49197e3_2 conda-forge r-rcpparmadillo: 0.11.4.2.1-r42hf5e6a41_1 local readline: 8.1.2-h3899abd_0 conda-forge sigtool: 0.1.3-h88f4db0_0 conda-forge tapi: 1100.0.11-h9ce4665_0 conda-forge tk: 8.6.12-h5dbffcc_0 conda-forge tktable: 2.10-h49f0cf7_3 conda-forge xz: 5.2.6-h775f41a_0 conda-forge zlib: 1.2.13-hfd90126_4 conda-forge zstd: 1.5.2-hfa58983_4 conda-forge ```

That is, something seems to not be working with llvm-openmp pinning.

mfansler commented 1 year ago

@conda-forge/help-c-cpp any ideas?

Pinning of llvm-openmp nominally appears to be working:

However, the test environments end up with llvm-openmp=15. I originally though this was an issue with host vs run requirements, but now I'm at a loss. The previous build ended up with dual entries in the final metadata:

- llvm-openmp >=14.0.6
- llvm-openmp >=15.0.4

which is what I was hoping to fix.

mfansler commented 1 year ago

@conda-forge-admin please rerender

github-actions[bot] commented 1 year ago

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you, but it looks like there was nothing to do.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/r-rcpparmadillo-feedstock/actions/runs/3916571532.