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

build fails with current source #26

Open
skochinsky opened this issue Nov 15, 2018 · 5 comments
Open

build fails with current source #26

skochinsky opened this issue Nov 15, 2018 · 5 comments

Comments

@skochinsky
Copy link

It may be that I fail at git but reporting this just in case. Using VS2015 but with Target Platform set to 10.0.10240.0 (not sure if important).

  1. recent git pull cleaned up the openssl directory with the exeption of a few includes so BaseCryptLib and OpenSslLib fail to build due to missing .c files

  2. GlueLib fails to compile guid.c due to last line:

1>------ Build started: Project: GlueLib, Configuration: Release x64 ------
1>  guid.c
1>guid.c(105): error C2065: 'EDKII_IFR_BIT_VARSTORE_GUID': undeclared identifier
1>guid.c(105): error C2099: initializer is not a constant

commenting it out allows build to succeed and I can build the samples.

@ionescu007
Copy link
Owner

ionescu007 commented Nov 15, 2018 via email

@skochinsky
Copy link
Author

Tried some git incantations but could not get it to work. Finally deleted everything and did a pull from scratch:

git clone https://github.com/ionescu007/VisualUefi.git --recurse-submodules

This seems to have worked - EDK-II compiled without errors and samples too. Got one minor warning though:

1>LINK : warning LNK4281: undesirable base address 0x0 for x64 image; set base address above 4GB for best ASLR optimization

@ionescu007
Copy link
Owner

ionescu007 commented Nov 16, 2018 via email

@skochinsky
Copy link
Author

I don't trust myself enough with git to make a PR, but /IGNORE:4281 in linker options seems to suppress it.

@Mattiwatti
Copy link
Contributor

It does, and I've created a PR to add this flag to the linker options in your stead 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants