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