Skip to content

Add per-step timing information#3716

Merged
smoors merged 6 commits intoeasybuilders:developfrom
Flamefire:timing
Jun 9, 2021
Merged

Add per-step timing information#3716
smoors merged 6 commits intoeasybuilders:developfrom
Flamefire:timing

Conversation

@Flamefire
Copy link
Copy Markdown
Contributor

@Flamefire Flamefire commented May 31, 2021

Shows e.g.:

== fetching files...
== creating build dir, resetting environment...
== ... (took 2 secs)
== unpacking...
== ... (took 15 secs)
== patching...
== ... (took 7 secs)

Requires Python 2.7/3.2

@Flamefire Flamefire force-pushed the timing branch 2 times, most recently from 390b855 to 31b7f85 Compare June 4, 2021 07:35
Flamefire added 5 commits June 8, 2021 14:23
Shows e.g.:
== fetching files...
== creating build dir, resetting environment...
== ... (took 2 secs)
== unpacking...
== ... (took 15 secs)
== patching...
== ... (took 7 secs)
@easybuilders easybuilders deleted a comment from boegelbot Jun 9, 2021
@easybuilders easybuilders deleted a comment from boegelbot Jun 9, 2021
@boegel boegel added this to the next release (4.4.1) milestone Jun 9, 2021
@Flamefire
Copy link
Copy Markdown
Contributor Author

@smoors Maybe you got time for this too?

Copy link
Copy Markdown
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor suggestion

Comment thread easybuild/tools/utilities.py Outdated
Comment thread easybuild/tools/utilities.py
Copy link
Copy Markdown
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Jun 9, 2021

Going in, thanks @Flamefire!

@smoors smoors merged commit 544061e into easybuilders:develop Jun 9, 2021
@Flamefire Flamefire deleted the timing branch June 10, 2021 06:39
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