[Cashtab] Also copy explorer in Dockerfile
ClosedPublic
Actions

Authored by bytesofman on Dec 8 2024, 01:22.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC5a2dab6f2181: [Cashtab] Also copy explorer in Dockerfile

Summary

CI builds are failing with an error suggesting that ./build-wasm cannot run without /web/explorer/ being in place. Looks like it could be related to D17292

Now that the explorer is in the workspace of the root Cargo.toml, we need it available for ./build-wasm

Test Plan

docker build -f cashtab.Dockerfile -t cashtab_local .

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bytesofman created this revision.Dec 8 2024, 01:22

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 8 2024, 01:22

Harbormaster completed remote builds in B31575: Diff 51476.Dec 8 2024, 01:22

bytesofman published this revision for review.Dec 8 2024, 01:23

bytesofman added a child revision: D17316: [Cashtab] Dockerfile lint.Dec 8 2024, 01:24

Fabien accepted this revision.Dec 8 2024, 01:37

This revision is now accepted and ready to land.Dec 8 2024, 01:37

Closed by commit rABC5a2dab6f2181: [Cashtab] Also copy explorer in Dockerfile (authored by bytesofman). · Explain WhyDec 8 2024, 02:25

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC5a2dab6f2181: [Cashtab] Also copy explorer in Dockerfile.

bytesofman mentioned this in D17317: [monorepo] Update dockerfiles that build-wasm to include explorer.Dec 8 2024, 02:33

bytesofman mentioned this in rABCcf0630445dc9: [monorepo] Update dockerfiles that build-wasm to include explorer.Dec 8 2024, 04:04

Revision Contents
Changeset List

Path

Size

cashtab.Dockerfile

4 lines

Diff 51478

View Options

[Cashtab] Also copy explorer in DockerfileClosedPublicActions