Skip to content

dgb: bootstrap src/impl/dgb skeleton (M2) - #60

Merged
frstrtr merged 3 commits into
masterfrom
dgb/bootstrap-skeleton
Jun 2, 2026
Merged

dgb: bootstrap src/impl/dgb skeleton (M2)#60
frstrtr merged 3 commits into
masterfrom
dgb/bootstrap-skeleton

Conversation

@frstrtr

@frstrtr frstrtr commented Jun 2, 2026

Copy link
Copy Markdown
Owner

M2: src/impl/dgb skeleton — DGB Scrypt-only V36 coin module

Structural bootstrap of the DGB Scrypt-only coin module, following the approved M1 implementation plan c2pool-dgb-embedded-impl-plan.md (frstrtr/the docs, 8ef8d2d). V36 ratification context: project_v36_dgb_scrypt_only.

Scope: M2 structural skeleton only. node.cpp + share commit/validation wiring, Scrypt-only template-builder fill, and DigiShield finalization land in M3. Mirrors the bch M2 bootstrap discipline.

Per-coin isolation invariant

  • src/impl/dgb/ only — no src/core/ or bitcoin_family/ touches.
  • No -DAUX_DOGE plumbing (the dual-parent stretch / §X is parked behind Phase 5.8; ltc-doge owns the shared DOGE-aux surface).
  • Verified diff vs origin/master: 6 files, all under src/impl/dgb/.

CMake registration held

  • No top-level CMakeLists.txt change in this PR. The if(COIN_DGB) guard lives inside src/impl/dgb/CMakeLists.txt (message-only at M2; coin/, daemon/, test/ subdirs commented out).
  • Top-level add_subdirectory(src/impl/dgb) registration is held until ci-steward's PR-CMake-object-library convention lands; impl_dgb registers as 1 of 5 per-coin OBJECT libs at that point (M3).

Files (6)

  • coin/header_chain.hpp — Scrypt-only validation point; non-Scrypt algos accept-by-continuity (full 5-algo deferred to V37); DigiShield insertion point.
  • coin/template_builder.hpp — GBT rules=[scrypt]; p2pool-merged-v36 parity point.
  • daemon/README.txt — embedded DigiByte Core slice scope (no in-tree vendor).
  • test/README.txt — Scrypt fixtures + multi-algo accept-by-continuity + parity outline.
  • CMakeLists.txt (+ coin/CMakeLists.txt) — -DCOIN_DGB=ON build gate, guarded.

Requested reviewers

  • bch-embedded-steward — cross-coin M2 parity (mirrors bch/bootstrap-skeleton).
  • ci-steward — OBJECT-lib seam compat (if(COIN_DGB) must compose with the forthcoming convention).
  • dash (optional) — sharechain-validation invariants.

frstrtr added 3 commits June 2, 2026 18:25
Scrypt-only V36 coin module scaffold per c2pool-dgb-embedded-impl-plan.md
(8ef8d2d). Mirrors src/impl/btc layout: coin/ + daemon/ + test/ slices,
-DCOIN_DGB=ON build gate (guarded, not yet registered top-level).

- coin/header_chain.hpp: Scrypt-only validation point; non-Scrypt algos
  accept-by-continuity, full 5-algo deferred to V37. DigiShield insertion pt.
- coin/template_builder.hpp: GBT rules=[scrypt]; p2pool-merged-v36 parity pt.
- daemon/README.txt: embedded DigiByte Core slice scope (no in-tree vendor).
- test/README.txt: Scrypt fixtures + multi-algo accept-by-continuity + parity.

Preserves pre-existing config_coin/config_pool/node stubs. No -DAUX_DOGE
(stretch parked behind Phase 5.8). Per-coin isolation held: src/impl/dgb only.

Scope: M2 structural skeleton only. node.cpp + share commit/validation
wiring (and Scrypt-only template-builder fill + DigiShield finalization)
land in M3. Mirrors bch M2 bootstrap discipline.
…tor-only DigiShield note into header_chain.hpp stub (PR #60, M3 prereq)
- Drop empty src/impl/dgb/coin/CMakeLists.txt (referenced only by a
  commented-out add_subdirectory(coin) in src/impl/dgb/CMakeLists.txt;
  bch carries none). Parity cleanup, no build impact.

config_coin.hpp NOT added: src/impl/dgb/config_coin.hpp already exists on
master (tracked, unmodified by this PR -> absent from the diff view) and
already defines CoinParams::GBT_ALGO = "scrypt". template_builder.hpp
reference is satisfied, not dangling.

Per bch-embedded-steward review (UID 632); no scope expansion.
@frstrtr
frstrtr merged commit cd32473 into master Jun 2, 2026
11 of 14 checks passed
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.

1 participant