Uploaded image for project: 'GPII - Global Public Inclusive Infrastructure'
  1. GPII - Global Public Inclusive Infrastructure
  2. GPII-1327

Grunt build fails for Windows when universal repo exists

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Windows Implementation
    • Labels:
      None

      Description

      In fixing #GPII-1000 I am working with locally checked out windows and universal but once the universal repo exists in the workspace it is impossible to build and run gpii.

      This may be a matter of it only works if you do things in the correct order. As my code was not working I switched back to master branch on both, merged upstream, removed the universal node-modules (not the parent of universal obviously and not windows as that gets checked out) and ran the npm install and grunt build steps.

      <pre>
      $ grunt build
      Running "build" task

      Running "gpii-universal" task

      Running "shell:git-clone" (shell) task
      fatal: destination path '../node_modules/universal' already exists and is not an
      empty directory.
      Warning: Command failed: fatal: destination path '../node_modules/universal' alr
      eady exists and is not an empty directory.
      Use --force to continue.

      Aborted due to warnings.
      </pre>

      This error shouldn't actually matter as it's just moaning about universal/ already existing. But the script should continue, not bail.

      How do others develop changes to universal on windows?

      I'll attach as patch rather than a PR as this file gets created by the @npm install rather than being in the repo and I haven't figured the source yet.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              amb26 Antranig Basman
              Reporter:
              stevelee Steve Lee
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: