[PATCH] D22310: Make the test for fno-pch-timestamp compatible with read-only checkouts.

2016-07-13 Thread pierre gousseau via cfe-commits
pgousseau created this revision. pgousseau added a reviewer: rsmith. pgousseau added subscribers: cfe-commits, trybka, bruno, gbedwell, wristow, probinson. The test added in r275267 does not work on read-only checkouts because of the use of touch -t. Following Tom Rybka suggestion, the test file

Re: [PATCH] D22310: Make the test for fno-pch-timestamp compatible with read-only checkouts.

2016-07-14 Thread Benjamin Kramer via cfe-commits
bkramer added a subscriber: bkramer. bkramer accepted this revision. bkramer added a reviewer: bkramer. bkramer added a comment. This revision is now accepted and ready to land. lg https://reviews.llvm.org/D22310 ___ cfe-commits mailing list cfe-com

Re: [PATCH] D22310: Make the test for fno-pch-timestamp compatible with read-only checkouts.

2016-07-14 Thread pierre gousseau via cfe-commits
pgousseau added a comment. In https://reviews.llvm.org/D22310#484017, @bkramer wrote: > lg Thanks for the review! https://reviews.llvm.org/D22310 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listin

Re: [PATCH] D22310: Make the test for fno-pch-timestamp compatible with read-only checkouts.

2016-07-14 Thread pierre gousseau via cfe-commits
pgousseau closed this revision. pgousseau added a comment. Committed in https://reviews.llvm.org/rL275415 https://reviews.llvm.org/D22310 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm