conda-forge / ctng-compilers-feedstock

A conda-smithy repository for ctng-compilers.
BSD 3-Clause "New" or "Revised" License
12 stars 28 forks source link

Backport fix for upstream issue 111413 to GCC13 #117

Closed traversaro closed 1 year ago

traversaro commented 1 year ago

Backport upstream fix https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111413 from GCC14 to GCC13. The change by itself is huge, but avoiding to to indentation changes it can be summarized to single line change, that we used for thet backport to reduce the risk of patch conflicts. See https://github.com/gcc-mirror/gcc/commit/15345980633c502f0486a2e40e96224f49134130?w=1 for a proof of this.

Fix https://github.com/conda-forge/ctng-compilers-feedstock/issues/114 Fix https://github.com/conda-forge/casadi-feedstock/issues/91

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.

traversaro commented 1 year ago

As far as I understand, rerendering is not the way to go for this feedstock.

traversaro commented 1 year ago

I build it locally, and with the following gomp package: libgomp-13.2.0-h807b86a_2.conda.zip the problem discussed in https://github.com/conda-forge/ctng-compilers-feedstock/issues/114 is solved.

traversaro commented 1 year ago

@conda-forge/ctng-compiler-activation the PR has been tested locally and it works fine, and CI is happy. The PR is ready for review, thanks!