Hi Michael,

On 2/23/22 14:05, Michael Opdenacker wrote:
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?


https://autobuilder.yoctoproject.org/typhoon/#/builders/114

seems to be a good place to start with?

Cheers,
Quentin
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#56269): https://lists.yoctoproject.org/g/yocto/message/56269
Mute This Topic: https://lists.yoctoproject.org/mt/89338411/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to