Skip to content

Commit

Permalink
Merge PR coq#18264: CI: use --real when generating per file timing ta…
Browse files Browse the repository at this point in the history
…bles

Reviewed-by: ppedrot
Co-authored-by: ppedrot <[email protected]>
  • Loading branch information
coqbot-app[bot] and ppedrot authored Dec 4, 2023
2 parents aba4c65 + c768e3d commit 5ccfab3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions dev/bench/bench.sh
Original file line number Diff line number Diff line change
Expand Up @@ -604,7 +604,7 @@ $skipped_packages"
for iteration in $(seq $num_of_iterations); do
# opam logs prefix the printing with "- " so remove that
# remove the base path for nicer printing and so the script can identify common files
"$program_path"/../../tools/make-both-time-files.py \
"$program_path"/../../tools/make-both-time-files.py --real \
<(sed -e 's/^- //' -e "s:$new_base_path::" "$log_dir/$coq_opam_package.NEW.opam_install.$iteration.stdout.log") \
<(sed -e 's/^- //' -e "s:$old_base_path::" "$log_dir/$coq_opam_package.OLD.opam_install.$iteration.stdout.log") \
> "$log_dir/$coq_opam_package.BOTH.perfile_timings.$iteration.log"
Expand Down Expand Up @@ -654,7 +654,7 @@ du -ha "$working_dir" > "$working_dir/files.listing"
new_base_path=$new_opam_root/ocaml-NEW/.opam-switch/build/
old_base_path=$old_opam_root/ocaml-OLD/.opam-switch/build/
for iteration in $(seq $num_of_iterations); do
"$program_path"/../../tools/make-both-time-files.py \
"$program_path"/../../tools/make-both-time-files.py --real \
<(sed -e 's/^- //' -e "s:$new_base_path::" "$log_dir/"*".NEW.opam_install.$iteration.stdout.log") \
<(sed -e 's/^- //' -e "s:$old_base_path::" "$log_dir/"*".OLD.opam_install.$iteration.stdout.log") \
> "$log_dir/ALL.BOTH.perfile_timings.$iteration.log"
Expand Down
2 changes: 1 addition & 1 deletion dev/ci/ci-wrapper.sh
Original file line number Diff line number Diff line change
Expand Up @@ -17,5 +17,5 @@ export TIMED=1
bash "${DIR}/${CI_SCRIPT}" 2>&1 | tee "$CI_NAME.log"
code=$?
echo 'Aggregating timing log...'
python ./tools/make-one-time-file.py "$CI_NAME.log"
python ./tools/make-one-time-file.py --real "$CI_NAME.log"
exit $code

0 comments on commit 5ccfab3

Please sign in to comment.