Skip to content

Conversation

@mloskot
Copy link
Member

@mloskot mloskot commented Nov 21, 2018

Disable headers tests on CI services and during regression tests.
Refines #525

@awulkiew
Copy link
Member

awulkiew commented Nov 21, 2018

What do you think about simply removing this script and relying on manually running tests in self_contained_headers directory? This way one wouldn't have to create any environment variable so the command would be shorter.

@awulkiew awulkiew added the tests label Nov 21, 2018
@awulkiew awulkiew added this to the 1.69 milestone Nov 21, 2018
@mloskot
Copy link
Member Author

mloskot commented Nov 21, 2018

I planned to add a CI build job on Travis CI or AppVeyor Linux where I would define TEST_HEADERS.
I can equally run cd self_contained_headers && b2.

So, no objection to remove it.

I will update this PR.

@mloskot mloskot self-assigned this Nov 21, 2018
Disable headers tests on CI services and during regression tests.
Refines boostorg#525
@mloskot mloskot force-pushed the ml/headers-test-on-demand branch from c46c2bc to 146f80e Compare November 21, 2018 15:28
@mloskot
Copy link
Member Author

mloskot commented Nov 21, 2018

@awulkiew I see you have added 1.69 milestone. Since you aim to merge this into master too, I will leave merge of this to you.

@mloskot mloskot assigned awulkiew and unassigned mloskot Nov 21, 2018
@awulkiew
Copy link
Member

Thanks.

@awulkiew awulkiew merged commit b432f28 into boostorg:develop Nov 21, 2018
@mloskot mloskot deleted the ml/headers-test-on-demand branch December 10, 2018 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants