Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

speed up fizz github CI by using the system deps #95

Closed
wants to merge 1 commit into from

Conversation

ahornby
Copy link
Contributor

@ahornby ahornby commented Jul 21, 2023

speed up fizz github CI by using the system deps so that boost & cmake etc aren't rebuilt each time

Summary:

In previous changed noticed the CI was quite slow as it was building things like boost every time. Regenerated it with:

./build/fbcode_builder/getdeps.py --allow-system-packages generate-github-actions --src-dir=. --output-dir=.github/workflows fizz

Test Plan:

CI

Summary:

In previous changed noticed the CI was quite slow as it was building things like boost every time.  Regenerated it with:

```
./build/fbcode_builder/getdeps.py --allow-system-packages generate-github-actions --src-dir=. --output-dir=.github/workflows fizz
```

Test Plan:

CI
@ahornby
Copy link
Contributor Author

ahornby commented Mar 13, 2024

cc @mingtaoy looks like this speeds up the github CI quite a lot

@bigfootjon bigfootjon requested a review from mingtaoy March 15, 2024 13:23
@facebook-github-bot
Copy link
Contributor

@mingtaoy has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot pushed a commit to facebook/hhvm that referenced this pull request Mar 15, 2024
Summary:
speed up fizz github CI by using the system deps so that boost & cmake etc aren't rebuilt each time

In previous changed noticed the CI was quite slow as it was building things like boost every time.  Regenerated it with:

```
./build/fbcode_builder/getdeps.py --allow-system-packages generate-github-actions --src-dir=. --output-dir=.github/workflows fizz
```

X-link: facebookincubator/fizz#95

Reviewed By: bigfootjon

Differential Revision: D54950338

Pulled By: mingtaoy

fbshipit-source-id: f14f90c2acca6bab7a8353836a6e492f1b14dc55
@facebook-github-bot
Copy link
Contributor

@mingtaoy merged this pull request in 9bf8074.

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