Skip to content

Disable access_marker tests while I fix the ASAN failure. #8681

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

Merged
merged 1 commit into from
Apr 11, 2017
Merged

Disable access_marker tests while I fix the ASAN failure. #8681

merged 1 commit into from
Apr 11, 2017

Conversation

atrick
Copy link
Contributor

@atrick atrick commented Apr 11, 2017

This is obviously a silly iterator invalidation thing, but will require a little
debugging.

https://ci.swift.org/job/oss-swift-incremental-ASAN-RA-osx/402/consoleFull

This is obviously a silly iterator invalidation thing, but will require a little
debugging.
@atrick
Copy link
Contributor Author

atrick commented Apr 11, 2017

@swift-ci smoke test and merge.

@swift-ci swift-ci merged commit b819ef4 into swiftlang:master Apr 11, 2017
@jrose-apple
Copy link
Contributor

Don't REQUIRES lines have to be valid identifiers now?

@atrick
Copy link
Contributor Author

atrick commented Apr 11, 2017

@jrose-apple Do REQUIRES lines have requirements? lit is all trial-and-error for me. Writing a random comment seemed to work well enough.

@jrose-apple
Copy link
Contributor

It might only be in master-next, but then the master-next people will be upset with you. cc @compnerd , @bob-wilson

@bob-wilson
Copy link
Contributor

Yeah, that will break. Can you remove the colon after "rdar" and put the other text on a separate comment line?

@atrick
Copy link
Contributor Author

atrick commented Apr 12, 2017

@bob-wilson this was just for future reference. I already removed those lines this morning. Just been waiting on PR testing to pass.

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.

4 participants