Skip to content

fastplong 0.7.1 (new formula) - #301438

Merged
BrewTestBot merged 2 commits into
Homebrew:mainfrom
BenjaminDEMAILLE:fastplong
Sep 17, 2026
Merged

BrewTestBot merged 2 commits into
Homebrew:mainfrom
BenjaminDEMAILLE:fastplong

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 by Claude Code (Claude Opus 5) of ~85% of the work: the AI wrote the formula and ran brew style, HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source, brew test, brew audit --strict --new and brew linkage --test on my machine. I reviewed the formula and those results before opening this PR, and will answer maintainer questions and review comments myself. The commit carries no AI attribution trailer.

Built and tested locally on macOS 26.6.2 running arm64.


fastplong is the long-read counterpart to fastp, from the same authors, and shares its dependency set (highway, isa-l, libdeflate).

The test writes a two-read FASTQ where the second read has a low-quality head, runs the trimmer, then parses the emitted report.json to assert the input read count, so it exercises the FASTQ output and the JSON/HTML reporting rather than just the binary starting up.

@github-actions github-actions Bot added the new formula PR adds a new formula to Homebrew/homebrew-core label Aug 30, 2026
Comment thread Formula/f/fastplong.rb Outdated
@BenjaminDEMAILLE

Copy link
Copy Markdown
Contributor Author

Status update: the upstream fix for the make install workaround is now open at OpenGene/fastplong#44 (install does not create $(BINDIR), so it fails on a fresh prefix). The same patch is open for fastp, which carries the identical Makefile: OpenGene/fastp#714

Once either lands in a release, the mkdir prefix/"bin" line here can go. CI is green on all platforms.

@BenjaminDEMAILLE BenjaminDEMAILLE changed the title fastplong 0.7.0 (new formula) fastplong 0.7.1 (new formula) Sep 11, 2026
@BenjaminDEMAILLE

Copy link
Copy Markdown
Contributor Author

Updated to 0.7.1, released today, which is the first tag containing the upstream fix (OpenGene/fastplong#44, merged on 2026-09-04). Its install target now creates $(BINDIR) itself:

install:
	mkdir -p $(BINDIR)
	install $(TARGET) $(BINDIR)/$(TARGET)

So the mkdir prefix/"bin" line is gone from the formula; nothing is patched here any more. Built from source, brew test and brew audit --new --strict --online pass on macOS 26.6.2 arm64, stable and --HEAD.

The same Makefile bug exists in fastp, fixed by OpenGene/fastp#714 (merged), but no release carries it yet, so Formula/f/fastp.rb still needs its own mkdir for now.

@SMillerDev
SMillerDev removed their request for review September 11, 2026 07:33
@BenjaminDEMAILLE
BenjaminDEMAILLE marked this pull request as draft September 17, 2026 18:18
@BenjaminDEMAILLE
BenjaminDEMAILLE marked this pull request as ready for review September 17, 2026 18:18
@github-actions

Copy link
Copy Markdown
Contributor

馃 An automated task has requested bottles to be published to this PR.

Caution

Please do not push to this PR branch before the bottle commits have been pushed, as this results in a state that is difficult to recover from. If you need to resolve a merge conflict, please use a merge commit. Do not force-push to this PR branch.

@github-actions github-actions Bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Sep 17, 2026
@BrewTestBot
BrewTestBot added this pull request to the merge queue Sep 17, 2026
Merged via the queue into Homebrew:main with commit a7a7755 Sep 17, 2026
19 checks passed
@BenjaminDEMAILLE
BenjaminDEMAILLE deleted the fastplong branch September 17, 2026 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. new formula PR adds a new formula to Homebrew/homebrew-core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants