Skip to content

rammap 1.1.2 (new formula) - #297323

Open
BenjaminDEMAILLE wants to merge 1 commit into
Homebrew:mainfrom
BenjaminDEMAILLE:migrate-rammap
Open

rammap 1.1.2 (new formula)#297323
BenjaminDEMAILLE wants to merge 1 commit into
Homebrew:mainfrom
BenjaminDEMAILLE:migrate-rammap

Conversation

@BenjaminDEMAILLE

Copy link
Copy Markdown
Contributor

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>?
  • Is your test running fine brew test <formula>?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

  • I did not use AI/LLM to create this PR, or I disclosed the tool/model below and reviewed its output; I did not attribute commits to AI and will answer maintainer questions and review comments myself without AI/LLM.

AI-assisted contribution: the formula was written with Claude Code (Claude Opus 5), which also ran brew install --build-from-source, brew test, brew audit --new --strict --online and brew style on macOS 15 arm64 on my machine. I reviewed the formula and those results before opening this PR. The commit carries no AI attribution trailer.


Built and tested locally on macOS arm64.

Adds rammap, a pure-Rust sequence aligner and read mapper that mirrors the minimap2 interface and output.

Notes for review:

  • Plain cargo install via std_cargo_args(path: "rammap"); the repository is a two-crate workspace and only the rammap binary crate is installed.
  • Bottle safety: upstream's .cargo/config.toml deliberately avoids -C target-cpu=native, and the SIMD kernels dispatch at runtime via is_x86_feature_detected!, so the build stays baseline-safe.
  • The test aligns a read lifted verbatim out of a generated reference and checks both output formats: target name, strand, reference length and MAPQ in the PAF record, and 200M with NM:i:0 in the SAM record.

@github-actions github-actions Bot added new formula PR adds a new formula to Homebrew/homebrew-core rust Rust use is a significant feature of the PR or issue labels Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new formula PR adds a new formula to Homebrew/homebrew-core rust Rust use is a significant feature of the PR or issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant