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

i#4083 func view: Allow for 7-digit thread id in func_view tests #4454

Merged
merged 1 commit into from
Sep 28, 2020

Conversation

derekbruening
Copy link
Contributor

After an upgdate, my machine has 7-digit thread id's, causing the
func_view tests to fail. Here I widen the printed field and update
the expected output for the stored trace.

Issue: #4083

After an upgdate, my machine has 7-digit thread id's, causing the
func_view tests to fail.  Here I widen the printed field and update
the expected output for the stored trace.

Issue: #4083
@derekbruening
Copy link
Contributor Author

The Travis failure is on Mac and is rather puzzling. I'm assuming it's unrelated to this PR and it's a Travis XCode upgrade or sthg? But Mac built for PR #4453 just prior to this.

debug-internal-64: **** 4 build errors ****
	/.../dynamorio/build_debug-internal-64/suite/tests/decode.c_asm.asm:554: error: mismatch in operand sizes

@derekbruening
Copy link
Contributor Author

I re-ran just the Mac job and the error went away...?!? Well unfortunately we do not have resources to figure out strange non-reproducible issues like that. Moving on.

@derekbruening derekbruening merged commit 25ca6af into master Sep 28, 2020
@derekbruening derekbruening deleted the i4083-long-thread-id branch September 28, 2020 20:47
derekbruening added a commit that referenced this pull request Sep 29, 2020
After an upgdate, my machine has 7-digit thread id's, causing the
func_view tests to fail.  Here I widen the printed field and update
the expected output for the stored trace.

Issue: #4083
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