Skip to content

Commit 2b2b22b

Browse files
authored
ci(release): clean bin directory before upload (MODFLOW-ORG#2691)
The bin/ folder in the distribution included some extra exes installed for testing, remove them. And some other minor fixes
1 parent 6d7c83a commit 2b2b22b

File tree

1 file changed

+23
-9
lines changed

1 file changed

+23
-9
lines changed

.github/workflows/release.yml

Lines changed: 23 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -113,14 +113,14 @@ jobs:
113113
ref: ${{ inputs.branch }}
114114

115115
- name: Checkout modflow6-testmodels
116-
if: inputs.run_tests == true && runner.os != 'Windows'
116+
if: inputs.run_tests == true && !(runner.os == 'Windows' && matrix.extended)
117117
uses: actions/checkout@v6
118118
with:
119119
repository: MODFLOW-ORG/modflow6-testmodels
120120
path: modflow6-testmodels
121121

122122
- name: Checkout modflow6-examples
123-
if: inputs.run_tests == true && runner.os != 'Windows'
123+
if: inputs.run_tests == true && !(runner.os == 'Windows' && matrix.extended)
124124
uses: actions/checkout@v6
125125
with:
126126
repository: MODFLOW-ORG/modflow6-examples
@@ -173,7 +173,7 @@ jobs:
173173
eval "$cmd"
174174
175175
- name: Update flopy
176-
if: inputs.run_tests == true && runner.os != 'Windows'
176+
if: inputs.run_tests == true && !(runner.os == 'Windows' && matrix.extended)
177177
working-directory: modflow6
178178
run: |
179179
cmd="pixi run update-flopy"
@@ -276,8 +276,17 @@ jobs:
276276
working-directory: modflow6/bin
277277
run: |
278278
otool -L mf6
279+
otool -L mf6.dylib
280+
otool -L mf5to6
281+
otool -L zbud6
279282
lipo -info mf6
283+
lipo -info mf6.dylib
284+
lipo -info mf5to6
285+
lipo -info zbud6
280286
287+
# TODO remove after next executables release. only necessary because
288+
# tests using gridgen were failing as the current gridgen depends on
289+
# some dylibs. new executables all statically link gcc/gfortran libs
281290
- name: Restore dylibs (macOS)
282291
if: runner.os == 'macOS'
283292
run: |
@@ -289,18 +298,23 @@ jobs:
289298
mv $libpath/libquadmath.0.dylib.bak $libpath/libquadmath.0.dylib
290299
291300
- name: Unit test MF6
292-
# temporarily skip macOS.. why don't these fail in the normal CI?
301+
# temporarily skip on macOS.. why don't these fail in the normal CI?
293302
if: (!(runner.os == 'Windows' && matrix.extended) && !(runner.os == 'macOS'))
294303
working-directory: modflow6
295304
run: pixi run test builddir
296305

297306
- name: Get executables
298-
if: inputs.run_tests == true && runner.os != 'Windows'
307+
if: inputs.run_tests == true && !(runner.os == 'Windows' && matrix.extended)
299308
working-directory: modflow6
300309
env:
301310
GITHUB_TOKEN: ${{ github.token }}
302311
run: pixi run get-exes -s
303312

313+
- name: Clean bin directory
314+
if: inputs.run_tests == true && !(runner.os == 'Windows' && matrix.extended)
315+
working-directory: modflow6/bin
316+
run: rm -rf rebuilt downloaded
317+
304318
- name: Build binaries (Windows)
305319
if: runner.os == 'Windows' && matrix.extended && !inputs.run_tests
306320
uses: ./modflow6/.github/actions/build-extended-win
@@ -360,21 +374,21 @@ jobs:
360374
path: modflow6/bin
361375

362376
- name: Test MF6
363-
if: inputs.run_tests == true && runner.os != 'Windows'
377+
if: inputs.run_tests == true && !(runner.os == 'Windows' && matrix.extended)
364378
working-directory: modflow6
365379
env:
366380
REPOS_PATH: ${{ github.workspace }}
367381
run: pixi run autotest -m "${{ steps.set_markers.outputs.markers }} and not external" -k "${{ steps.set_filters.outputs.filters }}"
368382

369383
- name: Test MF6 models
370-
if: inputs.run_tests == true && runner.os != 'Windows'
384+
if: inputs.run_tests == true && !(runner.os == 'Windows' && matrix.extended)
371385
working-directory: modflow6
372386
env:
373387
REPOS_PATH: ${{ github.workspace }}
374388
run: pixi run autotest -m "${{ steps.set_markers.outputs.markers }} and external" -k "${{ steps.set_filters.outputs.filters }}" --models-path ../../modflow6-testmodels/mf6
375389

376390
- name: Test converter models
377-
if: inputs.run_tests == true && runner.os != 'Windows'
391+
if: inputs.run_tests == true && !(runner.os == 'Windows' && matrix.extended)
378392
working-directory: modflow6
379393
env:
380394
REPOS_PATH: ${{ github.workspace }}
@@ -412,7 +426,7 @@ jobs:
412426
run: sudo ./install.sh --all-users
413427

414428
- name: Test distribution scripts
415-
if: ${{ inputs.run_tests == true && runner.os != 'Windows' }}
429+
if: ${{ inputs.run_tests == true && !(runner.os == 'Windows' && matrix.extended) }}
416430
working-directory: modflow6
417431
env:
418432
GITHUB_TOKEN: ${{ github.token }}

0 commit comments

Comments
 (0)