Skip to content

Commit

Permalink
Removes time measurement from the Makefile (#9305)
Browse files Browse the repository at this point in the history
`time` is not a valid command on Windows. This helps to setup the
environment on the Windows platform. We currently not monitoring the
built time so we can add it back once we have requests.

Issues fixed (partially) by this PR:

- #6443
- #8272
  • Loading branch information
AlexV525 authored Aug 26, 2023
1 parent 38a6e5f commit cb60593
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ emulate:
# WARNING this can take a while to run!
# Usage: `make test FLUTTER_TEST_BRANCH=<channel>`
test:
DOCKER_BUILDKIT=1 time docker build --rm --target tests -t flt-test .
DOCKER_BUILDKIT=1 docker build --rm --target tests -t flt-test .
docker run --rm --name flt-tests -t flt-test --target ${FLUTTER_TEST_BRANCH}

# Stop long running tests
Expand All @@ -110,13 +110,13 @@ debug-tests:
# those will be run on the Github action.
# Usage: `make build-image`
build-image:
DOCKER_BUILDKIT=1 time docker build --rm --no-cache --target build \
DOCKER_BUILDKIT=1 docker build --rm --no-cache --target build \
--build-arg BUILD_CONFIGS=${BUILD_CONFIGS} -t ${BUILD_TAG} .

# Build the production site & Run the link checker
# Usage: `make check-links`
build-image-and-check-links:
DOCKER_BUILDKIT=1 time docker build --rm --no-cache --target checklinks \
DOCKER_BUILDKIT=1 docker build --rm --no-cache --target checklinks \
--build-arg BUILD_CONFIGS=${BUILD_CONFIGS} -t ${BUILD_TAG} .
docker run --rm -t ${BUILD_TAG}

Expand Down

0 comments on commit cb60593

Please sign in to comment.