conda-forge / cmdstan-feedstock

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

Try clang post general availability #53

Open WardBrian opened 4 months ago

WardBrian commented 4 months ago

Checklist

31 again

WardBrian commented 4 months ago

@conda-forge-admin, please rerender

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

WardBrian commented 4 months ago

Getting hung up on

lld-link: error: could not open 'D:bldcmdstan_1719500406710_build_env/lib/clang/18/lib/windows/clang_rt.builtins-x86_64.lib': no such file or directory
WardBrian commented 4 months ago

@conda-forge-admin, please rerender

github-actions[bot] commented 4 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/cmdstan-feedstock/actions/runs/9715147705.

WardBrian commented 4 months ago

@conda-forge-admin, please rerender

WardBrian commented 4 months ago

@conda-forge-admin, please rerender

WardBrian commented 4 months ago

@conda-forge-admin, please rerender

WardBrian commented 4 months ago

I still want to try this on a local windows machine before merging

WardBrian commented 4 months ago

I suppose I should describe the contributions/reasons here:

  1. This allows us to support C++17, which Stan will require starting in the next version
  2. Building against the Microsoft ABI allows us to avoid building TBB
  3. The Microsoft ABI should have better performance when multithreading is enabled (this is part of what I need to confirm during testing)
conda-forge-webservices[bot] commented 2 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/meta.yaml) and found it was in an excellent condition.

WardBrian commented 2 months ago

@conda-forge-admin, please rerender

WardBrian commented 2 months ago

@conda-forge-admin, please rerender