Skip to content

Conversation

zanieb
Copy link
Member

@zanieb zanieb commented Oct 7, 2025

No description provided.

Previously, workspace discovery would miss a valid workspace root is the
project path has a trailing slash. This would cause `uv build` to use
the wrong target directory.

This is a breaking change. Specifically, the output path changes from
`<workspace root>/<child>/dist/` to `<workspace root>/dist/` when using
`uv build <child>/` in the workspace root.

**Setup** Create a workspace with a member.

```
uv init
uv init --lib child
```

**Before**

```
$ uv build child/
Building source distribution (uv build backend)...
Building wheel from source distribution (uv build backend)...
Successfully built child/dist/child-0.1.0.tar.gz
Successfully built child/dist/child-0.1.0-py3-none-any.whl
$ uv build child
Building source distribution (uv build backend)...
Building wheel from source distribution (uv build backend)...
Successfully built dist/child-0.1.0.tar.gz
Successfully built dist/child-0.1.0-py3-none-any.whl
```

**After**

```
$ uv build child/
Building source distribution (uv build backend)...
Building wheel from source distribution (uv build backend)...
Successfully built dist/child-0.1.0.tar.gz
Successfully built dist/child-0.1.0-py3-none-any.whl
$ uv build child
Building source distribution (uv build backend)...
Building wheel from source distribution (uv build backend)...
Successfully built dist/child-0.1.0.tar.gz
Successfully built dist/child-0.1.0-py3-none-any.whl
```

Fixes #13914
@konstin konstin temporarily deployed to uv-test-registries October 7, 2025 21:02 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-registries October 7, 2025 22:31 — with GitHub Actions Inactive
@zanieb
Copy link
Member Author

zanieb commented Oct 7, 2025

I'm not really sure why #15133 was merged into here rather than main, but just ignore that in the diff.

@zanieb zanieb marked this pull request as ready for review October 7, 2025 23:02
@zanieb
Copy link
Member Author

zanieb commented Oct 7, 2025

Loosely waiting on #16150 but we may need to ship without the Docker images.

@zanieb zanieb temporarily deployed to uv-test-registries October 7, 2025 23:08 — with GitHub Actions Inactive
@zanieb zanieb enabled auto-merge (squash) October 7, 2025 23:13
@zanieb zanieb merged commit 39b6886 into main Oct 7, 2025
157 of 158 checks passed
@zanieb zanieb deleted the zb/090 branch October 7, 2025 23:17
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