Skip to content

Comments

Because our test sid file contains EC, don't try it when configured no-ec#2564

Closed
levitte wants to merge 1 commit intoopenssl:masterfrom
levitte:fix-no-ec-20170206
Closed

Because our test sid file contains EC, don't try it when configured no-ec#2564
levitte wants to merge 1 commit intoopenssl:masterfrom
levitte:fix-no-ec-20170206

Conversation

@levitte
Copy link
Member

@levitte levitte commented Feb 6, 2017

Checklist
  • tests are added or updated

@levitte levitte added branch: master Applies to master branch hold labels Feb 6, 2017
@levitte
Copy link
Member Author

levitte commented Feb 6, 2017

Putting this on hold. It's unclear to me if this is the right move. The 1.1.0 branch has the same test with the same file and doesn't fail, so I'm unsure that disabling the test is the right move.

@levitte levitte removed the hold label Feb 8, 2017
@levitte
Copy link
Member Author

levitte commented Feb 8, 2017

Taking this off hold.

@levitte levitte force-pushed the fix-no-ec-20170206 branch from 9611fd6 to f898d80 Compare February 8, 2017 20:49
@levitte
Copy link
Member Author

levitte commented Feb 8, 2017

Merged

68a55f3

@levitte levitte closed this Feb 8, 2017
levitte added a commit that referenced this pull request Feb 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch: master Applies to master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants