23 May
2019
23 May
'19
3:04 a.m.
On Thu, 2019-05-23 at 11:36 +0200, Riccardo Pittau wrote:
Hello again!
I think this change https://review.opendev.org/659975 broke a couple of things.
For example, when running build-openstack-api-ref, the tox_envlist is correctly set to api-ref, overwriting the value of docs from the parent openstack-tox-docs That means that tox will never build docs, so the doc/build dir doesn't exist, that's why the rsync error afterwards.
The fetch-sphinx-output role should reflect that or at least evaluate the presence of the dir.
I think the correct fix is to keep the removed variable as proposed in https://review.opendev.org/660962 . Sorry for merging the broken patch in the first place. Jens