AlexCharlton / Hypergiant

An OpenGL-based game library for CHICKEN Scheme
BSD 2-Clause "Simplified" License
68 stars 5 forks source link

compilation problem #8

Closed kristianlm closed 8 years ago

kristianlm commented 8 years ago

Hey Alex!

When doing chicken-install hypergiant, I get this:

Warning: reference to possibly unbound identifier `pixel-density-ratio' in:
Warning:    start*

I believe this should fix it:

--- a/window.scm
+++ b/window.scm
@@ -91,5 +91,5 @@
   (resize #f width height)
   (set-pixel-density-ratio)
-  (pixel-density-ratio (ceiling *pixel-density-ratio*))
+  (set! *pixel-density-ratio* (ceiling *pixel-density-ratio*))
   (gl:get-error) ;; silence error caused by GLEW
   (glls:compile-pipelines)

I felt forking was a bit overkill for this one.

AlexCharlton commented 8 years ago

While the code isn't very clear, the pixel-density-ratio being referenced there is a parameter from gl-type. You probably have an old version lying around (not being able to version dependencies in Chicken is a real pain), so chicken-install gl-type hypergiant should work!

kristianlm commented 8 years ago

Ah, indeed!

I believe you can do this:

--- a/hypergiant.meta
+++ b/hypergiant.meta
@@ -7 +7 @@
- (depends  glfw3 glls opengl-glew gl-math gl-utils gl-type hyperscene
+ (depends  glfw3 glls opengl-glew gl-math gl-utils (gl-type 0.2.1) hyperscene
AlexCharlton commented 8 years ago

Oh wow, you can! This changes everything!

On Tue, Dec 29, 2015 at 2:23 PM Kristian Lein-Mathisen < notifications@github.com> wrote:

Ah, indeed!

I believe you can do this:

--- a/hypergiant.meta+++ b/hypergiant.meta@@ -7 +7 @@- (depends glfw3 glls opengl-glew gl-math gl-utils gl-type hyperscene+ (depends glfw3 glls opengl-glew gl-math gl-utils (gl-type 0.2.1) hyperscene

— Reply to this email directly or view it on GitHub https://github.com/AlexCharlton/Hypergiant/issues/8#issuecomment-167859476 .

dleslie commented 8 years ago

Oooh