[PR PATCH] river: install session file

[PR PATCH] river: install session file

[tranzystorek-io]

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

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

https://github.com/tranzystorek-io/void-packages river-desktop
https://github.com/void-linux/void-packages/pull/45193

river: install session file
<!-- Uncomment relevant sections and delete options which are not applicable -->

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

Ping @ifreund 

<!--
#### 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
-->


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

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

From bb4624cd949b844963174591cf9c2b2c1f0b00dc Mon Sep 17 00:00:00 2001
From: Marcin Puc <tranzystorek.io@protonmail.com>
Date: Sat, 22 Jul 2023 13:41:58 +0200
Subject: [PATCH] river: install session file

---
 srcpkgs/river/template | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/srcpkgs/river/template b/srcpkgs/river/template
index 506a54dc04910..24f9c53d45ea4 100644
--- a/srcpkgs/river/template
+++ b/srcpkgs/river/template
@@ -1,7 +1,7 @@
 # Template file for 'river'
 pkgname=river
 version=0.2.4
-revision=1
+revision=2
 archs="~i686* ~armv6l* ~armv7l*"
 build_style=zig-build
 configure_args="$(vopt_if xwayland -Dxwayland) -Dpie"
@@ -21,5 +21,6 @@ build_options_default="xwayland"
 desc_option_xwayland="Enable Xwayland support"
 
 post_install() {
+	vinstall contrib/river.desktop 644 usr/share/wayland-sessions
 	vsconf example/init
 }

Re: river: install session file

[ifreund]

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

New comment by ifreund on void-packages repository

https://github.com/void-linux/void-packages/pull/45193#issuecomment-1646578022

Comment:
I think this is a good idea that will benefit at least some users but I personally don't use any kind of display manager that reads these files and don't plan on trying to set one up to test this myself.

Take this as an ACK, I'm cool with this change as the listed package maintainer but haven't confirmed that it works.

Re: river: install session file

[tranzystorek-io]

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

New comment by tranzystorek-io on void-packages repository

https://github.com/void-linux/void-packages/pull/45193#issuecomment-1646578450

Comment:
Maybe a comment about my testing is in order: I've an `emptty` config and verified that it started picking up river after this change.

Re: river: install session file

[Bnyro]

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

New comment by Bnyro on void-packages repository

https://github.com/void-linux/void-packages/pull/45193#issuecomment-1646637344

Comment:
Tested with `emptty` as well here and works as expected

Re: [PR PATCH] [Merged]: river: install session file

[Duncaen]

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

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

river: install session file
https://github.com/void-linux/void-packages/pull/45193

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

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

Ping @ifreund 

<!--
#### 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
-->