conda-forge.github.io icon indicating copy to clipboard operation
conda-forge.github.io copied to clipboard

Clean up feedstocks still using `conda-forge-ci-setup=3`

Open h-vetinari opened this issue 1 year ago • 6 comments

I noticed this in pytorch, and according to a search for this, we have around 10 feedstocks that are currently affected.

The problem with this is that using such an old ci-setup forces a very old conda-build, which is - pretty much by sheer luck - just barely new enough to correctly handle {{ stdlib("c") }}, which we need to avoid creating packages with incorrect metadata on osx/linux.

There are many other relevant changes that should make us retry to make these feedstocks use up-to-date build tools (some may still need to keep pinning lief due to various issues, but aside from a performance regression on osx, lief 0.14 should have fixed most things).

h-vetinari avatar Sep 01 '24 04:09 h-vetinari

@h-vetinari Okay, so all we need to do is to remove the remote_ci_setup entries and check to see if all goes well?

We are currently preparing a new release of ifcopenshell in https://github.com/conda-forge/ifcopenshell-feedstock/pull/54. So I will try to make the changes there to see if that goes well.

Here's what we have. I remember pinning py-lief was necessary way back when due to the osx builds ran out of time. Changing the py-lief version at the time fixed it. Hopefully this is gone now.

Here's the remote_ci_setup in ifcopenshell

remote_ci_setup:
- conda-forge-ci-setup=3
- py-lief>=0.12.3

Krande avatar Sep 01 '24 14:09 Krande

pcl, orocos-kdl, libignition-common all done

Tobias-Fischer avatar Sep 02 '24 06:09 Tobias-Fischer

@h-vetinari ifcopenshell is updated with the outdated configs removed.

Krande avatar Sep 02 '24 14:09 Krande

tlsh is fixed (well, building)... I could probably use an assist on https://github.com/conda-forge/atheris-feedstock/pull/16 (my c++ chops are non-existent):

src/native/counters.cc:26:6: error: variable or field '__sanitizer_cov_8bit_counters_init' declared void
   26 | void __sanitizer_cov_8bit_counters_init(uint8_t* start, uint8_t* stop);
      |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/native/counters.cc:26:41: error: 'uint8_t' was not declared in this scope
   26 | void __sanitizer_cov_8bit_counters_init(uint8_t* start, uint8_t* stop);
      |                                         ^~~~~~~
src/native/counters.cc:24:1: note: 'uint8_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
   23 | #include "macros.h"
  +++ |+#include <cstdint>

bollwyvl avatar Sep 02 '24 21:09 bollwyvl

@bollwyvl: this patch should fix it: https://patch-diff.githubusercontent.com/raw/google/atheris/pull/91.patch

Tobias-Fischer avatar Sep 02 '24 21:09 Tobias-Fischer

Thanks! Still seeing some OSX issues on atheris (but not missing libfuzzer, which was happening previously)

bollwyvl avatar Sep 02 '24 22:09 bollwyvl

The last remaining usage was fixed in https://github.com/conda-forge/atheris-feedstock/pull/19, so I think we can close this now. Thanks everyone!

h-vetinari avatar Dec 12 '25 21:12 h-vetinari