Repository navigation
build: use sys-asm's committed bytecode; mine-deposit QoL knobs - #7
Merged
Merged
Conversation
Wire the miner's --start-nonce through the Makefile: the GPU search picks a random start offset by default (fresh range per run); set START_NONCE=<n> to pin it for a reproducible search. Passed only when non-empty, so it stays compatible with miners that predate the flag. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Previously mine-deposit only streamed to stdout; the result had to be captured by hand. Now it tees the full run to runs/mine-deposit-<ts>.log and extracts the Found block to runs/mine-deposit-<ts>.result.txt. Override the location with RUNS_DIR=. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
sys-asm commits the assembled bytecode under bytecode/<contract>/ctor.hex precisely so consumers don't need the geas assembler, whose output could differ between versions. Read those files instead of running geas. Also resolve the old TODO now that the builder contracts landed upstream: point the submodule at ethereum/sys-asm main and bump the gitlink to a0c61db (the EIP-8282 PR#52 merge). The committed bytecode there is byte-identical to the init code the existing vanity mining runs used, so mined addresses and presigned deploy txs remain valid. The Makefile no longer hardcodes a revision: the submodule gitlink is the pin, and REF=<sha|branch> stays available as an override. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Third round of build-system follow-ups from the
busa/sys-asm-initcodebranch (after #3/#4).What changed
354381f):sys-asm commits the assembled bytecode under
bytecode/<contract>/ctor.hexprecisely soconsumers don't need the geas assembler, whose output could differ between versions
(thanks @fjl for the pointer). The
initcode*/mine-deposittargets now read thosefiles; the geas dependency is gone entirely.
ethereum/sys-asm(the builder contracts landed on main,resolving the old TODO) and the gitlink bumped to
a0c61db, the EIP-8282 PR#52merge commit. Verified byte-identical to the init code prior mining runs used.
REF=<sha|branch>remains as an override, and the targets error clearly if thechecked-out revision predates the committed bytecode.
START_NONCEknob for mine-deposit (0615f7b): pins the GPU search start offsetfor reproducible runs. The flag is only passed when set — note it needs a miner with
--start-noncesupport (search: randomize GPU start nonce (add --start-nonce) #5), so merging search: randomize GPU start nonce (add --start-nonce) #5 first makes it usable.958180c): each run tees toruns/<ts>.logandextracts the
Found!block toruns/<ts>.result.txt.🤖 Generated with Claude Code