Repository navigation
'dune subst' has been disabled in this project #496
Description
dune-release fails with:
$ dune-release
[-] Building source archive
File "dune-project", line 4, characters 7-15:
4 | (subst disabled)
^^^^^^^^
Error: 'dune subst' has been disabled in this project. Any use of it is
forbidden.
Hint: If you wish to re-enable it, change to (subst enabled) in the
dune-project file.
dune-release: [ERROR] run ['dune' 'subst']: exited with 1
If subst is disabled, then it's not needed and dune-release should skip this, I think. Removing this line fixed it for me:
Line 446 in 80ce7b5
I also tried with dune-release --dry-run and that got further, but then crashed with:
-: exec: git --git-dir .git checkout --orphan gh-pages
dune-release: internal error, uncaught exception:
File "lib/sos.ml", line 35, characters 28-34: Assertion failed
Activity
Leonidas-from-XIV commentedon Jan 6, 2025
In a way I am not even sure that the
dunebehavior is even correct here; maybe it should just silently do nothing (or print a warning and succeed) since the point ofsubst disabledis to disable substitution; it doesn't need to break shell commands.Otherwise we'd need to parse
dune-projectand see if it is enabled or not; that's kind of annoying as I am not aware of a good way to queryduneitself for the information. Or calldune substand if it fails ignore the failure.psafont commentedon Jan 28, 2026
I'm able to get around the issue and produce the tarball with
dune-release --dry-run. It's unfortunate because I had to disable subst because of a dune bug in a past version.talex5 commentedon Jul 23, 2026
This is still failing with dune 3.23.1 and dune-release 2.2.1 (I've updated my fix branch at https://github.com/talex5/dune-release/tree/fix-subst in case that helps anyone else).
talex5 commentedon Jul 24, 2026
Actually, don't use that branch; it fails to update the documentation (due to #511). This one from 2.0.0 should work: