Skip to content

Add fxdiv#5005

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

Add fxdiv#5005
Alex-PLACET wants to merge 2 commits intoemscripten-forge:mainfrom
Alex-PLACET:add_fxdiv

Conversation

@Alex-PLACET
Copy link
Collaborator

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

Added recipes/recipes_emscripten/[package-name]/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

PR Formatting

  • PR title follows format: Add [package-name] or Update [package-name] to [version]
  • PR description includes:
    • Version being added/updated
    • Any special build considerations or patches applied

Package Details

  • Package Name: fxdiv
  • Version: 63058eff77e11aa15bf531df5dd34395ec3017c8

@Alex-PLACET Alex-PLACET self-assigned this Mar 4, 2026
@Alex-PLACET Alex-PLACET marked this pull request as ready for review March 4, 2026 12:58
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.

Since this package is a single header file, we should make it noarch: generic

Similar to https://github.com/emscripten-forge/recipes/blob/main/recipes/recipes_emscripten/clang-resource-headers/recipe.yaml

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