Skip to content

Add cpuinfo#5004

Open
Alex-PLACET wants to merge 2 commits intoemscripten-forge:mainfrom
Alex-PLACET:add_cpuinfo
Open

Add cpuinfo#5004
Alex-PLACET wants to merge 2 commits intoemscripten-forge:mainfrom
Alex-PLACET:add_cpuinfo

Conversation

@Alex-PLACET
Copy link
Collaborator

@Alex-PLACET Alex-PLACET commented Mar 4, 2026

Added recipes/recipes_emscripten/cpuinfo/recipe.yaml with proper structure:

  • context section with version (and optionally name)
  • package section with name and version using Jinja2 templates
  • source section with:
    • Source URL is valid and points to archive file (.tar.gz, .tar.bz2, .tar.xz, .tgz, or .zip)
    • Source URL contains ${{ version }} template for version updates
    • SHA256 hash is correct (verified with curl -sL <url> | sha256sum)
    • Patches (if any) are included in [package-name]/patches/ directory
  • build section with appropriate script/method
    • Python packages: ${PYTHON} -m pip install . ${PIP_ARGS}
    • R packages: $R CMD INSTALL $R_ARGS .
    • C++ packages: Uses emcmake/emmake or emconfigure/emmake
    • Rust packages: Uses rust-nightly and maturin or appropriate Rust build tool
    • Build number is 0
    • If the script is longer than 3 lines, a build.sh is included
  • requirements section (build, host, run as needed)
  • tests section
    • Python packages: test_import_[package].py file created and referenced
    • C++ packages: Test executable or package_contents test
    • R packages: Package contents test
  • about section with license, homepage, summary

Package Details

  • Package Name: cpuinfo
  • Version: 7364b490b5f78d58efe23ea76e74210fd6c3c76f

@Alex-PLACET Alex-PLACET marked this pull request as ready for review March 4, 2026 12:25
Copy link
Member

@IsabelParedes IsabelParedes left a comment

Choose a reason for hiding this comment

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

Minor change. Everything else looks good :)

#!/bin/bash
set -e

export PYTHON=$BUILD_PREFIX/bin/python
Copy link
Member

Choose a reason for hiding this comment

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

Is this still needed?

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