Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] niri: update to 25.01
@ 2025-01-12 18:27 trolljoe
  2025-01-12 20:40 ` trolljoe
                   ` (25 more replies)
  0 siblings, 26 replies; 28+ messages in thread
From: trolljoe @ 2025-01-12 18:27 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1400 bytes --]

There is a new pull request by trolljoe against master on the void-packages repository

https://github.com/trolljoe/void-packages niri
https://github.com/void-linux/void-packages/pull/53946

niri: update to 25.01
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **NO**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->

this goes by the new packaging niri page (https://github.com/YaLTeR/niri/wiki/Packaging-niri), if setting XDG_RUNTIME_DIR to `mktemp` isn't enough i'll skip the tests.

A patch file from https://github.com/void-linux/void-packages/pull/53946.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-niri-53946.patch --]
[-- Type: text/x-diff, Size: 1162 bytes --]

From 1eb21be3e4bae601fc0087a4d22b92012e1bac71 Mon Sep 17 00:00:00 2001
From: Joetroll <joetroll@proton.me>
Date: Sun, 12 Jan 2025 15:23:37 -0300
Subject: [PATCH] niri: update to 25.01

---
 srcpkgs/niri/template | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/niri/template b/srcpkgs/niri/template
index 0fb83c6b62dbec..26a83b97cf5d90 100644
--- a/srcpkgs/niri/template
+++ b/srcpkgs/niri/template
@@ -1,6 +1,6 @@
 # Template file for 'niri'
 pkgname=niri
-version=0.1.10.1
+version=25.01
 revision=1
 build_style=cargo
 configure_args="--no-default-features --features xdp-gnome-screencast"
@@ -13,7 +13,10 @@ license="GPL-3.0-or-later"
 homepage="https://github.com/YaLTeR/niri"
 changelog="https://github.com/YaLTeR/niri/releases"
 distfiles="https://github.com/YaLTeR/niri/archive/refs/tags/v${version}.tar.gz"
-checksum=d8854830436a87215b0bc6a60b6d43f350d927a03a2798c75f0fbda228bac8d3
+checksum=86b89bcfc3fc6a8ed81f9e3f0ac7a29bd30267515efb2c19e1e0bc2ccd67b649
+
+export NIRI_BUILD_COMMIT="e35c630"
+export XDG_RUNTIME_DIR="$(mktemp -d)"
 
 post_install() {
 	vinstall resources/niri.desktop 644 usr/share/wayland-sessions

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
@ 2025-01-12 20:40 ` trolljoe
  2025-01-13 10:53 ` lukeflo
                   ` (24 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: trolljoe @ 2025-01-12 20:40 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 176 bytes --]

New comment by trolljoe on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#issuecomment-2585909075

Comment:
tested and it seems to work fine.

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
  2025-01-12 20:40 ` trolljoe
@ 2025-01-13 10:53 ` lukeflo
  2025-01-13 23:45 ` oliver-cfc
                   ` (23 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: lukeflo @ 2025-01-13 10:53 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 217 bytes --]

New comment by lukeflo on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#issuecomment-2586779337

Comment:
> tested and it seems to work fine.

Can confirm that it works just fine!

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
  2025-01-12 20:40 ` trolljoe
  2025-01-13 10:53 ` lukeflo
@ 2025-01-13 23:45 ` oliver-cfc
  2025-01-14  1:09 ` chilledfrogs
                   ` (22 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: oliver-cfc @ 2025-01-13 23:45 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 214 bytes --]

New comment by oliver-cfc on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#issuecomment-2588467884

Comment:
Another confirmation that all is good... on a musl installation here.

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (2 preceding siblings ...)
  2025-01-13 23:45 ` oliver-cfc
@ 2025-01-14  1:09 ` chilledfrogs
  2025-01-14  1:09 ` [PR REVIEW] " classabbyamp
                   ` (21 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:09 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 227 bytes --]

New comment by chilledfrogs on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#issuecomment-2588543127

Comment:
Works fine for me!
Might I ask, however, why is the `dbus` feature not enabled?

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (3 preceding siblings ...)
  2025-01-14  1:09 ` chilledfrogs
@ 2025-01-14  1:09 ` classabbyamp
  2025-01-14  1:09 ` chilledfrogs
                   ` (20 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: classabbyamp @ 2025-01-14  1:09 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 345 bytes --]

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1913991066

Comment:
would be better to use something like `v$version`, as

1. hardcoding things like that is bad and will be forgotten at some point
2. we build from the tag, which is a reference to the commit

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (4 preceding siblings ...)
  2025-01-14  1:09 ` [PR REVIEW] " classabbyamp
@ 2025-01-14  1:09 ` chilledfrogs
  2025-01-14  1:10 ` chilledfrogs
                   ` (19 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:09 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 220 bytes --]

New review comment by chilledfrogs on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1913991355

Comment:
This variable is in accordance with Niri's own packaging guidelines

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (5 preceding siblings ...)
  2025-01-14  1:09 ` chilledfrogs
@ 2025-01-14  1:10 ` chilledfrogs
  2025-01-14  1:11 ` chilledfrogs
                   ` (18 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:10 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 267 bytes --]

New review comment by chilledfrogs on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1913991355

Comment:
This variable is in accordance with Niri's own packaging guidelines, it is _on top of_ the reported version number

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (6 preceding siblings ...)
  2025-01-14  1:10 ` chilledfrogs
@ 2025-01-14  1:11 ` chilledfrogs
  2025-01-14  1:12 ` classabbyamp
                   ` (17 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:11 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 296 bytes --]

New review comment by chilledfrogs on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1913991355

Comment:
This variable is in accordance with Niri's own packaging guidelines, it is _on top of_ the reported version number, i.e. `niri 25.01 (e35c630)`

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (7 preceding siblings ...)
  2025-01-14  1:11 ` chilledfrogs
@ 2025-01-14  1:12 ` classabbyamp
  2025-01-14  1:14 ` classabbyamp
                   ` (16 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: classabbyamp @ 2025-01-14  1:12 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 350 bytes --]

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1913992965

Comment:
maybe:
```sh
pre_check() {
    export XDG_RUNTIME_DIR="$mktemp -d "$wrksrc/runtime.XXXXXX")"
}
```

this will keep things contained inside the build directory and make it only used for tests

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (8 preceding siblings ...)
  2025-01-14  1:12 ` classabbyamp
@ 2025-01-14  1:14 ` classabbyamp
  2025-01-14  1:19 ` chilledfrogs
                   ` (15 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: classabbyamp @ 2025-01-14  1:14 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 420 bytes --]

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1913994542

Comment:
upstreams do stuff like this all the time. if they aren't leaving the commit in their release tarballs somehow (like a file), we shouldn't need to hard-code it. we aren't building some random commit, we're building the release, so the commit information is redundant.

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (9 preceding siblings ...)
  2025-01-14  1:14 ` classabbyamp
@ 2025-01-14  1:19 ` chilledfrogs
  2025-01-14  1:24 ` trolljoe
                   ` (14 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:19 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 354 bytes --]

New review comment by chilledfrogs on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1913997754

Comment:
Understood and agreed that it is redundant, however if it is expected by upstream, I'm not sure I really think it is worth it to break that expectation downstream is all (i.e. for bug reports and such)

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (10 preceding siblings ...)
  2025-01-14  1:19 ` chilledfrogs
@ 2025-01-14  1:24 ` trolljoe
  2025-01-14  1:26 ` chilledfrogs
                   ` (13 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: trolljoe @ 2025-01-14  1:24 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 277 bytes --]

New comment by trolljoe on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#issuecomment-2588560285

Comment:
> Works fine for me! Might I ask, however, why is the `dbus` feature not enabled?

i will update the template including that, my bad

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (11 preceding siblings ...)
  2025-01-14  1:24 ` trolljoe
@ 2025-01-14  1:26 ` chilledfrogs
  2025-01-14  1:26 ` chilledfrogs
                   ` (12 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:26 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 412 bytes --]

New comment by chilledfrogs on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#issuecomment-2588561840

Comment:
> i will update the template including that, my bad
Np, there may be legitimate reasons for it actually since I am not *100%* sure what it does, but I feel it may be useful, so I wouldn't mind if someone more experienced than me on these matters could weigh in too

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (12 preceding siblings ...)
  2025-01-14  1:26 ` chilledfrogs
@ 2025-01-14  1:26 ` chilledfrogs
  2025-01-14  1:27 ` chilledfrogs
                   ` (11 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:26 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 414 bytes --]

New comment by chilledfrogs on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#issuecomment-2588561840

Comment:
> i will update the template including that, my bad

Np, there may be legitimate reasons for it actually since I am not *100%* sure what it does, but I feel it may be useful, so I wouldn't mind if someone more experienced than me on these matters could weigh in too

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (13 preceding siblings ...)
  2025-01-14  1:26 ` chilledfrogs
@ 2025-01-14  1:27 ` chilledfrogs
  2025-01-14  1:27 ` chilledfrogs
                   ` (10 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:27 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 468 bytes --]

New comment by chilledfrogs on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#issuecomment-2588561840

Comment:
> i will update the template including that, my bad

Np, there may be legitimate reasons for it actually since I am not *100%* sure what it does (but Works For Me:tm: without `elogind` for instance), but I feel it may be useful, so I wouldn't mind if someone more experienced than me on these matters could weigh in too

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (14 preceding siblings ...)
  2025-01-14  1:27 ` chilledfrogs
@ 2025-01-14  1:27 ` chilledfrogs
  2025-01-14  1:30 ` [PR REVIEW] " trolljoe
                   ` (9 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:27 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 467 bytes --]

New comment by chilledfrogs on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#issuecomment-2588561840

Comment:
> i will update the template including that, my bad

Np, there may be legitimate reasons for it actually since I am not *100%* sure what it does (but Works For Me™ without `elogind` for instance), but I feel it may be useful, so I wouldn't mind if someone more experienced than me on these matters could weigh in too

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (15 preceding siblings ...)
  2025-01-14  1:27 ` chilledfrogs
@ 2025-01-14  1:30 ` trolljoe
  2025-01-14  1:35 ` chilledfrogs
                   ` (8 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: trolljoe @ 2025-01-14  1:30 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 298 bytes --]

New review comment by trolljoe on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1914005107

Comment:
is the syntax correct? wouldn't i use this instead?

```sh
pre_check() {
    export XDG_RUNTIME_DIR=$(mktemp -d "$wrksrc/runtime.XXXXXX")
}
```

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (16 preceding siblings ...)
  2025-01-14  1:30 ` [PR REVIEW] " trolljoe
@ 2025-01-14  1:35 ` chilledfrogs
  2025-01-14  1:36 ` classabbyamp
                   ` (7 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:35 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 308 bytes --]

New review comment by chilledfrogs on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1914008625

Comment:
Methinks it is a typo, but mind the additional quotes:
```shell
pre_check() {
    export XDG_RUNTIME_DIR="$(mktemp -d "$wrksrc/runtime.XXXXXX")"
}
```

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (17 preceding siblings ...)
  2025-01-14  1:35 ` chilledfrogs
@ 2025-01-14  1:36 ` classabbyamp
  2025-01-14  1:37 ` classabbyamp
                   ` (6 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: classabbyamp @ 2025-01-14  1:36 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 307 bytes --]

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1914008861

Comment:
If upstream wants this information to be accurate, they should add it to their release artifacts. they already generate them to vendor cargo dependencies.

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (18 preceding siblings ...)
  2025-01-14  1:36 ` classabbyamp
@ 2025-01-14  1:37 ` classabbyamp
  2025-01-14  1:42 ` [PR PATCH] [Updated] " trolljoe
                   ` (5 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: classabbyamp @ 2025-01-14  1:37 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 179 bytes --]

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1914009319

Comment:
yes mine was missing a `(`

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR PATCH] [Updated] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (19 preceding siblings ...)
  2025-01-14  1:37 ` classabbyamp
@ 2025-01-14  1:42 ` trolljoe
  2025-01-14  1:46 ` [PR REVIEW] " chilledfrogs
                   ` (4 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: trolljoe @ 2025-01-14  1:42 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1405 bytes --]

There is an updated pull request by trolljoe against master on the void-packages repository

https://github.com/trolljoe/void-packages niri
https://github.com/void-linux/void-packages/pull/53946

niri: update to 25.01
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **NO**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->

this goes by the new packaging niri page (https://github.com/YaLTeR/niri/wiki/Packaging-niri), if setting XDG_RUNTIME_DIR to `mktemp` isn't enough i'll skip the tests.

A patch file from https://github.com/void-linux/void-packages/pull/53946.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-niri-53946.patch --]
[-- Type: text/x-diff, Size: 2502 bytes --]

From 1eb21be3e4bae601fc0087a4d22b92012e1bac71 Mon Sep 17 00:00:00 2001
From: Joetroll <joetroll@proton.me>
Date: Sun, 12 Jan 2025 15:23:37 -0300
Subject: [PATCH 1/2] niri: update to 25.01

---
 srcpkgs/niri/template | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/niri/template b/srcpkgs/niri/template
index 0fb83c6b62dbec..26a83b97cf5d90 100644
--- a/srcpkgs/niri/template
+++ b/srcpkgs/niri/template
@@ -1,6 +1,6 @@
 # Template file for 'niri'
 pkgname=niri
-version=0.1.10.1
+version=25.01
 revision=1
 build_style=cargo
 configure_args="--no-default-features --features xdp-gnome-screencast"
@@ -13,7 +13,10 @@ license="GPL-3.0-or-later"
 homepage="https://github.com/YaLTeR/niri"
 changelog="https://github.com/YaLTeR/niri/releases"
 distfiles="https://github.com/YaLTeR/niri/archive/refs/tags/v${version}.tar.gz"
-checksum=d8854830436a87215b0bc6a60b6d43f350d927a03a2798c75f0fbda228bac8d3
+checksum=86b89bcfc3fc6a8ed81f9e3f0ac7a29bd30267515efb2c19e1e0bc2ccd67b649
+
+export NIRI_BUILD_COMMIT="e35c630"
+export XDG_RUNTIME_DIR="$(mktemp -d)"
 
 post_install() {
 	vinstall resources/niri.desktop 644 usr/share/wayland-sessions

From 490a876c38557c44e3a26469c9008a782980bfae Mon Sep 17 00:00:00 2001
From: Joetroll <joetroll@proton.me>
Date: Sun, 12 Jan 2025 15:23:37 -0300
Subject: [PATCH 2/2] niri: update to 25.01

---
 srcpkgs/niri/template | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/niri/template b/srcpkgs/niri/template
index 26a83b97cf5d90..652bc5b63866ab 100644
--- a/srcpkgs/niri/template
+++ b/srcpkgs/niri/template
@@ -3,7 +3,7 @@ pkgname=niri
 version=25.01
 revision=1
 build_style=cargo
-configure_args="--no-default-features --features xdp-gnome-screencast"
+configure_args="--no-default-features --features dbus,xdp-gnome-screencast"
 hostmakedepends="pkg-config clang18-devel"
 makedepends="eudev-libudev-devel libxkbcommon-devel libinput-devel libgbm-devel
  libdisplay-info-devel pipewire-devel pango-devel libseat-devel clang18-devel"
@@ -15,8 +15,9 @@ changelog="https://github.com/YaLTeR/niri/releases"
 distfiles="https://github.com/YaLTeR/niri/archive/refs/tags/v${version}.tar.gz"
 checksum=86b89bcfc3fc6a8ed81f9e3f0ac7a29bd30267515efb2c19e1e0bc2ccd67b649
 
-export NIRI_BUILD_COMMIT="e35c630"
-export XDG_RUNTIME_DIR="$(mktemp -d)"
+pre_check() {
+	export XDG_RUNTIME_DIR=$(mktemp -d "$wrksrc/runtime.XXXXXX")
+}
 
 post_install() {
 	vinstall resources/niri.desktop 644 usr/share/wayland-sessions

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (20 preceding siblings ...)
  2025-01-14  1:42 ` [PR PATCH] [Updated] " trolljoe
@ 2025-01-14  1:46 ` chilledfrogs
  2025-01-14  1:47 ` chilledfrogs
                   ` (3 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:46 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 355 bytes --]

New review comment by chilledfrogs on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1914017272

Comment:
Fair point, I might poke upstream to add that, but then tbh I think it is best to just not add any environment variable, since it will report `(unknown commit)` by default, which is more meaningful here

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR REVIEW] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (21 preceding siblings ...)
  2025-01-14  1:46 ` [PR REVIEW] " chilledfrogs
@ 2025-01-14  1:47 ` chilledfrogs
  2025-01-14  1:56 ` [PR PATCH] [Updated] " trolljoe
                   ` (2 subsequent siblings)
  25 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:47 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 376 bytes --]

New review comment by chilledfrogs on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#discussion_r1914017272

Comment:
Fair point, I might poke upstream to add that, but then tbh I think it is best to just not add any environment variable as @trolljoe did now, since it will report `(unknown commit)` by default, which is more meaningful here

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR PATCH] [Updated] niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (22 preceding siblings ...)
  2025-01-14  1:47 ` chilledfrogs
@ 2025-01-14  1:56 ` trolljoe
  2025-01-14  8:45 ` lukeflo
  2025-01-16 13:23 ` [PR PATCH] [Merged]: " classabbyamp
  25 siblings, 0 replies; 28+ messages in thread
From: trolljoe @ 2025-01-14  1:56 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1405 bytes --]

There is an updated pull request by trolljoe against master on the void-packages repository

https://github.com/trolljoe/void-packages niri
https://github.com/void-linux/void-packages/pull/53946

niri: update to 25.01
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **NO**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->

this goes by the new packaging niri page (https://github.com/YaLTeR/niri/wiki/Packaging-niri), if setting XDG_RUNTIME_DIR to `mktemp` isn't enough i'll skip the tests.

A patch file from https://github.com/void-linux/void-packages/pull/53946.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-niri-53946.patch --]
[-- Type: text/x-diff, Size: 1454 bytes --]

From b9118ffad7b68495e3dea8de55c88e7e7dd33f0d Mon Sep 17 00:00:00 2001
From: Joetroll <joetroll@proton.me>
Date: Sun, 12 Jan 2025 15:23:37 -0300
Subject: [PATCH] niri: update to 25.01

---
 srcpkgs/niri/template | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/niri/template b/srcpkgs/niri/template
index 0fb83c6b62dbec..652bc5b63866ab 100644
--- a/srcpkgs/niri/template
+++ b/srcpkgs/niri/template
@@ -1,9 +1,9 @@
 # Template file for 'niri'
 pkgname=niri
-version=0.1.10.1
+version=25.01
 revision=1
 build_style=cargo
-configure_args="--no-default-features --features xdp-gnome-screencast"
+configure_args="--no-default-features --features dbus,xdp-gnome-screencast"
 hostmakedepends="pkg-config clang18-devel"
 makedepends="eudev-libudev-devel libxkbcommon-devel libinput-devel libgbm-devel
  libdisplay-info-devel pipewire-devel pango-devel libseat-devel clang18-devel"
@@ -13,7 +13,11 @@ license="GPL-3.0-or-later"
 homepage="https://github.com/YaLTeR/niri"
 changelog="https://github.com/YaLTeR/niri/releases"
 distfiles="https://github.com/YaLTeR/niri/archive/refs/tags/v${version}.tar.gz"
-checksum=d8854830436a87215b0bc6a60b6d43f350d927a03a2798c75f0fbda228bac8d3
+checksum=86b89bcfc3fc6a8ed81f9e3f0ac7a29bd30267515efb2c19e1e0bc2ccd67b649
+
+pre_check() {
+	export XDG_RUNTIME_DIR=$(mktemp -d "$wrksrc/runtime.XXXXXX")
+}
 
 post_install() {
 	vinstall resources/niri.desktop 644 usr/share/wayland-sessions

^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (23 preceding siblings ...)
  2025-01-14  1:56 ` [PR PATCH] [Updated] " trolljoe
@ 2025-01-14  8:45 ` lukeflo
  2025-01-16 13:23 ` [PR PATCH] [Merged]: " classabbyamp
  25 siblings, 0 replies; 28+ messages in thread
From: lukeflo @ 2025-01-14  8:45 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 431 bytes --]

New comment by lukeflo on void-packages repository

https://github.com/void-linux/void-packages/pull/53946#issuecomment-2589330232

Comment:
> Might I ask, however, why is the `dbus` feature not enabled?

@trolljoe it wasn't enabled in the former version, if I'm right? However, I didn't have recognised any drawbacks even without `dbus` feature, simply starting niri with `dbus-session-run niri`. Just as information... 😉 


^ permalink raw reply	[flat|nested] 28+ messages in thread

* Re: [PR PATCH] [Merged]: niri: update to 25.01
  2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
                   ` (24 preceding siblings ...)
  2025-01-14  8:45 ` lukeflo
@ 2025-01-16 13:23 ` classabbyamp
  25 siblings, 0 replies; 28+ messages in thread
From: classabbyamp @ 2025-01-16 13:23 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1250 bytes --]

There's a merged pull request on the void-packages repository

niri: update to 25.01
https://github.com/void-linux/void-packages/pull/53946

Description:
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **NO**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->

this goes by the new packaging niri page (https://github.com/YaLTeR/niri/wiki/Packaging-niri), if setting XDG_RUNTIME_DIR to `mktemp` isn't enough i'll skip the tests.

^ permalink raw reply	[flat|nested] 28+ messages in thread

* [PR PATCH] niri: update to 25.01.
@ 2025-01-14  1:06 chilledfrogs
  0 siblings, 0 replies; 28+ messages in thread
From: chilledfrogs @ 2025-01-14  1:06 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1204 bytes --]

There is a new pull request by chilledfrogs against master on the void-packages repository

https://github.com/chilledfrogs/void-packages niri
https://github.com/void-linux/void-packages/pull/53963

niri: update to 25.01.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->

#### Local build testing
- I built this PR locally for my native architecture, (x86_64)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl




A patch file from https://github.com/void-linux/void-packages/pull/53963.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-niri-53963.patch --]
[-- Type: text/x-diff, Size: 1375 bytes --]

From 64be4aad7f4e861d1ff1dc58ff7cb3dbdc61f8ec Mon Sep 17 00:00:00 2001
From: Lydia Sobot <chilledfrogs@disroot.org>
Date: Tue, 14 Jan 2025 02:05:01 +0100
Subject: [PATCH] niri: update to 25.01.

---
 srcpkgs/niri/template | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/niri/template b/srcpkgs/niri/template
index 0fb83c6b62dbec..23c058f3c83cb0 100644
--- a/srcpkgs/niri/template
+++ b/srcpkgs/niri/template
@@ -1,9 +1,9 @@
 # Template file for 'niri'
 pkgname=niri
-version=0.1.10.1
+version=25.01
 revision=1
 build_style=cargo
-configure_args="--no-default-features --features xdp-gnome-screencast"
+configure_args="--no-default-features --features dbus,xdp-gnome-screencast"
 hostmakedepends="pkg-config clang18-devel"
 makedepends="eudev-libudev-devel libxkbcommon-devel libinput-devel libgbm-devel
  libdisplay-info-devel pipewire-devel pango-devel libseat-devel clang18-devel"
@@ -13,7 +13,7 @@ license="GPL-3.0-or-later"
 homepage="https://github.com/YaLTeR/niri"
 changelog="https://github.com/YaLTeR/niri/releases"
 distfiles="https://github.com/YaLTeR/niri/archive/refs/tags/v${version}.tar.gz"
-checksum=d8854830436a87215b0bc6a60b6d43f350d927a03a2798c75f0fbda228bac8d3
+checksum=86b89bcfc3fc6a8ed81f9e3f0ac7a29bd30267515efb2c19e1e0bc2ccd67b649
 
 post_install() {
 	vinstall resources/niri.desktop 644 usr/share/wayland-sessions

^ permalink raw reply	[flat|nested] 28+ messages in thread

end of thread, other threads:[~2025-01-16 13:23 UTC | newest]

Thread overview: 28+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-01-12 18:27 [PR PATCH] niri: update to 25.01 trolljoe
2025-01-12 20:40 ` trolljoe
2025-01-13 10:53 ` lukeflo
2025-01-13 23:45 ` oliver-cfc
2025-01-14  1:09 ` chilledfrogs
2025-01-14  1:09 ` [PR REVIEW] " classabbyamp
2025-01-14  1:09 ` chilledfrogs
2025-01-14  1:10 ` chilledfrogs
2025-01-14  1:11 ` chilledfrogs
2025-01-14  1:12 ` classabbyamp
2025-01-14  1:14 ` classabbyamp
2025-01-14  1:19 ` chilledfrogs
2025-01-14  1:24 ` trolljoe
2025-01-14  1:26 ` chilledfrogs
2025-01-14  1:26 ` chilledfrogs
2025-01-14  1:27 ` chilledfrogs
2025-01-14  1:27 ` chilledfrogs
2025-01-14  1:30 ` [PR REVIEW] " trolljoe
2025-01-14  1:35 ` chilledfrogs
2025-01-14  1:36 ` classabbyamp
2025-01-14  1:37 ` classabbyamp
2025-01-14  1:42 ` [PR PATCH] [Updated] " trolljoe
2025-01-14  1:46 ` [PR REVIEW] " chilledfrogs
2025-01-14  1:47 ` chilledfrogs
2025-01-14  1:56 ` [PR PATCH] [Updated] " trolljoe
2025-01-14  8:45 ` lukeflo
2025-01-16 13:23 ` [PR PATCH] [Merged]: " classabbyamp
2025-01-14  1:06 [PR PATCH] " chilledfrogs

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).