Skip to content

Add publish dates for cross build environments#6248

Merged
agriyakhetarpal merged 11 commits into
pyodide:mainfrom
agriyakhetarpal:publish-dates-for-cross-build-environments
May 21, 2026
Merged

Add publish dates for cross build environments#6248
agriyakhetarpal merged 11 commits into
pyodide:mainfrom
agriyakhetarpal:publish-dates-for-cross-build-environments

Conversation

@agriyakhetarpal

Copy link
Copy Markdown
Member

Description

Adds published_at field for cross-build environments. It's the same as what we get from the GitHub API for releases, so it's kind of self-documenting. I am trying to use this data downstream in cibuildwheel to improve its supply chain for Pyodide package builders via cooldowns. See pypa/cibuildwheel#2841.

@agriyakhetarpal

Copy link
Copy Markdown
Member Author

Ah, looks like we also need to update pyodide-build for this. Let me do that in a bit.

@ryanking13

Copy link
Copy Markdown
Member

Oh, wait. pyodide-build does not accept extra parameters? That's bad... I should've allowed it.

@agriyakhetarpal

agriyakhetarpal commented May 20, 2026

Copy link
Copy Markdown
Member Author

No worries! I think we should keep it that way. (edit: we shouldn't, as discussed in the pyodide-build PR). BTW, I am currently making similar changes for our nightly cross-build environments repo, as I'd like to use those sometimes for debugging purposes and to help downstream package builders. I am going to put together some more PRs later today.

@agriyakhetarpal agriyakhetarpal marked this pull request as ready for review May 20, 2026 10:49
Comment thread .gitmodules

@agriyakhetarpal agriyakhetarpal May 20, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This change is only made for this PR for the CI to be green, i.e., this branch points to my PR at pyodide/pyodide-build#349. We have a bit of a "bootstrapping problem" :D

I'll revert this change in another PR after we merge this PR and the v2 JSON goes live on GitHub Pages, and then CI will be green on that pyodide-build PR as well.

agriyakhetarpal added a commit to pyodide/pyodide-build-environment-nightly that referenced this pull request May 21, 2026
- Adds a script to add v2 metadata for nightly cross-build environments
in a JSON file (see generated JSON file)
- The `publish.yml` workflow file now pushes a commit to `main`. I chose
to not use a PR because it adds a bit of churn, but I can change the
approach if asked
- Added a `pages.yml` workflow to deploy the file on GitHub Pages

There is no v1 file here, just v2, because it's the first time we are
doing it and it doesn't make sense to go with the old spec.

I will follow this up by putting together a PR in pyodide-build to use
this metadata for those who would like to use nightly and nightly-debug
builds.

xref pyodide/pyodide#6248,
pyodide/pyodide-build#349

@ryanking13 ryanking13 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! As I commented, I think it is okay to freeze v1 and start publishing v2 only.

Comment thread .github/workflows/pages.yml Outdated
Comment thread .github/workflows/pages.yml Outdated
@agriyakhetarpal

Copy link
Copy Markdown
Member Author

Done!

@agriyakhetarpal agriyakhetarpal merged commit 3bc5c55 into pyodide:main May 21, 2026
8 of 10 checks passed
@agriyakhetarpal agriyakhetarpal deleted the publish-dates-for-cross-build-environments branch May 21, 2026 09:21
@agriyakhetarpal

agriyakhetarpal commented Jun 9, 2026

Copy link
Copy Markdown
Member Author

I think we shouldn't have frozen v1 so quickly:

Error: cibuildwheel: Could not find Pyodide cross-build environment version 314.0.0 in the available versions as reported by pyodide-build v0.34.4. Available pyodide xbuildenv versions are: 314.0.0a2, 314.0.0a1

I'm creating a pyodide-build v0.35.0 release and will manually patch the v1 JSON file here in this case to ease things.

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