Skip to content

Commit 317f88b

Browse files
committed
using system_template with join instead of in args
1 parent bd62d59 commit 317f88b

4 files changed

Lines changed: 19 additions & 15 deletions

File tree

.github/workflows/docker.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -212,6 +212,7 @@ jobs:
212212
fi
213213
214214
- name: Summary (manifest)
215+
if: ${{ always() }} # run regardless of earlier step outcomes
215216
shell: bash
216217
env:
217218
IMAGE: ${{ needs.discover.outputs.image }}
@@ -252,6 +253,7 @@ jobs:
252253
ls -l "$GITHUB_WORKSPACE/summaries"
253254
254255
- name: Upload summary artifact (manifest)
256+
if: ${{ always() }} # run regardless of earlier step outcomes
255257
uses: actions/upload-artifact@v4
256258
with:
257259
name: summary-${{ matrix.gemc_tag }}-${{ matrix.geant4_tag }}-${{ matrix.image }}-${{ matrix.image_tag }}-manifest

.github/workflows/sanitize.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ jobs:
7272
echo "Manual build for sanitizer: ${{ matrix.sanitizer }} completed"
7373
7474
- name: Package sanitizer logs
75-
if: ${{ always() }}
75+
if: ${{ always() }} # run regardless of earlier step outcomes
7676
shell: bash
7777
run: |
7878
set -euo pipefail
@@ -98,7 +98,7 @@ jobs:
9898

9999
build-with-sanitizer-final:
100100
name: Sanitize - summary
101-
if: ${{ always() }}
101+
if: ${{ always() }} # run regardless of earlier step outcomes
102102
needs: [ build-with-sanitizer ]
103103
runs-on: ubuntu-latest
104104
strategy:

api/meson.build

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,24 @@
1-
api_dir = get_option('prefix') + '/api/'
1+
api_dir = join_paths(get_option('prefix'), 'api')
22
build_root = meson.project_build_root()
33

44
test_env = environment()
55
test_env.set('PYTHONDONTWRITEBYTECODE', '1')
66
test_env.prepend('PYTHONPATH', api_dir)
77
python_exe = import('python').find_installation('python3')
8+
system_template = join_paths(api_dir, 'system_template.py')
89

910
install_subdir(meson.current_source_dir(), install_dir : '.')
1011

1112
test('api_template_help',
1213
python_exe,
1314
env: test_env,
14-
args : [api_dir + 'system_template.py', '-h'])
15+
args : [system_template, '-h'])
1516

1617
test('api_show_solid_creators',
1718
python_exe,
1819
env: test_env,
19-
args : [api_dir + 'system_template.py', '-sl'])
20+
args : [system_template, '-sl'],
21+
)
2022

2123
# Test all solid creators
2224
g4objects = {
@@ -35,27 +37,27 @@ foreach volume: g4objects.keys()
3537
python_exe,
3638
is_parallel: false,
3739
env: test_env,
38-
args : [api_dir + 'system_template.py', '-gv', volume])
40+
args : [system_template, '-gv', volume])
3941
test('api_template_show_code_for_volume' + volume + '_with_parameters_' + pars.replace(' ', ''),
4042
python_exe,
4143
is_parallel: false,
4244
env: test_env,
43-
args : [api_dir + 'system_template.py', '-gv', volume, '-gvp', pars.replace(' ', '')])
45+
args : [system_template, '-gv', volume, '-gvp', pars.replace(' ', '')])
4446
endforeach
4547

4648
test('api_show_template_code_for_G4Cons',
4749
python_exe,
4850
is_parallel: false,
4951
env: test_env,
50-
args : [api_dir + 'system_template.py', '-gv', 'G4Cons', '-silent'])
52+
args : [system_template, '-gv', 'G4Cons', '-silent'])
5153

5254
test_dir = join_paths(build_root, 'test')
5355
test('api_create_system',
5456
python_exe,
5557
is_parallel: false,
5658
env: test_env,
57-
args : [api_dir + 'system_template.py', '-s', 'test'],
58-
workdir : test_dir,
59+
args : [system_template, '-s', 'test'],
60+
workdir : build_root,
5961
priority : 5)
6062
foreach format : ['ascii', 'sqlite']
6163
test('api_template_build_geometry_in_test_for_format_' + format,
@@ -86,15 +88,15 @@ foreach volume: g4objects.keys()
8688
python_exe,
8789
is_parallel: false,
8890
env: test_env,
89-
args : [api_dir + 'system_template.py', '-s', test_name],
90-
workdir : test_dir,
91+
args : [system_template, '-s', test_name],
92+
workdir : build_root,
9193
priority : 4)
9294
test('api_template_replace_geometry_with' + volume,
9395
python_exe,
9496
is_parallel: false,
9597
env: test_env,
96-
args: [api_dir + 'system_template.py', '-gv', volume, '-gvp', pars, geo_py, sub_name],
97-
workdir : test_dir,
98+
args: [system_template, '-gv', volume, '-gvp', pars, geo_py, sub_name],
99+
workdir : build_root,
98100
priority : -3)
99101
foreach format : ['ascii', 'sqlite']
100102
test('api_template_build_geometry_in_test_of_replacing_geometry_with' + volume + '_with_format' + format,

meson.build

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ if get_option('buildtype') == 'debug'
2020
add_project_arguments('-O2', language : 'cpp')
2121
endif
2222

23-
api_dir = get_option('prefix') + '/api/'
23+
api_dir = join_paths(get_option('prefix'), 'api')
2424
test_env = environment()
2525
test_env.set('PYTHONDONTWRITEBYTECODE', '1')
2626
test_env.prepend('PYTHONPATH', api_dir)

0 commit comments

Comments
 (0)