Skip to content

[CircleCI] Fix short-tests-0#2892

Merged
terrelln merged 1 commit intofacebook:devfrom
terrelln:issue-2785
Dec 2, 2021
Merged

[CircleCI] Fix short-tests-0#2892
terrelln merged 1 commit intofacebook:devfrom
terrelln:issue-2785

Conversation

@terrelln
Copy link
Contributor

@terrelln terrelln commented Dec 1, 2021

short-tests-0 were silently failing. I think because of the && make clean construction. Switch to ; instead.

Fix the reported build errors.

@terrelln
Copy link
Contributor Author

terrelln commented Dec 1, 2021

So the aarch64 compiler error is still there. But, it turns out that our CircleCI tests are failing silently. Likely because they don't have a trailing && after make clean.

@terrelln terrelln force-pushed the issue-2785 branch 2 times, most recently from 77caec4 to 58ef167 Compare December 1, 2021 22:17
@terrelln terrelln changed the title [test] Re-enable aarch64build test [CircleCI] Fix short-tests-0 Dec 1, 2021
@terrelln terrelln force-pushed the issue-2785 branch 3 times, most recently from 6bca589 to 91f5891 Compare December 2, 2021 01:38
short-tests-0 were silently failing. I think because of the && make clean construction. Switch to ; instead.

Also fix all the test failures that were exposed.

`make all` is failing on CircleCI because it is missing Docker. Move that test
to GitHub actions, and switch the pedantic CircleCI test to `make allmost`.
@terrelln terrelln merged commit 01ecd6f into facebook:dev Dec 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants