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.
    • 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