Build with Guix

[LiberalArtist]

Current status: See #8 (comment) to try it out!

---

Original Post

Currently, building with guix time-machine -C channels.scm -- build -f guix.scm --keep-failed hits the following error: I've attaching the full log as 9f5hc6rgfdmwv8wkw6rwi8n79nbkr5np-tangerine-0.0.drv.log.

/tmp/guix-build-tangerine-0.0.drv-0/source/tangerine/lua_sdf.cpp:533:11: error: no matching function for call to 'min'
                        Dist = glm::min(abs(Dist), MaxDist);
                               ^~~~~~~~
/tmp/guix-build-tangerine-0.0.drv-0/source/third_party/glm/detail/func_common.inl:17:43: note: candidate template ignored: deduced conflicting types for parameter 'genType' ('int' vs. 'float')
        GLM_FUNC_QUALIFIER GLM_CONSTEXPR genType min(genType x, genType y)

There are a bunch of warnings like this that might or might not be related:

/tmp/guix-build-tangerine-0.0.drv-0/source/tangerine/lua_sdf.cpp:533:20: warning: using integer absolute value function 'abs' when argument is of floating point type [-Wabsolute-value]
                        Dist = glm::min(abs(Dist), MaxDist);
                                        ^
/tmp/guix-build-tangerine-0.0.drv-0/source/tangerine/lua_sdf.cpp:533:20: note: use function 'std::abs' instead
                        Dist = glm::min(abs(Dist), MaxDist);
                                        ^~~
                                        std::abs

[LiberalArtist]

The problem very well could be in the Guix environment!

[LiberalArtist]

Rebasing on 2ef412e fixed the above! Now I get the same error I saw with GCC (i.e. without clang-toolchain):

ld: CMakeFiles/tangerine.dir/third_party/imgui/backends/imgui_impl_opengl3.cpp.o: undefined reference to symbol 'dlclose@@GLIBC_2.2.5'
ld: /gnu/store/5h2w4qi9hk1qzzgi1w83220ydslinr4s-glibc-2.33/lib/libdl.so.2: error adding symbols: DSO missing from command line
clang-13: error: linker command failed with exit code 1 (use -v to see invocation)
make[2]: *** [CMakeFiles/tangerine.dir/build.make:1109: tangerine] Error 1
make[2]: Leaving directory '/tmp/guix-build-tangerine-0.0.drv-0/build'
make[1]: *** [CMakeFiles/Makefile2:86: CMakeFiles/tangerine.dir/all] Error 2
make[1]: Leaving directory '/tmp/guix-build-tangerine-0.0.drv-0/build'
make: *** [Makefile:94: all] Error 2
error: in phase 'build': uncaught exception:
%exception #<&invoke-error program: "make" arguments: ("-j" "16") exit-status: 2 term-signal: #f stop-signal: #f> 
phase `build' failed after 27.8 seconds
command "make" "-j" "16" failed with status 2
note: keeping build directory `/tmp/guix-build-tangerine-0.0.drv-1'
builder for `/gnu/store/8bk8ia9f0pi8ziasrvh5iiwbdq46f2if-tangerine-0.0.drv' failed with exit code 1
build of /gnu/store/8bk8ia9f0pi8ziasrvh5iiwbdq46f2if-tangerine-0.0.drv failed
View build log at '/var/log/guix/drvs/8b/k8ia9f0pi8ziasrvh5iiwbdq46f2if-tangerine-0.0.drv.bz2'.
guix build: error: build of `/gnu/store/8bk8ia9f0pi8ziasrvh5iiwbdq46f2if-tangerine-0.0.drv' failed

Maybe I just need to add -ld in to target_link_libraries for cmake? I'll look into that.

[LiberalArtist]

Everything now builds! But the result can't find the shaders, as I explain in more detail in a1f7315 (it might really be a linker problem).

[LiberalArtist]

It works! You can build it with guix time-machine -C channels.scm -- build -f guix.scm. Finding the shaders is still a bit fragile (e.g. guix time-machine -C channels.scm -- shell -f guix.scm -- tangerine doesn't work yet), and I want to clean this up before marking it as ready for review, but you can run it with:

$(guix time-machine -C channels.scm -- build -f guix.scm)/bin/tangerine

[LiberalArtist]

I think this is ready! I've pared down this PR to just ("just") building with Guix, but I have work in progress toward enabling Racket, too.

The main caveat is that I don't have a setup to test that the Windows build still works.

[Aeva]

The main caveat is that I don't have a setup to test that the Windows build still works.

That's fine, I can fix any regressions that show up on the Windows side.

[LiberalArtist]

I just force-pushed a typo fix to the commit message:

manually more `tangerine`
           ⬆
           v

I'm happy to use _WIN_64 or make other changes if you want, or feel free to just do them yourself.