Skip to content

Commit 9755d6f

Browse files
ahornbyfacebook-github-bot
authored andcommitted
skip unnecessary github actions steps
Summary: X-link: facebookincubator/zstrong#1100 Update generated github actions to only run the fetch and and build steps when there are sources expected for a manifest For local github actions testing using `act` this speeds up the test runs, and in real github CI it makes it clearer which steps are actually doing something on the given runner (we don't know exactly what it has installed beforehand) Also set the windows git config the same as on internal CI Reviewed By: bigfootjon Differential Revision: D67839708 fbshipit-source-id: 0a60c6fc89e8c6abb2464f879459aa23d5aec969
1 parent dcaa9b1 commit 9755d6f

File tree

5 files changed

+266
-41
lines changed

5 files changed

+266
-41
lines changed

.github/workflows/getdeps_linux.yml

+70-3
Original file line numberDiff line numberDiff line change
@@ -27,140 +27,207 @@ jobs:
2727
- name: Update system package info
2828
run: sudo apt-get update
2929
- name: Install system deps
30-
run: sudo python3 build/fbcode_builder/getdeps.py --allow-system-packages install-system-deps --recursive fbthrift
31-
- name: Install packaging system deps
32-
run: sudo python3 build/fbcode_builder/getdeps.py --allow-system-packages install-system-deps --recursive patchelf
30+
run: sudo python3 build/fbcode_builder/getdeps.py --allow-system-packages install-system-deps --recursive fbthrift && sudo python3 build/fbcode_builder/getdeps.py --allow-system-packages install-system-deps --recursive patchelf
31+
- id: paths
32+
name: Query paths
33+
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages query-paths --recursive --src-dir=. fbthrift >> "$GITHUB_OUTPUT"
3334
- name: Fetch xxhash
35+
if: ${{ steps.paths.outputs.xxhash_SOURCE }}
3436
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests xxhash
3537
- name: Fetch ninja
38+
if: ${{ steps.paths.outputs.ninja_SOURCE }}
3639
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests ninja
3740
- name: Fetch cmake
41+
if: ${{ steps.paths.outputs.cmake_SOURCE }}
3842
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests cmake
3943
- name: Fetch zlib
44+
if: ${{ steps.paths.outputs.zlib_SOURCE }}
4045
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests zlib
4146
- name: Fetch zstd
47+
if: ${{ steps.paths.outputs.zstd_SOURCE }}
4248
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests zstd
4349
- name: Fetch fmt
50+
if: ${{ steps.paths.outputs.fmt_SOURCE }}
4451
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests fmt
4552
- name: Fetch boost
53+
if: ${{ steps.paths.outputs.boost_SOURCE }}
4654
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests boost
4755
- name: Fetch double-conversion
56+
if: ${{ steps.paths.outputs.double-conversion_SOURCE }}
4857
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests double-conversion
4958
- name: Fetch fast_float
59+
if: ${{ steps.paths.outputs.fast_float_SOURCE }}
5060
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests fast_float
5161
- name: Fetch gflags
62+
if: ${{ steps.paths.outputs.gflags_SOURCE }}
5263
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests gflags
5364
- name: Fetch glog
65+
if: ${{ steps.paths.outputs.glog_SOURCE }}
5466
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests glog
5567
- name: Fetch googletest
68+
if: ${{ steps.paths.outputs.googletest_SOURCE }}
5669
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests googletest
5770
- name: Fetch libdwarf
71+
if: ${{ steps.paths.outputs.libdwarf_SOURCE }}
5872
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests libdwarf
5973
- name: Fetch libevent
74+
if: ${{ steps.paths.outputs.libevent_SOURCE }}
6075
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests libevent
6176
- name: Fetch lz4
77+
if: ${{ steps.paths.outputs.lz4_SOURCE }}
6278
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests lz4
6379
- name: Fetch snappy
80+
if: ${{ steps.paths.outputs.snappy_SOURCE }}
6481
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests snappy
6582
- name: Fetch bz2
83+
if: ${{ steps.paths.outputs.bz2_SOURCE }}
6684
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests bz2
6785
- name: Fetch openssl
86+
if: ${{ steps.paths.outputs.openssl_SOURCE }}
6887
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests openssl
6988
- name: Fetch liboqs
89+
if: ${{ steps.paths.outputs.liboqs_SOURCE }}
7090
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests liboqs
7191
- name: Fetch autoconf
92+
if: ${{ steps.paths.outputs.autoconf_SOURCE }}
7293
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests autoconf
7394
- name: Fetch automake
95+
if: ${{ steps.paths.outputs.automake_SOURCE }}
7496
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests automake
7597
- name: Fetch libtool
98+
if: ${{ steps.paths.outputs.libtool_SOURCE }}
7699
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests libtool
77100
- name: Fetch libsodium
101+
if: ${{ steps.paths.outputs.libsodium_SOURCE }}
78102
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests libsodium
79103
- name: Fetch libiberty
104+
if: ${{ steps.paths.outputs.libiberty_SOURCE }}
80105
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests libiberty
81106
- name: Fetch libunwind
107+
if: ${{ steps.paths.outputs.libunwind_SOURCE }}
82108
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests libunwind
83109
- name: Fetch xz
110+
if: ${{ steps.paths.outputs.xz_SOURCE }}
84111
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests xz
85112
- name: Fetch folly
113+
if: ${{ steps.paths.outputs.folly_SOURCE }}
86114
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests folly
87115
- name: Fetch fizz
116+
if: ${{ steps.paths.outputs.fizz_SOURCE }}
88117
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests fizz
89118
- name: Fetch mvfst
119+
if: ${{ steps.paths.outputs.mvfst_SOURCE }}
90120
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests mvfst
91121
- name: Fetch libffi
122+
if: ${{ steps.paths.outputs.libffi_SOURCE }}
92123
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests libffi
93124
- name: Fetch ncurses
125+
if: ${{ steps.paths.outputs.ncurses_SOURCE }}
94126
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests ncurses
95127
- name: Fetch python
128+
if: ${{ steps.paths.outputs.python_SOURCE }}
96129
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests python
97130
- name: Fetch wangle
131+
if: ${{ steps.paths.outputs.wangle_SOURCE }}
98132
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages fetch --no-tests wangle
99133
- name: Build xxhash
134+
if: ${{ steps.paths.outputs.xxhash_SOURCE }}
100135
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests xxhash
101136
- name: Build ninja
137+
if: ${{ steps.paths.outputs.ninja_SOURCE }}
102138
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests ninja
103139
- name: Build cmake
140+
if: ${{ steps.paths.outputs.cmake_SOURCE }}
104141
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests cmake
105142
- name: Build zlib
143+
if: ${{ steps.paths.outputs.zlib_SOURCE }}
106144
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests zlib
107145
- name: Build zstd
146+
if: ${{ steps.paths.outputs.zstd_SOURCE }}
108147
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests zstd
109148
- name: Build fmt
149+
if: ${{ steps.paths.outputs.fmt_SOURCE }}
110150
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests fmt
111151
- name: Build boost
152+
if: ${{ steps.paths.outputs.boost_SOURCE }}
112153
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests boost
113154
- name: Build double-conversion
155+
if: ${{ steps.paths.outputs.double-conversion_SOURCE }}
114156
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests double-conversion
115157
- name: Build fast_float
158+
if: ${{ steps.paths.outputs.fast_float_SOURCE }}
116159
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests fast_float
117160
- name: Build gflags
161+
if: ${{ steps.paths.outputs.gflags_SOURCE }}
118162
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests gflags
119163
- name: Build glog
164+
if: ${{ steps.paths.outputs.glog_SOURCE }}
120165
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests glog
121166
- name: Build googletest
167+
if: ${{ steps.paths.outputs.googletest_SOURCE }}
122168
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests googletest
123169
- name: Build libdwarf
170+
if: ${{ steps.paths.outputs.libdwarf_SOURCE }}
124171
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests libdwarf
125172
- name: Build libevent
173+
if: ${{ steps.paths.outputs.libevent_SOURCE }}
126174
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests libevent
127175
- name: Build lz4
176+
if: ${{ steps.paths.outputs.lz4_SOURCE }}
128177
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests lz4
129178
- name: Build snappy
179+
if: ${{ steps.paths.outputs.snappy_SOURCE }}
130180
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests snappy
131181
- name: Build bz2
182+
if: ${{ steps.paths.outputs.bz2_SOURCE }}
132183
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests bz2
133184
- name: Build openssl
185+
if: ${{ steps.paths.outputs.openssl_SOURCE }}
134186
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests openssl
135187
- name: Build liboqs
188+
if: ${{ steps.paths.outputs.liboqs_SOURCE }}
136189
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests liboqs
137190
- name: Build autoconf
191+
if: ${{ steps.paths.outputs.autoconf_SOURCE }}
138192
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests autoconf
139193
- name: Build automake
194+
if: ${{ steps.paths.outputs.automake_SOURCE }}
140195
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests automake
141196
- name: Build libtool
197+
if: ${{ steps.paths.outputs.libtool_SOURCE }}
142198
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests libtool
143199
- name: Build libsodium
200+
if: ${{ steps.paths.outputs.libsodium_SOURCE }}
144201
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests libsodium
145202
- name: Build libiberty
203+
if: ${{ steps.paths.outputs.libiberty_SOURCE }}
146204
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests libiberty
147205
- name: Build libunwind
206+
if: ${{ steps.paths.outputs.libunwind_SOURCE }}
148207
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests libunwind
149208
- name: Build xz
209+
if: ${{ steps.paths.outputs.xz_SOURCE }}
150210
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests xz
151211
- name: Build folly
212+
if: ${{ steps.paths.outputs.folly_SOURCE }}
152213
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests folly
153214
- name: Build fizz
215+
if: ${{ steps.paths.outputs.fizz_SOURCE }}
154216
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests fizz
155217
- name: Build mvfst
218+
if: ${{ steps.paths.outputs.mvfst_SOURCE }}
156219
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests mvfst
157220
- name: Build libffi
221+
if: ${{ steps.paths.outputs.libffi_SOURCE }}
158222
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests libffi
159223
- name: Build ncurses
224+
if: ${{ steps.paths.outputs.ncurses_SOURCE }}
160225
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests ncurses
161226
- name: Build python
227+
if: ${{ steps.paths.outputs.python_SOURCE }}
162228
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests python
163229
- name: Build wangle
230+
if: ${{ steps.paths.outputs.wangle_SOURCE }}
164231
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --free-up-disk --no-tests wangle
165232
- name: Build fbthrift
166233
run: python3 build/fbcode_builder/getdeps.py --allow-system-packages build --src-dir=. fbthrift --project-install-prefix fbthrift:/usr/local

0 commit comments

Comments
 (0)