ionescu007 / VisualUefi

A project for allowing EDK-II Development with Visual Studio
480 stars 124 forks source link

build fails with current source #26

Open skochinsky opened 5 years ago

skochinsky commented 5 years ago

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 commented 5 years ago

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 notifications@github.com 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 https://github.com/ionescu007/VisualUefi/issues/26, or mute the thread https://github.com/notifications/unsubscribe-auth/AFxIeEHYNzPV4camSnhmRuFxThW9--6xks5uvcApgaJpZM4Yh9U3 .

skochinsky commented 5 years ago

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 commented 5 years ago

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 notifications@github.com 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 https://github.com/ionescu007/VisualUefi/issues/26#issuecomment-439360412, or mute the thread https://github.com/notifications/unsubscribe-auth/AFxIeLjAjFDBfp591QnQoRmJdc5c7fALks5uvpwGgaJpZM4Yh9U3 .

-- Best regards, Alex Ionescu

skochinsky commented 5 years ago

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

Mattiwatti commented 5 years ago

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