Skip to content
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

Update PolarSSL to current stable version #102

Merged
merged 4 commits into from Feb 24, 2014

Conversation

Tilka
Copy link
Member

@Tilka Tilka commented Feb 23, 2014

Some of these commits can obviously be squashed but I'll leave them for now for easier review.

Compiles on Linux and Windows, nothing else tested yet.

@delroth
Copy link
Member

delroth commented Feb 23, 2014

Please buildbot-try this PR.

@Tilka
Copy link
Member Author

Tilka commented Feb 23, 2014

I can't, it's too large.

@delroth
Copy link
Member

delroth commented Feb 23, 2014

At least try an OSX build then.

@Tilka
Copy link
Member Author

Tilka commented Feb 23, 2014

21:50:07 pauldachz> flacs: Currently building your PolarSSL PR on OS X for your convenience.
[...]
22:04:46 pauldachz> flacs: That PR builds fine, no extra warnings or anything.

@Tilka
Copy link
Member Author

Tilka commented Feb 24, 2014

I'll go ahead and squash this.

I just removed Externals/polarssl/, added the new version, then deleted
the following files/directories:

DartConfiguration.tcl
Makefile
doxygen/
library/Makefile
programs/
scripts/
tests/
visualc/
- strip down PolarSSL's CMakeLists.txt
- switch to the PolarSSL 1.3 API
- use entropy interface instead of havege (PolarSSL 1.3 has disabled
  havege by default because it is "considered unsafe for primary usage")
- add VS2013 .vcxproj file
@Parlane
Copy link
Member

Parlane commented Feb 24, 2014

Please add a version check

Matthew Parlane
On 25/02/2014 2:55 AM, "Tillmann Karras" notifications@github.com wrote:

I'll go ahead an squash this.

Reply to this email directly or view it on GitHubhttps://github.com//pull/102#issuecomment-35887875
.

@Tilka
Copy link
Member Author

Tilka commented Feb 24, 2014

OK, I think this PR is merge-ready.

EDIT:
@Parlane: will do

delroth added a commit that referenced this pull request Feb 24, 2014
Update PolarSSL to current stable version
@delroth delroth merged commit 6959a93 into dolphin-emu:master Feb 24, 2014
@Tilka Tilka deleted the polarssl1.3 branch February 24, 2014 19:10
TellowKrinkle pushed a commit to TellowKrinkle/dolphin that referenced this pull request Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants