Skip to content

Build for all platforms#57

Closed
matrss wants to merge 4 commits intoconda-forge:mainfrom
matrss:patch-2
Closed

Build for all platforms#57
matrss wants to merge 4 commits intoconda-forge:mainfrom
matrss:patch-2

Conversation

@matrss
Copy link
Copy Markdown

@matrss matrss commented Jun 10, 2025

Checklist

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

@matrss
Copy link
Copy Markdown
Author

matrss commented Jun 10, 2025

@conda-forge-admin, please rerender

@conda-forge-admin
Copy link
Copy Markdown
Contributor

conda-forge-admin commented Jun 10, 2025

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.

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

For recipe/meta.yaml:

  • ℹ️ No valid build backend found for Python recipe for package annexremote using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.
  • ℹ️ PyPI default URL is now pypi.org, and not pypi.io. You may want to update the default source url.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/15570813092. Examine the logs at this URL for more detail.

@yarikoptic
Copy link
Copy Markdown
Contributor

FWIW, I added running tests on osx and windows for upstream's github:

and it seems was all green, so indeed might be worth ensuring we have for all OSes here.

@yarikoptic
Copy link
Copy Markdown
Contributor

@conda-forge-admin, please rerender

@conda-forge-admin
Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please try to merge or rebase with the base branch to resolve this conflict.

Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

@yarikoptic
Copy link
Copy Markdown
Contributor

uff, we forgot about this one . @MattRs do you have juice to finish it up. I am afraid that is what causes datalad issues as well leading to attempts like

@conda-forge-admin, please rerender

@yarikoptic
Copy link
Copy Markdown
Contributor

still conflicts after rerendering??? that's odd... may be worth just to start from beginning for the PR -- I really do not get why this simple python package causes issues... 'no linux' was added in prehistoric times in commit 2042459 "py27 compatibility (#6)"... I will resubmit PR from clean state.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants