GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Rework alias linking to ensure links are only created within bin directory by zooba · Pull Request #277 · python/pymanager

Conversation

@zooba

@zooba zooba commented Feb 19, 2026

Copy link
Copy Markdown
Member

Fixes #258

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@zooba
Rework alias linking to ensure links are only created within bin dire…
624ffb6 
…ctory.

Fixes python#258
@zooba
zooba requested a review from Copilot February 19, 2026 13:01
Copilot started reviewing on behalf of zooba February 19, 2026 13:01
View session
Copilot AI reviewed Feb 19, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Pull request overview

This pull request fixes broken executables in the Python bin directory by reworking the alias linking mechanism to ensure hard links are only created within the bin directory itself, rather than linking to files in the Python installation directory.

Changes:

  • Modified alias creation logic to search for existing launchers within the bin directory for hard linking instead of linking directly to installation directory files
  • Added support for the force flag to unconditionally recreate aliases
  • Updated test expectations to reflect new debug log messages

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/manage/aliasutils.py Refactored _create_alias function to search for existing launchers in the bin directory and link to those instead of linking to installation files, avoiding cross-directory linking issues
tests/test_alias.py Added force = False attribute to test Cmd class and updated test assertions to match new logging behavior

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
zooba added 3 commits February 19, 2026 16:09
@zooba
Slightly improved logic, significantly improve logging
768deb8
@zooba
Allow install_dir to be optional for logging
c1bfec8
@zooba
Remove unnecessary message and fix tests
46c0854
@zooba
zooba requested a review from Copilot February 19, 2026 19:30
Copilot started reviewing on behalf of zooba February 19, 2026 19:31
View session
Copilot AI reviewed Feb 19, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread src/manage/pathutils.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
@zooba
Update src/manage/pathutils.py
2b619dc 
Co-authored-by: Copilot <[email protected]>
@zooba
zooba merged commit c6ee3eb into python:main Feb 23, 2026
4 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@zooba
zooba deleted the gh-258 branch February 23, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Broken executables with 26.0b1

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@zooba
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.