Skip to content
This repository was archived by the owner on Jan 20, 2026. It is now read-only.

fix: adjust license file paths for subdirectory manifests#538

Merged
nichmor merged 8 commits intoprefix-dev:mainfrom
nichmor:fix/license-file-paths-subdirectory
Jan 16, 2026
Merged

fix: adjust license file paths for subdirectory manifests#538
nichmor merged 8 commits intoprefix-dev:mainfrom
nichmor:fix/license-file-paths-subdirectory

Conversation

@nichmor
Copy link
Contributor

@nichmor nichmor commented Jan 14, 2026

No description provided.

When building from a git source with a subdirectory (like numpy's
`pixi-packages/default/pixi.toml` with `source.path = "../.."`),
license files from pyproject.toml are relative to the source directory
but rattler-build looks for them relative to the recipe directory
(manifest's parent).

This fix adds `adjust_license_file_paths()` which prefixes license
file paths with the relative path from the recipe directory back to
the source directory when they differ.

Fixes: prefix-dev/rattler-build#2094

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@nichmor nichmor marked this pull request as draft January 14, 2026 20:56
nichmor and others added 4 commits January 15, 2026 12:08
rattler-build expects multiple license files as a YAML list, not a
comma-separated string. Changed license_file type from Option<Value<String>>
to Option<Vec<Value<String>>> to produce proper YAML output:

  license_file:
    - LICENSE.txt
    - COPYING.txt

Removed the adjust_license_file_paths function as the path adjustment
will be handled by rattler-build.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@nichmor nichmor marked this pull request as ready for review January 16, 2026 14:08
Copy link
Member

@wolfv wolfv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perfect!

@nichmor nichmor enabled auto-merge (squash) January 16, 2026 15:01
@nichmor nichmor merged commit 750cb88 into prefix-dev:main Jan 16, 2026
14 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants