Hi Quentin, On 2/23/22 12:31, Quentin Schulz wrote: > From: Quentin Schulz <[email protected]> > > There are some intermittent issues with the script not publishing all > versions. So let's go extreme and fail the script if any error happens: > - a command returns a non-zero code, even if piped, > - a variable is used uninitialized, > > This also makes the script print each and every command being run so we > have a better idea where the script struggles. > > Cc: Quentin Schulz <[email protected]> > Signed-off-by: Quentin Schulz <[email protected]> > --- > > RFC: > - I highly suspect this change to break the building script because > Sphinx probably throws errors at old doc builds, > - Also, with Sphinx updates, new warnings might arrise on > old/unmaintained branches, we probably want to use virtualenv with > specific Sphinx versions to make sure this is not an issue in the > future? (and allows to bump Sphinx (or other dependencies) requirement > for newer doc branches), > > scripts/run-docs-build | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/scripts/run-docs-build b/scripts/run-docs-build > index 3bc35b3..b7b5773 100755 > --- a/scripts/run-docs-build > +++ b/scripts/run-docs-build > @@ -2,6 +2,12 @@ > # Called with $1 as the build directory > # $2 as the path to yocto-docs > # $3 as the path to bitbake > + > +set -e > +set -u > +set -o pipefail > +set -x > + > builddir=$1 > ypdocs=$2/documentation/ > bbdocs=$3/doc/
Agreeing with the change. How would we get the script output? Thanks Michael. -- Michael Opdenacker, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#56262): https://lists.yoctoproject.org/g/yocto/message/56262 Mute This Topic: https://lists.yoctoproject.org/mt/89338411/21656 Group Owner: [email protected] Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
