Skip to content

PEP 825: add a non-normative section on multiple sources - Michal#62

Open
mgorny wants to merge 3 commits into
pep-wheel-variants-acceptancefrom
pep825-merging-sources
Open

PEP 825: add a non-normative section on multiple sources - Michal#62
mgorny wants to merge 3 commits into
pep-wheel-variants-acceptancefrom
pep825-merging-sources

Conversation

@mgorny

@mgorny mgorny commented Jun 30, 2026

Copy link
Copy Markdown

No description provided.

Signed-off-by: Michał Górny <mgorny@quansight.com>
@read-the-docs-community

read-the-docs-community Bot commented Jun 30, 2026

Copy link
Copy Markdown

Documentation build overview

📚 wheelnext-peps | 🛠️ Build #33493778 | 📁 Comparing 6135285 against latest (0cde95d)

  🔍 Preview build  

740 files changed · ± 740 modified

± Modified

@DEKHTIARJonathan DEKHTIARJonathan changed the title PEP 825: add a non-normative section on multiple sources PEP 825: add a non-normative section on multiple sources - Michal Jul 1, 2026
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny

mgorny commented Jul 3, 2026

Copy link
Copy Markdown
Author

I've integrated some of Jonathan's ideas from #63. One thing I've left were PEP 766 references; while it's a draft, I think it's fine to mention it in non-normative section as a prior art.

Comment thread peps/pep-0825.rst
Comment on lines +816 to +819

It is entirely valid for tools not to support this behavior, either by
not providing support for using multiple sources at all, or by rejecting
to proceed if more than one of the sources includes variant wheels.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The easiest for tools is to always have an index order which avoids inter-index comparisons.

@rgommers rgommers left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM, thanks for writing this! A couple of small textual suggestions only.

Comment thread peps/pep-0825.rst Outdated
Comment thread peps/pep-0825.rst
not providing support for using multiple sources at all, or by rejecting
to proceed if more than one of the sources includes variant wheels.

When processing variant wheels from different sources, it is recommended

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

"sources" could be mildly confusing perhaps (could also be read as different authors/orgs, or source code), so I suggest:

Suggested change
When processing variant wheels from different sources, it is recommended
When processing variant wheels from different indexes, it is recommended

Should be clearer, unless there's also an issue with non-index sources (i.e., a local build?)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yeah, my understanding was that we want to cover non-index sources as well.

Co-authored-by: Ralf Gommers <ralf.gommers@gmail.com>
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.

4 participants