conda-forge / ucx-split-feedstock

A conda-smithy repository for ucx-split.
BSD 3-Clause "New" or "Revised" License
0 stars 23 forks source link

Do not call ./autogen.sh and use release tarball #181

Closed SeyedMir closed 4 months ago

SeyedMir commented 5 months ago

./autogen.sh will incorrectly update the git commit hash that will be later on shown by ucx_info. Also, without ./sutogen, we can use the release tarballs instead of the raw source tarballs. Resovles https://github.com/conda-forge/ucx-split-feedstock/issues/180

Checklist

SeyedMir commented 5 months ago

@conda-forge-admin, please rerender

conda-forge-webservices[bot] commented 5 months 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.

github-actions[bot] commented 5 months 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/ucx-split-feedstock/actions/runs/8941271911.

jakirkham commented 5 months ago

@conda-forge-admin , please restart CI

jakirkham commented 4 months ago

@conda-forge-admin , please restart CI

jakirkham commented 4 months ago

Travis CI is stuck in queue for the past few hours (this is a known issue). Unfortunately am having issues with Travis CI, which I've reported to them already. Otherwise I would try restarting, which likely would fix the issue. For now will go ahead and merge (as other builds seem ok)

Am hoping this will help unlock cross-compiling ( https://github.com/conda-forge/ucx-split-feedstock/pull/136 ), which would allow us to use Azure for all builds. Thus avoiding issues with Travis here

jakirkham commented 4 months ago

Thanks Hessam! 🙏