Skip to content

Update readme for building on macOS#110

Closed
SeekingMeaning wants to merge 1 commit into
xtreme8000:standalonefrom
SeekingMeaning:patch-1
Closed

Update readme for building on macOS#110
SeekingMeaning wants to merge 1 commit into
xtreme8000:standalonefrom
SeekingMeaning:patch-1

Conversation

@SeekingMeaning
Copy link
Copy Markdown

@SeekingMeaning SeekingMeaning commented Jun 27, 2020

Homebrew provides glew and libdeflate

(Maybe closes #89)

Homebrew provides glew and libdeflate
@SeekingMeaning
Copy link
Copy Markdown
Author

Steps needed to compile on macOS:

  • Add #include <stddef.h> to utils.h (Make it so CMake now grabs the latest dr_wav version instead of 0.8.1 #105)

  • Currently, only part of the server list is rendered:

    A workaround for this is commenting out these lines in hud.c:

     glEnable(GL_SCISSOR_TEST);
     glScissor((settings.window_width - 600 * scaley) / 2.0F, 50 * scaley, 600 * scaley, (430 - 50) * scaley);
    

    and this line:

     glDisable(GL_SCISSOR_TEST);
    

    This workaround results in this behavior when scrolling:

@xtreme8000
Copy link
Copy Markdown
Owner

Currently, only part of the server list is rendered

This happens because the actual window size is different from what the game thinks it is, try adjusting your window dimensions in settings.

@SeekingMeaning
Copy link
Copy Markdown
Author

Strangely, it seems multiplying each parameter of glScissor by 2 fixes the problem:

   float scissor_x = (settings.window_width - 600 * scaley) / 2.0F;
   float scissor_y = 50 * scaley;
   float scissor_w = 600 * scaley;
   float scissor_h = (430 - 50) * scaley;

#if __APPLE__
   scissor_x *= 2;
   scissor_y *= 2;
   scissor_w *= 2;
   scissor_h *= 2;
#endif
   glScissor(scissor_x, scissor_y, scissor_w, scissor_h);

@SeekingMeaning
Copy link
Copy Markdown
Author

Bug with picking up intel:

Before: Screen Shot 2020-06-29 at 10 43 02 AM
After: Screen Shot 2020-06-29 at 10 43 12 AM

@xtreme8000
Copy link
Copy Markdown
Owner

possibly related to #145 and #146

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants