[libcxx-commits] [PATCH] D111926: [libc++][doc] Use sphinx variables to make updating the docs version easier

[Tom Stellard via Phabricator via libcxx-commits]

tstellar marked an inline comment as done.
tstellar added inline comments.


================
Comment at: libcxx/docs/conf.py:54
 # The full version, including alpha/beta/rc tags.
-release = '14.0'
+release = version + 'git'
 
----------------
Mordante wrote:
> Mordante wrote:
> > The current output "libc++ 14.0.0git documentation" looks weird.
> I expect for a release I'd have to remove `+ 'git'`, but doing so results in errors when building the documentation.
> ```
> make[3]: *** [projects/libcxx/docs/CMakeFiles/docs-libcxx-html.dir/build.make:58: projects/libcxx/docs/CMakeFiles/docs-libcxx-html] Error 2
> make[2]: *** [CMakeFiles/Makefile2:25091: projects/libcxx/docs/CMakeFiles/docs-libcxx-html.dir/all] Error 2
> make[1]: *** [CMakeFiles/Makefile2:25098: projects/libcxx/docs/CMakeFiles/docs-libcxx-html.dir/rule] Error 2
> make: *** [Makefile:7544: docs-libcxx-html] Error 2
> ```
> Do I misunderstand the intention or is there a real error?
I decided to drop the 'git' suffix in order to simplify the config.  I can add it back (with the '-' in front of it) if you think it helps remind readers that it is a pre-release.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111926/new/

https://reviews.llvm.org/D111926