@@ -2154,9 +2154,6 @@ build_start = "echo build started"
21542154build_failed = " notify-send 'build failed'"
21552155build_finished = " notify-send 'build finished'"
21562156
2157- # Runs alongside the build and is stopped when it ends.
2158- during_build = { cmd = " mcpp-hooks-audioplayer bgm" , loop = true }
2159-
21602157# Optional; these are the defaults.
21612158timeout_seconds = 10
21622159enabled = true
@@ -2207,8 +2204,8 @@ build_start
22072204 └─ build fails → during_build closes → build_failed
22082205```
22092206
2210- ` during_build ` closes ** before** the terminal hook, so a "build finished" sound
2211- is not competing with the background music it replaces .
2207+ ` during_build ` closes ** before** the terminal hook, so the two commands never
2208+ overlap .
22122209
22132210` build_failed ` and ` build_finished ` are mutually exclusive, and both are
22142211reachable only after ` build_start ` has run. A project that cannot be * prepared*
@@ -2286,7 +2283,6 @@ than adding media handling to mcpp:
22862283
22872284``` toml
22882285[hooks ]
2289- during_build = { cmd = " mcpp-hooks-audioplayer bgm" , loop = true }
22902286build_finished = " mcpp-hooks-audioplayer niulai-mm"
22912287build_failed = " mcpp-hooks-audioplayer niulai-niulai"
22922288side_effect = false
@@ -2295,11 +2291,10 @@ side_effect = false
22952291deps = [" xim:mcpp-hooks-audioplayer@0.0.1" ]
22962292```
22972293
2298- Background music for the length of the build, and a different sound for how it
2299- ended. ` side_effect = false ` is written out rather than left to the default:
2300- it is the value this manifest wants on its own terms — a missing audio device
2301- should never fail a build — so it will still say so once the key has more than
2302- one accepted value.
2294+ A different sound for a successful or failed build. ` side_effect = false ` is
2295+ written out rather than left to the default: it is the value this manifest
2296+ wants on its own terms — a missing audio device should never fail a build — so
2297+ it will still say so once the key has more than one accepted value.
23032298
23042299## Appendix A. Schema Ownership Principle (admission criteria for new fields)
23052300
0 commit comments