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

    'yarn build' failure

    Scheduled Pinned Locked Moved Xen Orchestra
    19 Posts 5 Posters 3.6k 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

                                          Hello! It looks like you're interested in this conversation, but you don't have an account yet.

                                          Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.

                                          With your input, this post could be even better 💗

                                          Register Login
                                          • First post
                                            Last post