Skip to content
This repository has been archived by the owner on Apr 20, 2022. It is now read-only.

Readme unclear #8

Open
zeroZshadow opened this issue Oct 21, 2016 · 2 comments
Open

Readme unclear #8

zeroZshadow opened this issue Oct 21, 2016 · 2 comments

Comments

@zeroZshadow
Copy link

The Readme is missing out a few important points.
The SDK folder junctions. What exact folder should be junctioned, and what should it look like after this task is completed?

When trying to recompile after following the steps, the following error will be throw when trying to rebuild the UnrealBuildToold.

System\ActionGraph.cs(429,6,429,15): error CS0246: The type or namespace name 'FASTBuild' could not be found (are you missing a using directive or an assembly reference?)

The README should specify the steps to avoid this.

@MrCorzell
Copy link

MrCorzell commented Feb 11, 2017

It looks like in FastBuild.cs he defines the relative paths in

public static bool IsAvailable

for the public they are as following:

Assume UnrealEngine located here:

C:\UnrealEngine\Engine\Extras\FASTBuild\

Paths of interest are:

C:\UnrealEngine\Engine\Extras\FASTBuild\External/VS15.0
C:\UnrealEngine\Engine\Extras\FASTBuild\External/Windows8.1

I'm guessing this should be Windows10 for the Junction folder link that was made in the ReadMe.

Also - Regenerate your project (I also added all C# files into UBT project directly (was before I regen'ed, so not sure if that helped or not)

I am thinking of making environmental variables so that this symbolic link crap can go away and a more defined step that doesn't rely on hard-coded values can be part of the system. I'll update with pull request.

@ClxS
Copy link
Owner

ClxS commented Feb 11, 2017

@MrCorzell The reason for using symlinks is to avoid cache misses due to different VS install paths (they're made into absolute path comments during preprocessing, so have to be the same for the hash to match on different machines), so I wont be able to accept the PR.
If you're only using it on your own setup though you could use the VS140COMNTOOLS path :)

I agree I should better document it. I'm hoping to come back to doing more work on this project soon.

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

3 participants