Skip to content

Comments

Remove duplicate paths in managed Python errors#18008

Merged
konstin merged 2 commits intomainfrom
konsti/executable-directory-error-cleanup
Feb 16, 2026
Merged

Remove duplicate paths in managed Python errors#18008
konstin merged 2 commits intomainfrom
konsti/executable-directory-error-cleanup

Conversation

@konstin
Copy link
Member

@konstin konstin commented Feb 13, 2026

Fixes #17988

@konstin konstin added the error messages Messaging when something goes wrong label Feb 13, 2026
@konstin konstin changed the title Remove duplicate path in pyhton install error Remove duplicate path in python install error Feb 13, 2026
Copy link
Contributor

@EliteTK EliteTK left a comment

Choose a reason for hiding this comment

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

Don't the other errors (above and below this one) also have a similar problem?

@konstin
Copy link
Member Author

konstin commented Feb 16, 2026

I've cleaned the others up too

@EliteTK
Copy link
Contributor

EliteTK commented Feb 16, 2026

Looks good, don't forget to update the issue title as well.

@konstin konstin changed the title Remove duplicate path in python install error Remove duplicate path in python install errors Feb 16, 2026
@konstin konstin changed the title Remove duplicate path in python install errors Remove duplicate paths in managed Python errors Feb 16, 2026
@konstin konstin enabled auto-merge (squash) February 16, 2026 17:25
@konstin konstin merged commit 23234a3 into main Feb 16, 2026
189 of 191 checks passed
@konstin konstin deleted the konsti/executable-directory-error-cleanup branch February 16, 2026 17:27
Comment on lines +63 to 65
#[error("Failed to create Python executable link")]
LinkExecutable {
from: PathBuf,
to: PathBuf,
Copy link
Member

Choose a reason for hiding this comment

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

Here it looks like you retained to but dropped from and to is no longer used in the error message?

Comment on lines -67 to -73
#[error("Failed to create canonical Python executable at {} from {}", to.user_display(), from.user_display())]
CanonicalizeExecutable {
from: PathBuf,
to: PathBuf,
#[source]
err: io::Error,
},
Copy link
Member

Choose a reason for hiding this comment

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

Here you both dropped to and from, aren't we going to lose context in this error message?

tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Feb 18, 2026
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [uv](https://github.com/astral-sh/uv) | patch | `0.10.3` → `0.10.4` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>astral-sh/uv (uv)</summary>

### [`v0.10.4`](https://github.com/astral-sh/uv/blob/HEAD/CHANGELOG.md#0104)

[Compare Source](astral-sh/uv@0.10.3...0.10.4)

Released on 2026-02-17.

##### Enhancements

- Remove duplicate references to the affected paths when showing `uv python` errors ([#&#8203;18008](astral-sh/uv#18008))
- Skip discovery of workspace members that contain only git-ignored files, including in sub-directories ([#&#8203;18051](astral-sh/uv#18051))

##### Bug fixes

- Don't panic when initialising a package at the filesystem root (e.g. `uv init / --name foo`) ([#&#8203;17983](astral-sh/uv#17983))
- Fix permissions on `wheel` and `sdist` files produced by the `uv_build` build backend ([#&#8203;18020](astral-sh/uv#18020))
- Revert locked file change to fix locked files on NFS mounts ([#&#8203;18071](astral-sh/uv#18071))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4xOS4yIiwidXBkYXRlZEluVmVyIjoiNDMuMTkuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiUmVub3ZhdGUgQm90IiwiYXV0b21hdGlvbjpib3QtYXV0aG9yZWQiLCJkZXBlbmRlbmN5LXR5cGU6OnBhdGNoIl19-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

error messages Messaging when something goes wrong

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The ExecutableDirectory error should not need the path attached

3 participants