-
Notifications
You must be signed in to change notification settings - Fork 124
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
Comments
You might have only done a shadow pull and still using an older edk2
upstream.
Best regards,
Alex Ionescu
…On Thu, Nov 15, 2018 at 8:27 PM Igor Skochinsky ***@***.***> wrote:
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.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#26>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFxIeEHYNzPV4camSnhmRuFxThW9--6xks5uvcApgaJpZM4Yh9U3>
.
|
Tried some git incantations but could not get it to work. Finally deleted everything and did a pull from scratch:
This seems to have worked - EDK-II compiled without errors and samples too. Got one minor warning though:
|
Yep known annoying issue. If you find a compiler flag to make it go away,
I’d appreciate a PR.
On Fri, Nov 16, 2018 at 12:05 PM Igor Skochinsky ***@***.***> wrote:
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
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#26 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFxIeLjAjFDBfp591QnQoRmJdc5c7fALks5uvpwGgaJpZM4Yh9U3>
.
--
Best regards,
Alex Ionescu
|
I don't trust myself enough with git to make a PR, but |
It does, and I've created a PR to add this flag to the linker options in your stead 😉 |
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).
recent
git pull
cleaned up theopenssl
directory with the exeption of a few includes soBaseCryptLib
andOpenSslLib
fail to build due to missing.c
filesGlueLib fails to compile
guid.c
due to last line:commenting it out allows build to succeed and I can build the samples.
The text was updated successfully, but these errors were encountered: