Skip to content

Commit a27f312

Browse files
authored
Fix xscrnsaver hash (#7)
Fixes the xscrnsaver dependency hash. xscrnsaver wasn't actually triggering the hash bug that pulseaudio triggers, I just made a mistake and didn't realize the previous result was cached.
1 parent 5cfb0a6 commit a27f312

File tree

3 files changed

+1
-21
lines changed

3 files changed

+1
-21
lines changed

.github/workflows/ci.yaml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,6 @@ jobs:
1616
matrix:
1717
zig-version: [master]
1818
os: [ubuntu-latest, windows-latest]
19-
include:
20-
- zig-version: "0.15.0-dev.1218+bc8e1a74c"
21-
os: ubuntu-latest
2219
runs-on: ${{ matrix.os }}
2320
steps:
2421
- name: Checkout

build.zig.zon

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -115,15 +115,7 @@
115115
.xscrnsaver = .{
116116
// This dependency must be kept in sync with `xscrnsaver_windows` for similar reasons explained in the `pulseaudio` dependency.
117117
.url = "https://gitlab.freedesktop.org/xorg/lib/libxscrnsaver/-/archive/libXScrnSaver-1.2.4/libxscrnsaver-libXScrnSaver-1.2.4.tar.gz",
118-
.hash = "N-V-__8AAMXRAAAKIBB3YHvwcykG71lH-HtylfJ0AV048a7V",
119-
.lazy = true,
120-
},
121-
.xscrnsaver_windows = .{
122-
// This is a duplicate of `xscrnsaver`, but with a different hash due to a Zig bug that
123-
// results in a different hash on Windows than Linux.
124-
.url = "https://gitlab.freedesktop.org/xorg/lib/libxscrnsaver/-/archive/libXScrnSaver-1.2.4/libxscrnsaver-libXScrnSaver-1.2.4.tar.gz",
125118
.hash = "N-V-__8AAEO3AADhS3i2HiVniZnPqWJh33VqmmUp7JO33xqe",
126-
.lazy = true,
127119
},
128120
.alsa = .{
129121
.url = "https://github.com/alsa-project/alsa-lib/archive/refs/tags/v1.2.14.tar.gz",

src/linux.zig

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -436,16 +436,6 @@ pub fn build(
436436
lib.addIncludePath(b.path("deps/alsa/include"));
437437
}
438438

439-
// Provide the xscrnsaver headers. Conditional is workaround for cross compilation, see
440-
// comment in `build.zig.zon`
441-
const xscrnsaver_name = switch (@import("builtin").os.tag) {
442-
.windows => "xscrnsaver_windows",
443-
else => "xscrnsaver",
444-
};
445-
if (b.lazyDependency(xscrnsaver_name, .{})) |xscrnsaver| {
446-
lib.addIncludePath(xscrnsaver.path("include"));
447-
}
448-
449439
// Provide upstream headers that don't require any special handling
450440
lib.addIncludePath(b.dependency("egl", .{}).path("api"));
451441
lib.addIncludePath(b.dependency("opengl", .{}).path("api"));
@@ -457,6 +447,7 @@ pub fn build(
457447
lib.addIncludePath(b.dependency("xfixes", .{}).path("include"));
458448
lib.addIncludePath(b.dependency("xrandr", .{}).path("include"));
459449
lib.addIncludePath(b.dependency("xrender", .{}).path("include"));
450+
lib.addIncludePath(b.dependency("xscrnsaver", .{}).path("include"));
460451
lib.addIncludePath(b.dependency("jack", .{}).path("common"));
461452
lib.addIncludePath(b.dependency("sndio", .{}).path("libsndio"));
462453
lib.addIncludePath(b.path("deps/wayland/protocols"));

0 commit comments

Comments
 (0)