XCP-ng
    • Categories
    • Recent
    • Tags
    • Popular
    • Users
    • Groups
    • Register
    • Login

    'yarn build' failure

    Scheduled Pinned Locked Moved Xen Orchestra
    19 Posts 5 Posters 1.8k Views 2 Watching
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • C Offline
      ci4ic4
      last edited by

      Hi,

      On a freshly installed Debian 10.9 amd64 system, strictly following the build-from-sources procedure - just for reference:

      apt-get update
      apt-get upgrade
      curl -fsSL https://deb.nodesource.com/setup_14.x | bash -
      apt-get install gcc g++ make
      apt-get install -y nodejs
      curl -sL https://dl.yarnpkg.com/debian/pubkey.gpg | gpg --dearmor | tee /usr/share/keyrings/yarnkey.gpg >/dev/null
      echo "deb [signed-by=/usr/share/keyrings/yarnkey.gpg] https://dl.yarnpkg.com/debian stable main" | tee /etc/apt/sources.list.d/yarn.list
      apt-get update && sudo apt-get install yarn
      apt-get install build-essential redis-server libpng-dev git python-minimal libvhdi-utils lvm2 cifs-utils
      
      yarn build
      

      fails with

      
      Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
      Successfully compiled 131 files with Babel (31694ms).
      [09:53:39] Finished 'buildStyles' after 14 s
      [09:53:39] Finished 'copyAssets' after 15 s
      [09:54:44] 'buildScripts' errored after 1.32 min
      [09:54:44] GulpUglifyError: unable to minify JavaScript
      Caused by: SyntaxError: Unexpected token name «await», expected punc «(»
      File: /home/xci/xen-orchestra/packages/xo-web/src/index.js
      Line: 219419
      Col: 8
      [09:54:44] 'build' errored after 1.32 min
      * xo-web:build − Error: 1
      ✖ 1
      error Command failed with exit code 1.
      info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
      

      (I had a fully working Debian Stretch VM with xen-orchestra, but on the last 'git pull --ff-only' I was forced to upgrade nodejs from 14.6 to 14.7, which failed, so I decided to start from scratch).

      Any ideas?

      DanpD M 2 Replies Last reply Reply Quote 1
      • DanpD Offline
        Danp Pro Support Team @ci4ic4
        last edited by Danp

        @ci4ic4 Have you verified your versions of yarn and node?

        There have been some source code updates since your post, so you could try again with the latest source.

        1 Reply Last reply Reply Quote 0
        • M Offline
          Meth0d
          last edited by

          It fails for me too now (just updated):

          Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
          [11:53:28] Finished 'buildStyles' after 9.46 s
          [11:53:28] Finished 'copyAssets' after 9.47 s
          [11:54:25] 'buildScripts' errored after 1.1 min
          [11:54:25] GulpUglifyError: unable to minify JavaScript
          Caused by: SyntaxError: Unexpected token name «await», expected punc «(»
          File: /opt/xen-orchestra/packages/xo-web/src/index.js
          Line: 219845
          Col: 8
          [11:54:25] 'build' errored after 1.1 min
          * xo-web:build − Error: 1
          ✖ 2
          error Command failed with exit code 1.
          info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
          
          1 Reply Last reply Reply Quote 0
          • M Offline
            Meth0d
            last edited by

            Damn always when you forget to take a snapshot before trying to build from sources it fails 😕

            1 Reply Last reply Reply Quote 0
            • M Offline
              Meth0d
              last edited by

              error xo-server@5.79.5: The engine "node" is incompatible with this module. Expected version ">=14.17". Got "14.16.0"

              Did someone bump that ?

              889838c5-df38-4d02-80f0-308745792694-image.png

              DanpD 1 Reply Last reply Reply Quote 0
              • DanpD Offline
                Danp Pro Support Team @Meth0d
                last edited by

                @meth0d Yes, the requirements were recently changed so that v14.17 or later is required.

                The pertinent phase is "Please use Node LTS version 14". I don't believe the intent is to show the correct / required version when displaying the output of node -v. 😉

                M julien-fJ 2 Replies Last reply Reply Quote 0
                • M Offline
                  Meth0d @Danp
                  last edited by

                  @danp Well I will try to update node later but since you got the exact same error something else seems to be going on.

                  DanpD 1 Reply Last reply Reply Quote 0
                  • DanpD Offline
                    Danp Pro Support Team @Meth0d
                    last edited by

                    Maybe @julien-f can shed some light on the issue.

                    1 Reply Last reply Reply Quote 0
                    • M Offline
                      Meth0d
                      last edited by

                      Hmm something is off here.
                      I grabbed the latest node 14 from nodesource:

                      nodejs is already the newest version (14.17.0-deb-1nodesource1).
                      

                      But:

                      root@xoa:~# node -v
                      v14.16.0
                      
                      1 Reply Last reply Reply Quote 0
                      • julien-fJ Offline
                        julien-f Vates 🪐 Co-Founder XO Team @Danp
                        last edited by

                        Indeed, due to conversion of xo-server to ES modules, Node 14.17 is now required.

                        I've just updated the docs to clarify that we target latest Node LTS.

                        julien-fJ 1 Reply Last reply Reply Quote 0
                        • julien-fJ Offline
                          julien-f Vates 🪐 Co-Founder XO Team @julien-f
                          last edited by

                          I confirm an issue with xo-web though, investigating…

                          A 1 Reply Last reply Reply Quote 0
                          • A Offline
                            AlexD2006 @julien-f
                            last edited by

                            @julien-f : thx for investigating.

                            Tried a fresh install half of the day with the same error and thought i am doing something wrong.
                            Then i tried updateing a running installation and hit the same error.

                            I keep investigating by myself too.

                            julien-fJ 1 Reply Last reply Reply Quote 0
                            • julien-fJ Offline
                              julien-f Vates 🪐 Co-Founder XO Team @AlexD2006
                              last edited by

                              Should now be fixed, sorry for this.

                              A 1 Reply Last reply Reply Quote 2
                              • M Offline
                                Meth0d @ci4ic4
                                last edited by

                                @ci4ic4 how did you manage to update from 14.16 to 14.17 on Debian 10?

                                deb.nodesource.com/setup_14.x just seems to give me 14.16

                                DanpD C 2 Replies Last reply Reply Quote 0
                                • DanpD Offline
                                  Danp Pro Support Team @Meth0d
                                  last edited by

                                  @meth0d I use n lts to install the correct version

                                  M 1 Reply Last reply Reply Quote 1
                                  • M Offline
                                    Meth0d @Danp
                                    last edited by

                                    @danp Thanks that worked.

                                    @julien-f yes seems fixed, thanks.

                                    1 Reply Last reply Reply Quote 0
                                    • C Offline
                                      ci4ic4 @Meth0d
                                      last edited by

                                      @meth0d See the reference sequence provided in the original post. As I said, my reason to go to Buster was my inability to upgrade to nodejs 14.17.

                                      1 Reply Last reply Reply Quote 0
                                      • C Offline
                                        ci4ic4
                                        last edited by

                                        Builds now, thanks for the quick response.

                                        1 Reply Last reply Reply Quote 0
                                        • A Offline
                                          AlexD2006 @julien-f
                                          last edited by

                                          @julien-f
                                          thx for the quick fix. works perfectly now.

                                          1 Reply Last reply Reply Quote 0
                                          • First post
                                            Last post