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

Grunt on windows builds the entire tree, every time.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.9
    • Fix Version/s: 2.9.5, 3.0.3
    • Component/s: JavaScript
    • Labels:
    • Testing Instructions:
      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
    • Affected Branches:
      MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_29_STABLE, MOODLE_30_STABLE
    • Pull Master Branch:
      MDL-50115-master

      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

              • Votes:
                2 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/Mar/16