[PR PATCH] i3status-rust: update to 0.32.3

[PR PATCH] i3status-rust: update to 0.32.3

[desertsagebrush]

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

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

https://github.com/desertsagebrush/void-packages update-i3status-rust
https://github.com/void-linux/void-packages/pull/47870

i3status-rust: update to 0.32.3
Update the version of `i3status-rust`

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

#### Local build testing
- I built this PR locally for my native architecture, amd64/x86_64-libc
- `xbps-src -Q pkg i3status-rust` completes with no errors

Glad to update anything else if needed!

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-update-i3status-rust-47870.patch --]
[-- Type: text/x-diff, Size: 1719 bytes --]

From b3f42594bbf8ac26bbf8b9a14aad4ee221ecfa55 Mon Sep 17 00:00:00 2001
From: Riley Barello-Myers <10702176+desertsagebrush@users.noreply.github.com>
Date: Thu, 21 Dec 2023 22:38:29 -0700
Subject: [PATCH] i3status-rust: update to 0.32.3

Adds in a hostdependancy for pandoc, as the process for generating
the man page has changed.
---
 srcpkgs/i3status-rust/template | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/srcpkgs/i3status-rust/template b/srcpkgs/i3status-rust/template
index 5e51b37ac809d..df8d85cd7673b 100644
--- a/srcpkgs/i3status-rust/template
+++ b/srcpkgs/i3status-rust/template
@@ -1,10 +1,10 @@
 # Template file for 'i3status-rust'
 pkgname=i3status-rust
-version=0.22.0
-revision=4
+version=0.32.3
+revision=1
 build_style=cargo
 make_check_args="--bins"
-hostmakedepends="pkg-config"
+hostmakedepends="pkg-config pandoc"
 makedepends="dbus-devel pulseaudio-devel openssl-devel libsensors-devel"
 short_desc="Replacement for i3status, written in Rust"
 maintainer="Jan Christian Grünhage <jan.christian@gruenhage.xyz>"
@@ -12,12 +12,13 @@ license="GPL-3.0-only"
 homepage="https://github.com/greshake/i3status-rust"
 changelog="https://raw.githubusercontent.com/greshake/i3status-rust/master/NEWS.md"
 distfiles="https://github.com/greshake/i3status-rust/archive/refs/tags/v${version}.tar.gz"
-checksum=cd28a90ccb2f9faaaef1e528619f1018981609d77f409abe4877350d810d3324
+checksum=6a2c37d0e424d666f297d7ec36279b54a522acf5bf77af883be1991513e4da61
 
 post_install() {
 	vmkdir usr/share/i3status-rust
 	vcopy ${wrksrc}/files/icons usr/share/i3status-rust/
 	vcopy ${wrksrc}/files/themes usr/share/i3status-rust/
 
+    cargo xtask generate-manpage
 	vman man/i3status-rs.1
 }

Re: [PR PATCH] [Updated] i3status-rust: update to 0.32.3

[desertsagebrush]

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

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

https://github.com/desertsagebrush/void-packages update-i3status-rust
https://github.com/void-linux/void-packages/pull/47870

i3status-rust: update to 0.32.3
Update the version of `i3status-rust`

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

#### Local build testing
- I built this PR locally for my native architecture, amd64/x86_64-libc
- `xbps-src -Q pkg i3status-rust` completes with no errors

Glad to update anything else if needed!

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-update-i3status-rust-47870.patch --]
[-- Type: text/x-diff, Size: 1716 bytes --]

From 64fe585b95d65ab958d49f7d2a56b5e84ab4db18 Mon Sep 17 00:00:00 2001
From: Riley Barello-Myers <10702176+desertsagebrush@users.noreply.github.com>
Date: Thu, 21 Dec 2023 22:38:29 -0700
Subject: [PATCH] i3status-rust: update to 0.32.3

Adds in a hostdependancy for pandoc, as the process for generating
the man page has changed.
---
 srcpkgs/i3status-rust/template | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/srcpkgs/i3status-rust/template b/srcpkgs/i3status-rust/template
index 5e51b37ac809d..5fc59d612e685 100644
--- a/srcpkgs/i3status-rust/template
+++ b/srcpkgs/i3status-rust/template
@@ -1,10 +1,10 @@
 # Template file for 'i3status-rust'
 pkgname=i3status-rust
-version=0.22.0
-revision=4
+version=0.32.3
+revision=1
 build_style=cargo
 make_check_args="--bins"
-hostmakedepends="pkg-config"
+hostmakedepends="pkg-config pandoc"
 makedepends="dbus-devel pulseaudio-devel openssl-devel libsensors-devel"
 short_desc="Replacement for i3status, written in Rust"
 maintainer="Jan Christian Grünhage <jan.christian@gruenhage.xyz>"
@@ -12,12 +12,13 @@ license="GPL-3.0-only"
 homepage="https://github.com/greshake/i3status-rust"
 changelog="https://raw.githubusercontent.com/greshake/i3status-rust/master/NEWS.md"
 distfiles="https://github.com/greshake/i3status-rust/archive/refs/tags/v${version}.tar.gz"
-checksum=cd28a90ccb2f9faaaef1e528619f1018981609d77f409abe4877350d810d3324
+checksum=6a2c37d0e424d666f297d7ec36279b54a522acf5bf77af883be1991513e4da61
 
 post_install() {
 	vmkdir usr/share/i3status-rust
 	vcopy ${wrksrc}/files/icons usr/share/i3status-rust/
 	vcopy ${wrksrc}/files/themes usr/share/i3status-rust/
 
+	cargo xtask generate-manpage
 	vman man/i3status-rs.1
 }

Re: [PR PATCH] [Updated] i3status-rust: update to 0.32.3

[desertsagebrush]

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

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

https://github.com/desertsagebrush/void-packages update-i3status-rust
https://github.com/void-linux/void-packages/pull/47870

i3status-rust: update to 0.32.3
Update the version of `i3status-rust`

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

#### Local build testing
- I built this PR locally for my native architecture, amd64/x86_64-libc
- `xbps-src -Q pkg i3status-rust` completes with no errors

Glad to update anything else if needed!

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-update-i3status-rust-47870.patch --]
[-- Type: text/x-diff, Size: 1713 bytes --]

From 425a3384710c1278ba577023bfa4886adc3f74e5 Mon Sep 17 00:00:00 2001
From: Riley Barello-Myers <desertsagebrush@pm.me>
Date: Fri, 22 Dec 2023 11:22:22 -0700
Subject: [PATCH] i3status-rust: update to 0.32.3

Adds in a hostdependancy for pandoc, as the process for generating
the man page has changed.
---
 srcpkgs/i3status-rust/template | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/srcpkgs/i3status-rust/template b/srcpkgs/i3status-rust/template
index 5e51b37ac809d..dd59f7898a616 100644
--- a/srcpkgs/i3status-rust/template
+++ b/srcpkgs/i3status-rust/template
@@ -1,23 +1,24 @@
 # Template file for 'i3status-rust'
 pkgname=i3status-rust
-version=0.22.0
-revision=4
+version=0.32.3
+revision=1
 build_style=cargo
 make_check_args="--bins"
 hostmakedepends="pkg-config"
-makedepends="dbus-devel pulseaudio-devel openssl-devel libsensors-devel"
+makedepends="dbus-devel pulseaudio-devel openssl-devel libsensors-devel pandoc"
 short_desc="Replacement for i3status, written in Rust"
 maintainer="Jan Christian Grünhage <jan.christian@gruenhage.xyz>"
 license="GPL-3.0-only"
 homepage="https://github.com/greshake/i3status-rust"
 changelog="https://raw.githubusercontent.com/greshake/i3status-rust/master/NEWS.md"
 distfiles="https://github.com/greshake/i3status-rust/archive/refs/tags/v${version}.tar.gz"
-checksum=cd28a90ccb2f9faaaef1e528619f1018981609d77f409abe4877350d810d3324
+checksum=6a2c37d0e424d666f297d7ec36279b54a522acf5bf77af883be1991513e4da61
 
 post_install() {
 	vmkdir usr/share/i3status-rust
 	vcopy ${wrksrc}/files/icons usr/share/i3status-rust/
 	vcopy ${wrksrc}/files/themes usr/share/i3status-rust/
 
+	cargo xtask generate-manpage
 	vman man/i3status-rs.1
 }

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[classabbyamp]

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1435604564

Comment:
pandoc can't be cross-built so this makes it impossible to build this for arm/aarch64

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[desertsagebrush]

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

New review comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1435703241

Comment:
Ok; With pandoc as a hostmakedepend, it fails the CI for other architectures (presumably because the cross-compiled cargo tries to call the different-architecture pandoc and then fails). I can't seem to find those CI runs, but I'm glad to change it back if you think that is the way to go.

Alternatively, since pandoc is now required to generate the man page for i3status-rust, do you have any other suggestions?

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[desertsagebrush]

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

New review comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1435703241

Comment:
Ok; With pandoc as a hostmakedepend, it fails the CI for other architectures (presumably because the cross-compiled cargo tries to call the different-architecture pandoc and then fails). Specifically, cargo-xtask fails like this:
```
...
=> i3status-rust-0.32.3_1: running post_install ...
target/aarch64-unknown-linux-gnu/debug/xtask: target/aarch64-unknown-linux-gnu/debug/xtask: cannot execute binary file
=> ERROR: i3status-rust-0.32.3_1: post_install: 'cargo xtask generate-manpage' exited with 126
=> ERROR:   in post_install() at srcpkgs/i3status-rust/template:22
```

Alternatively, since pandoc is now required to generate the man page for i3status-rust, do you have any other suggestions?

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[desertsagebrush]

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

New review comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1435703241

Comment:
Ok; With pandoc as a hostmakedepend, it fails the CI for other architectures (presumably because the cross-compiled cargo tries to call the different-architecture pandoc and then fails). Specifically, cargo-xtask fails like this:
```
...
=> i3status-rust-0.32.3_1: running post_install ...
target/aarch64-unknown-linux-gnu/debug/xtask: target/aarch64-unknown-linux-gnu/debug/xtask: cannot execute binary file
=> ERROR: i3status-rust-0.32.3_1: post_install: 'cargo xtask generate-manpage' exited with 126
=> ERROR:   in post_install() at srcpkgs/i3status-rust/template:22
```

Since pandoc is now required to generate the man page for i3status-rust, do you have any other suggestions?

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[classabbyamp]

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1435706967

Comment:
see how ncspot does it

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[classabbyamp]

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1435707939

Comment:
maybe not using RUST_TARGET so that it builds for the host

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[desertsagebrush]

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

New review comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1436157880

Comment:
So after poking at this a little while, this seems to be complicated by the fact that i3status-rs generates its man page very differently from ncspot (it uses environment variables like CARGO_MANIFEST_DIR and explicitly walks the filesystem to build it, rather than ncspot's approach of just getting the args from the main file; this makes sense as it includes configuration documentation in various source files pertaining to the thing being configured).

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[desertsagebrush]

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

New review comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1436158467

Comment:
Part of the problem is that there are few diagnostics coming out of `vtargetrun`; whenever there is an error (other than an error with args for qemu) it just pops up:
```
=> i3status-rust-0.32.3_1: running post_build ...
    Finished release [optimized] target(s) in 0.09s
=> ERROR: i3status-rust-0.32.3_1: post_build: '"/usr/bin/qemu-${XBPS_TARGET_QEMU_MACHINE}-static" "$@"' exited with 1
=> ERROR:   in vtargetrun() at common/build-helper/qemu.sh:10
=> ERROR:   in post_build() at srcpkgs/i3status-rust/template:20
```
While I can manually poke the generated binary (and the command line I'm passing to it) for x86_64, the lack of diagnostics means that trying to make sure everything works for cross-compiling is a bit more challenging.

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[desertsagebrush]

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

New review comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1436159058

Comment:
Specifically, I've updated it to have this for a post_build (with just this and the addition of the build-helper, it is now failing on all builds...hence why I'm not pushing it):
```
post_build() {
	cargo auditable build --release --package xtask
	vtargetrun  "CARGO_MANIFEST_DIR=./xtask target/x86_64-unknown-linux-gnu/release/xtask generate-manpage"
}
```

Right now the rust target triple for the host architecture is hard-coded, as there doesn't seem to be a generic `xbps-src` variable for that since `XBPS_RUST_TARGET` is for the target. Definitely not

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[desertsagebrush]

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

New review comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1436159058

Comment:
Specifically, I've updated it to have this for a post_build (with just this and the addition of the build-helper, it is now failing on all builds...hence why I'm not pushing it):
```
post_build() {
	cargo auditable build --release --package xtask
	vtargetrun  "CARGO_MANIFEST_DIR=./xtask target/x86_64-unknown-linux-gnu/release/xtask generate-manpage"
}
```

Right now the rust target triple for the host architecture is hard-coded, as there doesn't seem to be a generic `xbps-src` variable for that since `XBPS_RUST_TARGET` is for the target. Definitely not quite ready to be merged yet.

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[desertsagebrush]

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

New review comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1436158467

Comment:
Part of the problem is that there are few diagnostics coming out of `vtargetrun`; whenever there is an error (other than an error with args for qemu) it just pops up:
```
=> i3status-rust-0.32.3_1: running post_build ...
    Finished release [optimized] target(s) in 0.09s
=> ERROR: i3status-rust-0.32.3_1: post_build: '"/usr/bin/qemu-${XBPS_TARGET_QEMU_MACHINE}-static" "$@"' exited with 1
=> ERROR:   in vtargetrun() at common/build-helper/qemu.sh:10
=> ERROR:   in post_build() at srcpkgs/i3status-rust/template:20
```
While I can manually poke the generated binary (and the command line I'm passing to it) for x86_64, the lack of diagnostics means that trying to make sure everything works for cross-compiling is a bit more challenging. Using the qemu helper also means that it is specifically calling the target version of qemu-static...

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[desertsagebrush]

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

New review comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1436159058

Comment:
Specifically, I've updated it to have this for a post_build (with just this and the addition of the build-helper, it is now failing on all builds...hence why I'm not pushing it):
```
post_build() {
	cargo auditable build --release --target x86_64-unknown-linux-gnu --package xtask
	vtargetrun  "CARGO_MANIFEST_DIR=./xtask target/x86_64-unknown-linux-gnu/release/xtask generate-manpage"
}
```

Right now the rust target triple for the host architecture is hard-coded, as there doesn't seem to be a generic `xbps-src` variable for that since `XBPS_RUST_TARGET` is for the target. Definitely not quite ready to be merged yet.

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[desertsagebrush]

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

New review comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1436159058

Comment:
Specifically, I've updated it to have this for a post_build (with just this and the addition of the build-helper, it is now failing on all builds...hence why I'm not pushing it):
```
post_build() {
	cargo auditable build --release --target x86_64-unknown-linux-gnu --package xtask
	vtargetrun  "CARGO_MANIFEST_DIR=./xtask target/x86_64-unknown-linux-gnu/release/xtask generate-manpage"
}
```

Right now the rust target triple for the host architecture is hard-coded, as there doesn't seem to be a generic `xbps-src` variable for that since `XBPS_RUST_TARGET` is for the target. Definitely not quite ready to be merged yet. 

For cross-compiling in particular this seems to fail because the x86_64 rust stdlib isn't installed.

Re: [PR PATCH] [Updated] i3status-rust: update to 0.32.3

[desertsagebrush]

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

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

https://github.com/desertsagebrush/void-packages update-i3status-rust
https://github.com/void-linux/void-packages/pull/47870

i3status-rust: update to 0.32.3
Update the version of `i3status-rust`

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

#### Local build testing
- I built this PR locally for my native architecture, amd64/x86_64-libc
- `xbps-src -Q pkg i3status-rust` completes with no errors

Glad to update anything else if needed!

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-update-i3status-rust-47870.patch --]
[-- Type: text/x-diff, Size: 2009 bytes --]

From cea1ac8bcb8ebe79f96bdc4b59cd322e5479c6e5 Mon Sep 17 00:00:00 2001
From: Riley Barello-Myers <desertsagebrush@pm.me>
Date: Fri, 22 Dec 2023 11:22:22 -0700
Subject: [PATCH] i3status-rust: update to 0.32.3

Adds in a hostdependancy for pandoc, as the process for generating
the man page has changed. This also necessitates a more complex
post-build procedure.
---
 srcpkgs/i3status-rust/template | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

diff --git a/srcpkgs/i3status-rust/template b/srcpkgs/i3status-rust/template
index 5e51b37ac809d..8397c4da9818b 100644
--- a/srcpkgs/i3status-rust/template
+++ b/srcpkgs/i3status-rust/template
@@ -1,10 +1,10 @@
 # Template file for 'i3status-rust'
 pkgname=i3status-rust
-version=0.22.0
-revision=4
+version=0.32.3
+revision=1
 build_style=cargo
 make_check_args="--bins"
-hostmakedepends="pkg-config"
+hostmakedepends="pkg-config pandoc qemu-user-static"
 makedepends="dbus-devel pulseaudio-devel openssl-devel libsensors-devel"
 short_desc="Replacement for i3status, written in Rust"
 maintainer="Jan Christian Grünhage <jan.christian@gruenhage.xyz>"
@@ -12,7 +12,18 @@ license="GPL-3.0-only"
 homepage="https://github.com/greshake/i3status-rust"
 changelog="https://raw.githubusercontent.com/greshake/i3status-rust/master/NEWS.md"
 distfiles="https://github.com/greshake/i3status-rust/archive/refs/tags/v${version}.tar.gz"
-checksum=cd28a90ccb2f9faaaef1e528619f1018981609d77f409abe4877350d810d3324
+checksum=6a2c37d0e424d666f297d7ec36279b54a522acf5bf77af883be1991513e4da61
+
+post_build() {
+	cargo auditable build --release --target x86_64-unknown-linux-gnu --package xtask
+	
+#	if [ "$CROSS_BUILD" ]; then
+#		/usr/bin/qemu-x86_64-static -E "CARGO_MANIFEST_DIR=./xtask" target/x86_64-unknown-linux-gnu/release/xtask generate-manpage
+#	else
+#		cargo xtask generate-manpage
+#	fi
+	CARGO_MANIFEST_DIR=./xtask target/x86_64-unknown-linux-gnu/release/xtask generate-manpage
+}
 
 post_install() {
 	vmkdir usr/share/i3status-rust

Re: i3status-rust: update to 0.32.3

[desertsagebrush]

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

New comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#issuecomment-1869141630

Comment:
It's not worth re-running CI on this newest push; it still fails for cross-compiling (seemingly) due to the lack of an x86_64 rust stdlib in the cross-compilation environment.

Re: [PR REVIEW] i3status-rust: update to 0.32.3

[desertsagebrush]

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

New review comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#discussion_r1436159058

Comment:
Specifically, I've updated it to have this for a post_build (with just this and the addition of the build-helper, it is now failing on all builds...hence why I'm not pushing it):
```
post_build() {
	cargo auditable build --release --target x86_64-unknown-linux-gnu --package xtask
	vtargetrun  "CARGO_MANIFEST_DIR=./xtask target/x86_64-unknown-linux-gnu/release/xtask generate-manpage"
}
```

Right now the rust target triple for the host architecture is hard-coded, as there doesn't seem to be a generic `xbps-src` variable for that since `XBPS_RUST_TARGET` is for the target. Definitely not quite ready to be merged yet. 

For cross-compiling in particular this seems to fail because the x86_64 rust stdlib isn't installed.

EDIT: This code snippet is out-of-date, see the updated push.

Re: i3status-rust: update to 0.32.3

[desertsagebrush]

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

New comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#issuecomment-1869141630

Comment:
It's not worth re-running CI on this newest push; it still fails for cross-compiling (seemingly) due to the lack of an x86_64 rust stdlib in the cross-compilation environment. It now (correctly) builds for x86_64 though.....

Re: i3status-rust: update to 0.32.3

[desertsagebrush]

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

New comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#issuecomment-1873023583

Comment:
I'm still (slightly) poking at this, but if anyone has suggestions off the top of their head, I'm glad to see about incorporating them.

Re: [PR PATCH] [Updated] i3status-rust: update to 0.32.3

[classabbyamp]

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

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

https://github.com/desertsagebrush/void-packages update-i3status-rust
https://github.com/void-linux/void-packages/pull/47870

i3status-rust: update to 0.32.3
Update the version of `i3status-rust`

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

#### Local build testing
- I built this PR locally for my native architecture, amd64/x86_64-libc
- `xbps-src -Q pkg i3status-rust` completes with no errors

Glad to update anything else if needed!

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-update-i3status-rust-47870.patch --]
[-- Type: text/x-diff, Size: 1973 bytes --]

From 390940b9c0496bcc5a5d77e45f0a691de11a769c Mon Sep 17 00:00:00 2001
From: Riley Barello-Myers <desertsagebrush@pm.me>
Date: Fri, 22 Dec 2023 11:22:22 -0700
Subject: [PATCH] i3status-rust: update to 0.32.3

Adds in a hostdependancy for pandoc, as the process for generating
the man page has changed.
---
 srcpkgs/i3status-rust/template | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/srcpkgs/i3status-rust/template b/srcpkgs/i3status-rust/template
index 5e51b37ac809d..7db6e3a5171e2 100644
--- a/srcpkgs/i3status-rust/template
+++ b/srcpkgs/i3status-rust/template
@@ -1,10 +1,11 @@
 # Template file for 'i3status-rust'
 pkgname=i3status-rust
-version=0.22.0
-revision=4
+version=0.32.3
+revision=1
 build_style=cargo
+build_helper="qemu"
 make_check_args="--bins"
-hostmakedepends="pkg-config"
+hostmakedepends="pkg-config pandoc"
 makedepends="dbus-devel pulseaudio-devel openssl-devel libsensors-devel"
 short_desc="Replacement for i3status, written in Rust"
 maintainer="Jan Christian Grünhage <jan.christian@gruenhage.xyz>"
@@ -12,12 +13,16 @@ license="GPL-3.0-only"
 homepage="https://github.com/greshake/i3status-rust"
 changelog="https://raw.githubusercontent.com/greshake/i3status-rust/master/NEWS.md"
 distfiles="https://github.com/greshake/i3status-rust/archive/refs/tags/v${version}.tar.gz"
-checksum=cd28a90ccb2f9faaaef1e528619f1018981609d77f409abe4877350d810d3324
+checksum=6a2c37d0e424d666f297d7ec36279b54a522acf5bf77af883be1991513e4da61
+
+post_build() {
+	cargo auditable build --release --target "${RUST_TARGET}" --package xtask
+	CARGO_MANIFEST_DIR=./xtask vtargetrun target/"${RUST_TARGET}"/release/xtask generate-manpage
+}
 
 post_install() {
 	vmkdir usr/share/i3status-rust
-	vcopy ${wrksrc}/files/icons usr/share/i3status-rust/
-	vcopy ${wrksrc}/files/themes usr/share/i3status-rust/
-
+	vcopy files/icons usr/share/i3status-rust/
+	vcopy files/themes usr/share/i3status-rust/
 	vman man/i3status-rs.1
 }

Re: [PR PATCH] [Updated] i3status-rust: update to 0.32.3

[classabbyamp]

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

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

https://github.com/desertsagebrush/void-packages update-i3status-rust
https://github.com/void-linux/void-packages/pull/47870

i3status-rust: update to 0.32.3
Update the version of `i3status-rust`

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

#### Local build testing
- I built this PR locally for my native architecture, amd64/x86_64-libc
- `xbps-src -Q pkg i3status-rust` completes with no errors

Glad to update anything else if needed!

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-update-i3status-rust-47870.patch --]
[-- Type: text/x-diff, Size: 1973 bytes --]

From e798372007e50eb4da82f76b7b3682594982b737 Mon Sep 17 00:00:00 2001
From: Riley Barello-Myers <desertsagebrush@pm.me>
Date: Fri, 22 Dec 2023 11:22:22 -0700
Subject: [PATCH] i3status-rust: update to 0.32.3

Adds in a hostdependancy for pandoc, as the process for generating
the man page has changed.
---
 srcpkgs/i3status-rust/template | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/srcpkgs/i3status-rust/template b/srcpkgs/i3status-rust/template
index 5e51b37ac809d..7db6e3a5171e2 100644
--- a/srcpkgs/i3status-rust/template
+++ b/srcpkgs/i3status-rust/template
@@ -1,10 +1,11 @@
 # Template file for 'i3status-rust'
 pkgname=i3status-rust
-version=0.22.0
-revision=4
+version=0.32.3
+revision=1
 build_style=cargo
+build_helper="qemu"
 make_check_args="--bins"
-hostmakedepends="pkg-config"
+hostmakedepends="pkg-config pandoc"
 makedepends="dbus-devel pulseaudio-devel openssl-devel libsensors-devel"
 short_desc="Replacement for i3status, written in Rust"
 maintainer="Jan Christian Grünhage <jan.christian@gruenhage.xyz>"
@@ -12,12 +13,16 @@ license="GPL-3.0-only"
 homepage="https://github.com/greshake/i3status-rust"
 changelog="https://raw.githubusercontent.com/greshake/i3status-rust/master/NEWS.md"
 distfiles="https://github.com/greshake/i3status-rust/archive/refs/tags/v${version}.tar.gz"
-checksum=cd28a90ccb2f9faaaef1e528619f1018981609d77f409abe4877350d810d3324
+checksum=6a2c37d0e424d666f297d7ec36279b54a522acf5bf77af883be1991513e4da61
+
+post_build() {
+	cargo auditable build --release --target "${RUST_TARGET}" --package xtask
+	CARGO_MANIFEST_DIR=./xtask vtargetrun target/"${RUST_TARGET}"/release/xtask generate-manpage
+}
 
 post_install() {
 	vmkdir usr/share/i3status-rust
-	vcopy ${wrksrc}/files/icons usr/share/i3status-rust/
-	vcopy ${wrksrc}/files/themes usr/share/i3status-rust/
-
+	vcopy files/icons usr/share/i3status-rust/
+	vcopy files/themes usr/share/i3status-rust/
 	vman man/i3status-rs.1
 }

Re: i3status-rust: update to 0.32.3

[classabbyamp]

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#issuecomment-1879940686

Comment:
fixed it up

Re: [PR PATCH] [Merged]: i3status-rust: update to 0.32.3

[classabbyamp]

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

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

i3status-rust: update to 0.32.3
https://github.com/void-linux/void-packages/pull/47870

Description:
Update the version of `i3status-rust`

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

#### Local build testing
- I built this PR locally for my native architecture, amd64/x86_64-libc
- `xbps-src -Q pkg i3status-rust` completes with no errors

Glad to update anything else if needed!

Re: i3status-rust: update to 0.32.3

[desertsagebrush]

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

New comment by desertsagebrush on void-packages repository

https://github.com/void-linux/void-packages/pull/47870#issuecomment-1890726685

Comment:
Thank you!