Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make upload job concurrent with lldb job #14

Merged
merged 3 commits into from
Aug 17, 2024
Merged

Conversation

vtjnash
Copy link
Member

@vtjnash vtjnash commented Jun 21, 2024

No description provided.

@vtjnash vtjnash requested a review from staticfloat June 21, 2024 15:15
hooks/post-command Outdated Show resolved Hide resolved
hooks/post-command Outdated Show resolved Hide resolved
hooks/post-command Outdated Show resolved Hide resolved
@@ -32,22 +41,12 @@ if [[ "${#CORE_DUMPS[@]}" > 0 ]]; then
(sleep "${DEBUGGER_TIMEOUT}"; kill "${DBG_PID}" 2>/dev/null >/dev/null; echo "DEBUGGER KILLED BY WATCHDOG TIMER") &

# Wait for one debugging command to finish before starting the next
wait "${DBG_PID}" || true
wait -f "${DBG_PID}" || true
done
Copy link
Member Author

Choose a reason for hiding this comment

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

If this async printing isn't reliable enough for you, we could also pipe this through dd like so:

Suggested change
done
done | dd bs=64k status=none

such that all of the content printed here gets buffered via dd up to to the specified chunk size (64k) to make it atomic (and the same for each upload upon success of it as well)

@staticfloat staticfloat merged commit 3610c81 into main Aug 17, 2024
1 check passed
@staticfloat
Copy link
Member

good enough for me, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants