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

Error building raylib #40

Open
GustavoJCL opened this issue Mar 24, 2024 · 7 comments · May be fixed by #41
Open

Error building raylib #40

GustavoJCL opened this issue Mar 24, 2024 · 7 comments · May be fixed by #41

Comments

@GustavoJCL
Copy link

Hi, i followed the instruction in the documentation, but i have this error:

error: struct 'target.Target' has no member named 'Query'
pub fn addTo(b: *std.Build, exe: *std.Build.Step.Compile, target: std.Target.Query, optimize: std.builtin.Mode, raylibOptions: raylib_build.Options) void {
                                                                  ~~~~~~~~~~^~~~~~
/usr/lib/zig/std/target.zig:6:20: note: struct declared here
pub const Target = struct {
                   ^~~~~~
referenced by:
    build: /media/gus/proyectos/wolfenstein_3d_zig/build.zig:28:11
    runBuild__anon_7136: /usr/lib/zig/std/Build.zig:1638:27
    remaining reference traces hidden; use '-freference-trace' to see all reference traces

my zig version is: 0.11.0

@Axiomatic-Mind
Copy link

I'm having a similar issue. When looking at the Zig standard library documentation, Query certainly exists as a type inside of Target, but when I checked my local files, I found no definition or even reference to Query. When I ran zig version, I also received 0.11.0.

@ProIcons ProIcons linked a pull request Mar 31, 2024 that will close this issue
@fenilli
Copy link

fenilli commented May 23, 2024

Same problem but with master version: 0.13.0-dev.245+5fe9f88b1:

const std = @import("std");
const rl = @import("raylib/build.zig");

pub fn build(b: *std.Build) void {
    const target = b.standardTargetOptions(.{});
    const optimize = b.standardOptimizeOption(.{});

    const exe = b.addExecutable(.{
        .name = "zigaroids",
        .root_source_file = b.path("src/main.zig"),
        .target = target,
        .optimize = optimize,
    });

    rl.addTo(b, exe, target, optimize, .{});

    b.installArtifact(exe);

    const run_exe = b.addRunArtifact(exe);
    b.step("run", "Run Zigaroids!").dependOn(&run_exe.step);
}

With error:

expected type 'Target.Query', found 'Build.ResolvedTarget'
    rl.addTo(b, exe, target, optimize, .{});
                     ^~~~~~
/home/fenilli/.zig/zig-0.13.0-dev/lib/std/Build.zig:2485:28: note: struct declared here
pub const ResolvedTarget = struct {
                           ^~~~~~
/home/fenilli/.zig/zig-0.13.0-dev/lib/std/Target/Query.zig:1:1: note: struct declared here
//! Contains all the same data as `Target`, additionally introducing the

Edit:

Changing the target rl.addTo(b, exe, target, optimize, .{}); to query rl.addTo(b, exe, target.query, optimize, .{}); gives this error:

 error: no field named 'path' in union 'Build.LazyPath'
    exe.root_module.addAnonymousImport("raylib", .{ .root_source_file = .{ .path = cwd ++ sep ++ "raylib.zig" } });
                                                                            ^~~~
/home/fenilli/.zig/zig-0.13.0-dev/lib/std/Build.zig:2133:22: note: union declared here
pub const LazyPath = union(enum) {

@leg7
Copy link

leg7 commented May 24, 2024

same problem on 0.12.0

@cattokomo
Copy link

path field in LazyPath is deprecated as of 0.12.0. We can probably fix this by using std.Build.path instead.

@ziontee113
Copy link

Maybe try #41 (comment) like I did? Worked for me at Zig 0.12

@fenilli
Copy link

fenilli commented Jun 12, 2024

FYI this is all builtin to raylib:

Instructions here worked for me: https://www.reddit.com/r/Zig/comments/16r0fj6/ysk_you_can_use_native_raylib_directly_in_zig_for/

You can, that is what I'm using, but those are not crafted with zig in mind, they are just direct translations so you have to do bit casting and all by hand, just a bit more work over a working binding.

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 a pull request may close this issue.

7 participants