mach/dev/template
Lue a1fe671db8
all: build: fix sdkPath for relative @src.file / fix autocompletion with ZLS / IDEs (#661)
* all: build: fix sdkPath for relative @src.file

Prior to this commit, the build system heavily assumed that the result
`@src.file` would always be absolute, but this is no longer
guaranteed, likely due to there being no such thing as an "absolute
path" in WASI.

It appears that for normal invocations of `zig build`, it is safe to
assume that `@src.file` is absolute. However, when ZLS uses a custom
`build_runner.zig` to collect build configuration, `@src.file` is
actually relative to the current working directory, at least on my
system. For a while, this led to ZLS completions breaking entirely,
but presently it actually causes ZLS to crash!

The solution is not as simple as using relative `sdkPath` results
as-is, because the build system may attempt to resolve these paths
relative to build root, when the paths are actually relative to the
current working directory.

This leads to a sticky situation: the current working directory is a
runtime concept, but `@src.file` is resolved at compile time. However,
it appears that the build runner does not change current working
directory in between compilation and execution, so it is probably safe
to calculate `sdkPath` using runtime current working directory.

Still, this requires major changes with how `sdkPath` works, since
runtime computation and allocations are required. So pretty much
anything that relied on `sdkPath` being comptime-known has been
refactored in this commit.

The most severe result of this is that, for example, `gpu.pkg` can no
longer be a comptime-known constant: it has to be a runtime function
that takes a `*Builder` and returns a `Pkg`.

This commit deals with usages of `*.pkg` and `sdkPath` within Mach
itself, but projects that depend on Mach such as `mach-examples` will
almost certainly require changes as well.

* all: update README to reflect change in pkg usage

For details on updating your code to use this version, see: 88b1106953

Signed-off-by: Stephen Gutekanst <stephen@hexops.com>
Co-authored-by: Stephen Gutekanst <stephen@hexops.com>
2023-01-02 01:23:46 -07:00
..
.github all: update Zig to version 0.11.0-dev.1023+1c711b0a6 2023-01-01 02:40:57 -07:00
.gitattributes dev/template: add template for creating new sub-projects 2022-08-22 20:49:44 -07:00
.gitignore dev/template: add template for creating new sub-projects 2022-08-22 20:49:44 -07:00
LICENSE dev/template: make LICENSE year consistent 2022-08-23 08:13:54 -07:00
LICENSE-APACHE dev/template: add template for creating new sub-projects 2022-08-22 20:49:44 -07:00
LICENSE-MIT dev/template: make LICENSE year consistent 2022-08-23 08:13:54 -07:00
README.md all: build: fix sdkPath for relative @src.file / fix autocompletion with ZLS / IDEs (#661) 2023-01-02 01:23:46 -07:00

mach/foobar - a project template

This repository is a separate copy of the same library in the main Mach repository, and is automatically kept in sync, so that anyone can use this library in their own project if they like!

Getting started

Adding dependency

In a libs subdirectory of the root of your project:

git clone https://github.com/hexops/mach-foobar

Then in your build.zig add:

...
const foobar = @import("libs/mach-foobar/build.zig");

pub fn build(b: *Builder) void {
    ...
    exe.addPackage(foobar.pkg(b));
    foobar.link(b, exe, .{});
}

Join the community

Join the Mach community on Discord or Matrix to discuss this project, ask questions, get help, etc.

Issues

Issues are tracked in the main Mach repository.

Contributing

Contributions are very welcome. Pull requests must be sent to the main repository to avoid some complex merge conflicts we'd get by accepting contributions in both repositories. Once the changes are merged there, they'll get sync'd to this repository automatically.