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