Skip to content

Migrate vcstool to use vcs2l shim#18

Merged
traversaro merged 5 commits intoconda-forge:mainfrom
traversaro:updatetovcs2lstrikesback
Mar 27, 2026
Merged

Migrate vcstool to use vcs2l shim#18
traversaro merged 5 commits intoconda-forge:mainfrom
traversaro:updatetovcs2lstrikesback

Conversation

@traversaro
Copy link
Copy Markdown
Contributor

@traversaro traversaro commented Mar 27, 2026

The old vcstool is not compatible with recent setuptools. To deal with that, in vcs2l a vcstool shim was added in ros-infrastructure/vcs2l#94, and it provides the same python3-vcstool package for apt debian metadata.

As in conda we do not have the Provides3: python3-vcstool and Replaces3: python3-vcstool metadata capability, to provide the same behaviour we just transform vcstool >= 1.1.7 in being a simple noarch: generic shim that depends on the vcs2l package with the same version.

Users that need to be compatibility with older vcstool can just pin the vcstool version to vcstool < 1.0.0.

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.

@traversaro traversaro changed the title Migrate vcstool to be use vcs2l shim Migrate vcstool to use vcs2l shim Mar 27, 2026
@traversaro
Copy link
Copy Markdown
Contributor Author

@conda-forge-admin, please rerender

@conda-forge-admin
Copy link
Copy Markdown
Contributor

conda-forge-admin commented Mar 27, 2026

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:

  • ℹ️ top-level output has some malformed specs:
  • In section run: vcs2l $1.1.7.*
    Requirement spec fields should match the syntax name [version [build]]to avoid known issues in conda-build. For example, instead of name =version=build, use name version.* build. There should be no spaces between version operators and versions either: python >= 3.8 should be python >=3.8.

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

@conda-forge-admin
Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you but ran into some issues. Please check the output logs of the GitHub Actions workflow below for more details. You can also ping conda-forge/core (using the @ notation) for further assistance or you can try rerendering locally.

The following suggestions might help debug any issues:

  • Is the recipe/{meta.yaml,recipe.yaml} file valid?
  • If there is a recipe/conda-build-config.yaml file in the feedstock make sure that it is compatible with the current global pinnings.
  • Is the fork used for this PR on an organization or user GitHub account? Automated rerendering via the webservices admin bot only works for user GitHub accounts.

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

@traversaro
Copy link
Copy Markdown
Contributor Author

@conda-forge-admin, please rerender

@conda-forge-admin
Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you but ran into some issues. Please check the output logs of the GitHub Actions workflow below for more details. You can also ping conda-forge/core (using the @ notation) for further assistance or you can try rerendering locally.

The following suggestions might help debug any issues:

  • Is the recipe/{meta.yaml,recipe.yaml} file valid?
  • If there is a recipe/conda-build-config.yaml file in the feedstock make sure that it is compatible with the current global pinnings.
  • Is the fork used for this PR on an organization or user GitHub account? Automated rerendering via the webservices admin bot only works for user GitHub accounts.

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

@traversaro
Copy link
Copy Markdown
Contributor Author

@conda-forge-admin, please rerender

@conda-forge-admin
Copy link
Copy Markdown
Contributor

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.

@traversaro traversaro merged commit c97fcab into conda-forge:main Mar 27, 2026
2 checks passed
@traversaro traversaro deleted the updatetovcs2lstrikesback branch March 27, 2026 13:57
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.

2 participants