Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build on Linux: mismatched bound float[4] #75

Open
phraggers opened this issue Jan 9, 2022 · 2 comments
Open

build on Linux: mismatched bound float[4] #75

phraggers opened this issue Jan 9, 2022 · 2 comments

Comments

@phraggers
Copy link

phraggers commented Jan 9, 2022

I'm sure I'm doing something silly and rookie-ish so I apologize in advance.
I'm on Arch Linux, I've typed the git clone command as per the readme, cd minecraft-weekend and make
and I receive the following error while trying to make cglm:
(I've shortened the paths below, sorry the new lines and spacing is kinda gone)
..minecraft-weekend/lib/cglm/src/vec4.c:127:25: error: argument 1 of type 'float[4]' with mismatched bound [-Werror=array-parameter=] 127 | glmc_vec4_scale_as(vec4 v, float s, vec4 dest) { | ~~~~~^ In file included from ../minecraft-weekend/lib/cglm/src/../include/cglm/call.h:17, from ../minecraft-weekend/lib/cglm/src/vec4.c:9: ../minecraft-weekend/lib/cglm/src/../include/cglm/call/vec4.h:102:25: note: previously declared as 'float[3]' 102 | glmc_vec4_scale_as(vec3 v, float s, vec3 dest); | ~~~~~^ ../minecraft-weekend/lib/cglm/src/vec4.c:127:42: error: argument 3 of type 'float[4]' with mismatched bound [-Werror=array-parameter=] 127 | glmc_vec4_scale_as(vec4 v, float s, vec4 dest) { | ~~~~~^~~~ In file included from ../minecraft-weekend/lib/cglm/src/../include/cglm/call.h:17, from ../minecraft-weekend/lib/cglm/src/vec4.c:9: ../minecraft-weekend/lib/cglm/src/../include/cglm/call/vec4.h:102:42: note: previously declared as 'float[3]' 102 | glmc_vec4_scale_as(vec3 v, float s, vec3 dest); | ~~~~~^~~~ cc1: all warnings being treated as errors

ScoreUnder added a commit to ScoreUnder/minecraft-weekend that referenced this issue Feb 9, 2022
ScoreUnder added a commit to ScoreUnder/minecraft-weekend that referenced this issue Feb 9, 2022
@jmaargh
Copy link

jmaargh commented Mar 28, 2022

To help anybody else who comes across this, simply update the libs/cglm submodule to the v0.8.4 tag and it should build fine.

@Bonekazz
Copy link

go to minecraft-weekend/lib/cglm/include/cglm/call/vec4.h in line 102, and change

glmc_vec4_scale_as(vec3 v, float s, vec3 dest);
to
glmc_vec4_scale_as(vec4 v, float s, vec4 dest);

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

No branches or pull requests

3 participants