Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

Build: fix MinGW-w64 build. Closes #36 #435

Closed
wants to merge 4 commits into from

Conversation

anonimal
Copy link
Collaborator

@anonimal anonimal commented Nov 5, 2016

By submitting this pull-request, I confirm the following:

  • I have read and understood the Contributing Guide.
  • I have checked that another pull-request for this purpose does not exist.
  • I have considered and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used and that this pull-request may be closed by the will of the maintainer.
  • I give this submission freely under the BSD 3-clause license.

Place an X inside the bracket (or click the box in preview) to confirm

  • I confirm.

Based off #431 (of which will be merged once armv8 box is back online).

References #36 and cpp-netlib/cpp-netlib#705

…ect#285

- Fixes CMake for ARM, add compile-time patches for run-time AES-NI
- Preprocessor: also added MSVS macro for future MSVS testing
- Cleaned-up CMake / shared hardening flags with Clang
- Completely replaces our current (dismal) recipe
- Courtesy of Mark Vejvoda (2011) and monero-project/monero
There have been repeated needs to build against the patches that I send
upstream. As is common practice, by pointing our submodules to my forks,
we can base against fixes without having to wait for upstream to merge
pulls.

This most recently applies to cpp-netlib/cpp-netlib#705 (thus why this
commit is in this branch)

References monero-project#36
@anonimal
Copy link
Collaborator Author

anonimal commented Nov 7, 2016

pigeons | anonimal: to trigger the #435 build again for the platforms (windows) that weren't configured to report to github at the time the PR was made, would you close the PR and re-open it?
&anonimal | Sure, just a moment.

@anonimal anonimal closed this Nov 7, 2016
@anonimal anonimal mentioned this pull request Nov 7, 2016
1 task
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant