Skip to content
This repository has been archived by the owner on Aug 9, 2020. It is now read-only.

build error with not found Classes/Classes.h and IHttpRequest.h #1

Closed
dotcom opened this issue Aug 2, 2020 · 2 comments
Closed

build error with not found Classes/Classes.h and IHttpRequest.h #1

dotcom opened this issue Aug 2, 2020 · 2 comments

Comments

@dotcom
Copy link

dotcom commented Aug 2, 2020

I faced a build error and I share it

'Classes/Classes.h': No such file or directory
'IHttpRequest.h': No such file or directory
'IHttpResponse.h': No such file or directory

And, I modified the code as follows to avoid this error, is it correct?

AgonesComponent.cpp

- #include "Classes/Classes.h"
- #include "IHttpRequest.h"
- #include "IHttpResponse.h"
+ #include "Classes.h"
+ #include "Interfaces/IHttpRequest.h"
+ #include "Interfaces/IHttpResponse.h"

AgonesComponent.h

- #include "Classes/Classes.h"
- #include "IHttpRequest.h"
+ #include "Classes.h"
+ #include "Interfaces/IHttpRequest.h"

UE version=4.24.3-0+++Release-4.24 source build

@domgreen
Copy link
Contributor

domgreen commented Aug 5, 2020

Thanks for reporting just been having a play with this.

Did you just compile this via the Visual Studio and which version of VS we are on 2019? Could be that how MSBuild works changes between the two.

0>Using Visual Studio 2019 14.26.28805 toolchain (C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.26.28801) and Windows 10.0.18362.0 SDK (C:\Program Files (x86)\Windows Kits\10).

Is compiling (ReBuild Solution) fine in UE-Rider and against our 4.25 on 2019 fork of the engine on Linux and Mac, with your suggested changes it also works fine so will commit them.

Also; your change is actually more technically correct 👍

Gonna also remove the includes from the cpp file that are in the header file as they are not needed in both if they are in the header.

I will add this change locally and include it as I push this version of the SDK to Agones main.

@domgreen
Copy link
Contributor

domgreen commented Aug 9, 2020

Thanks @dotcom issue has been addressed and merged in #googleforgames/agones/1739

@domgreen domgreen closed this as completed Aug 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants