Visitar URL original
Build: fix MinGW-w64 build. Closes #36 by anonimal · Pull Request #435 · monero-project/kovri · GitHub
Skip to content
This repository was archived by the owner on Sep 1, 2022. It is now read-only.

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

Closed
anonimal wants to merge 4 commits into
monero-project:masterfrom
anonimal:fix-win-build
Closed

anonimal wants to merge 4 commits into
monero-project:masterfrom
anonimal:fix-win-build

Conversation

@anonimal

@anonimal anonimal commented Nov 5, 2016

Copy link
Copy Markdown
Collaborator

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

anonimal commented Nov 7, 2016

Copy link
Copy Markdown
Collaborator Author
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 done
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