dillongoostudios / goo-engine

Custom build of blender with some extra NPR features.
Other
982 stars 129 forks source link

warning C4100: 'con'/C7555 error #80

Open Elementdestroyer opened 11 months ago

Elementdestroyer commented 11 months ago

C:\blender-git\goo-engine\source\blender\editors\object\object_constraint.cc(2209,43): warning C4100: 'con': unreferenced formal parameter [C:\blender-git\build_windows_x64_vc17_Release\source\blender\editors\object\bf_editor_object.vcxproj] C:\blender-git\goo-engine\source\blender\editors\object\object_constraint.cc(2292,15): error C7555: use of designated initializers requires at least '/std:c++20' [C:\blender-git\build_windows_x64_vc17_Release\source\blender\editors\object\bf_editor_object.vcxproj]

Compiling the main branch though I have the additional problem such as python no being found when installing windows library I'm not sure if that's possibly effecting I have installed python but it doesn't seem to fix that problem. though I have tried to seek answers via discord server nobody wants to give a proper answer. I've restarted to compile the software multiple times and the error still pops up in the console.

Edit: Credit to: .thorn._ on the goo discord server Ok, I've tried the follow this morning, and it works.

first, delete the build_windows_x64_vc17_Release folder if it's there. This is recreated upon the new build, so not needed.

Based on your previous, it looks like your c\blender-git\goo-engine folder might be fine. So, nav to that folder (cd c\blender-git\goo-engine)

next, we're going to switch to a different branch of the GE fork: Edit: git checkout [engine version] e.g: git checkout goo-engine-v4.0-release (this post was made before the v4.1 release and will always continue from this post) then make update (let it finish) make (final step)

if successful, then \build_windows_x64_vc17_Release\bin\Release is the full application folder. You'll find the executable in here. You can rename "RELEASE" to gooengine 4.0 and drag the folder whereever you like

Note: This is not the same version as on patreon (4.0 vs 4.01). I'm not able to determine how to flow the changes made a couple weeks ago (the .01 update) into the git update, but that's the best i can do. Note that the screenspace shader does NOT work in 4.0; this was fixed in 4.01, along with a couple of other things.

Edit 25/3/24 git checkout since this post was made before 4.1 public github release as well as to be left open for more visibility.

modforge2 commented 9 months ago

Thx for this article(?), it works for me. Have a nice day dude

bizarro2000 commented 8 months ago

this worked! thank you!!!

ludrol commented 8 months ago

Please open the issue as it isn't resolved yet in the repository and it is easier to find with default github search.

To actually resolve the issue:

cd c:\blender-git\goo-engine this will go to the correct folder git fetch https://github.com/illusion0001/goo-engine.git patch-4:patch-4 this will download the fix git checkout goo-engine-main this will make sure that we are in correct git branch git merge patch-4 this will apply the fix

then the usual

make update make

and viola you have made your own goo

tagging @ThatCasual as he had the same problem

Elementdestroyer commented 8 months ago

Please open the issue as it isn't resolved yet in the repository and it is easier to find with default github search.

To actually resolve the issue:

cd c:\blender-git\goo-engine this will go to the correct folder git fetch https://github.com/illusion0001/goo-engine.git patch-4:patch-4 this will download the fix git checkout goo-engine-main this will make sure that we are in correct git branch git merge patch-4 this will apply the fix

then the usual

make update make

and viola you have made your own goo

tagging @ThatCasual as he had the same problem

make update

Please open the issue as it isn't resolved yet in the repository and it is easier to find with default github search.

To actually resolve the issue:

cd c:\blender-git\goo-engine this will go to the correct folder git fetch https://github.com/illusion0001/goo-engine.git patch-4:patch-4 this will download the fix git checkout goo-engine-main this will make sure that we are in correct git branch git merge patch-4 this will apply the fix

then the usual

make update make

and viola you have made your own goo

tagging @ThatCasual as he had the same problem

Please open the issue as it isn't resolved yet in the repository and it is easier to find with default github search.

To actually resolve the issue:

cd c:\blender-git\goo-engine this will go to the correct folder git fetch https://github.com/illusion0001/goo-engine.git patch-4:patch-4 this will download the fix git checkout goo-engine-main this will make sure that we are in correct git branch git merge patch-4 this will apply the fix

then the usual

make update make

and viola you have made your own goo

tagging @ThatCasual as he had the same problem

Does it not mean that the file missing was from the 4.1 build since this post was made before 4.1 github release?

ludrol commented 8 months ago

I think from reading the patch, there is a bug when compilling with VS2022.

If you are curious when it was introduced you can run git blame

PR that solves the issue: https://github.com/dillongoostudios/goo-engine/pull/81

ghost commented 7 months ago

Many thanks for this fix, as I was having the same problem. ^^