add osx_arm64 support - Githubissues

conda-forge / occt-feedstock

A conda-smithy repository for occt.

BSD 3-Clause "New" or "Revised" License

4 stars 21 forks source link

Closed matthiasdiener closed 3 years ago

matthiasdiener commented 3 years ago

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase code>@<space/conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

conda-forge-linter commented 3 years 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.

I do have some suggestions for making it better though...

For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

conda-forge-linter commented 3 years 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.

looooo commented 3 years ago

@isuruf how to distinguish lgpl2-only and lgpl2-or-later? Need this for some other recipes too.

matthiasdiener commented 3 years ago

@conda-forge-admin rerender

matthiasdiener commented 3 years ago

This is ready @looooo @conda-forge/occt

isuruf commented 3 years ago

how to distinguish lgpl2-only and lgpl2-or-later?

If the package doesn't explicitly mention that the lciense is lgpl2-or-later, then go with lgpl2-only.

looooo commented 3 years ago

Thanks @matthiasdiener

looooo commented 3 years ago

The master branch has travis osx enabled which fails. So we need travis/osx? @matthiasdiener

matthiasdiener commented 3 years ago

The master branch has travis osx enabled which fails. So we need travis/osx? @matthiasdiener

I'm not sure why this is happening, maybe @isuruf knows more.