Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-50115

Grunt on windows builds the entire tree, every time.

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.9
    • 2.9.5, 3.0.3
    • JavaScript
    • MOODLE_29_STABLE
    • MOODLE_29_STABLE, MOODLE_30_STABLE
    • MDL-50115-master
    • Hide
      1. On windows command prompt do grunt --root=admin/tool/templatelibrary/amd and ensure it only runs only the amd tasks.
      2. Try some other variations and ensure grunt --root=path/to/dir matches the behaviour of cd path/to/dir && grunt (you can't test the non-root alternitve on windows)
      3. Provide a --root which doesn't exist and ensure the job fails gracefully with a fatal error
      4. Repeat the same tests on linux and ensure they match the cd alteratnive
      Show
      On windows command prompt do grunt --root=admin/tool/templatelibrary/amd and ensure it only runs only the amd tasks. Try some other variations and ensure grunt --root=path/to/dir matches the behaviour of cd path/to/dir && grunt (you can't test the non-root alternitve on windows) Provide a --root which doesn't exist and ensure the job fails gracefully with a fatal error Repeat the same tests on linux and ensure they match the cd alteratnive

    Description

      On linux it will let you run grunt in a subdirectory and it will only build that directory. On windows we cant do this easily because grunt hides the current working directory.

      I think it's possible to get around by requiring windows devs using "grunt -b ." to change the base directory - which is what this issue is about.

      Attachments

        Issue Links

          Activity

            People

              poltawski Dan Poltawski
              damyon Damyon Wiese
              Andrew Lyons Andrew Lyons
              David Monllaó David Monllaó
              John Okely John Okely
              David Woloszyn, Huong Nguyen, Jake Dallimore, Michael Hawkins, Stevani Andolo
              Votes:
              2 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                14/Mar/16