This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Re-enable macOS back-deployment testing
ClosedPublic

Authored by ldionne on Mar 25 2021, 11:15 AM.

Details

Reviewers
None
Group Reviewers
Restricted Project
Commits
rG478d1eded229: [libc++] Re-enable macOS back-deployment testing
Summary

Download older roots from Dropbox instead of Green Dragon, which is too
unreliable.

Diff Detail

Event Timeline

ldionne created this revision.Mar 25 2021, 11:15 AM
ldionne requested review of this revision.Mar 25 2021, 11:15 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 25 2021, 11:15 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne updated this revision to Diff 333540.Mar 26 2021, 5:23 AM

Fix tests broken for back-deployment.

ldionne accepted this revision as: Restricted Project.Mar 26 2021, 5:24 AM
ldionne added a subscriber: Quuxplusone.
ldionne added inline comments.
libcxx/test/std/input.output/filesystems/class.directory_entry/directory_entry.obs/last_write_time.pass.cpp
13

@Quuxplusone In case you have a better idea.

mstorsjo added inline comments.
libcxx/test/std/input.output/filesystems/class.directory_entry/directory_entry.obs/last_write_time.pass.cpp
13

I guess it'd be nice to make the checks for error strings a bit more lenient, to match both older and newer versions of the lib.

ldionne added inline comments.Mar 26 2021, 6:41 AM
libcxx/test/std/input.output/filesystems/class.directory_entry/directory_entry.obs/last_write_time.pass.cpp
13

It'd be nice to match old and new versions, but not weaken the tests too much nor add too much complexity. I looked into fixing this in check_libcxx_string(), but it's not clear to me it's worth the complexity.

This revision was not accepted when it landed; it landed in state Needs Review.Mar 29 2021, 7:09 PM
This revision was automatically updated to reflect the committed changes.