Skip to content

fix: use PEP 440 direct URL syntax instead of deprecated #egg= fragment#280

Closed
aldbr wants to merge 1 commit intodevelfrom
fix-egg
Closed

fix: use PEP 440 direct URL syntax instead of deprecated #egg= fragment#280
aldbr wants to merge 1 commit intodevelfrom
fix-egg

Conversation

@aldbr
Copy link
Copy Markdown
Contributor

@aldbr aldbr commented Apr 2, 2026

Summary

  • Replace deprecated #egg= fragment syntax with PEP 440 direct URL syntax (name[extra] @ URL) in pilotCommands.py
  • Fixes pip install failures when using --modules with pip >= 25.0, which rejects #egg= fragments with error: invalid-egg-fragment

Motivation

Since pip 25.0 (January 2025), the #egg= URL fragment is no longer accepted and raises an invalid-egg-fragment error:

× The 'DIRAC[pilot]' egg fragment is invalid
╰─> from 'git+https://github.com/ryuwd/DIRAC.git@feat/cwl-job-submission#egg=DIRAC[pilot]'

hint: Try using the Direct URL requirement syntax: 'name[extra] @ URL'

This breaks any pilot installation using the modules option (e.g. url:::project:::branch syntax) to install a non-released DIRAC version from a git repository.

Changes

# Before
pip install git+https://github.com/fstagni/DIRAC.git@v7r2-fixes33#egg=DIRAC[pilot]

# After
pip install DIRAC[pilot] @ git+https://github.com/fstagni/DIRAC.git@v7r2-fixes33

@aldbr aldbr requested a review from fstagni April 2, 2026 16:07
@aldbr aldbr changed the base branch from master to devel April 7, 2026 10:06
@aldbr
Copy link
Copy Markdown
Contributor Author

aldbr commented Apr 7, 2026

Closing because not coming from my fork.
Here is the right one: #281

@aldbr aldbr closed this Apr 7, 2026
@aldbr aldbr deleted the fix-egg branch April 7, 2026 15:37
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.

1 participant