Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] New package: eww
@ 2024-02-20  2:03 shmishtopher
  2024-02-20  2:04 ` shmishtopher
                   ` (19 more replies)
  0 siblings, 20 replies; 21+ messages in thread
From: shmishtopher @ 2024-02-20  2:03 UTC (permalink / raw)
  To: ml

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

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

https://github.com/shmishtopher/void-packages package/eww
https://github.com/void-linux/void-packages/pull/48844

New package: eww
#### Testing the changes
- I tested the changes in this PR: **briefly**

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

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
- I built this PR locally for these architectures:
  - aarch64-musl (crossbuild)
  - armv7l (crossbuild)
  - armv6l-musl (crossbuild)

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

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

From 48c64bb62b6c68868e8b629e38a213a0e526676e Mon Sep 17 00:00:00 2001
From: "Christopher K. \"Shmish\" Schmitt" <me@shmish.dev>
Date: Mon, 19 Feb 2024 20:29:43 -0500
Subject: [PATCH] package: Add eww

---
 srcpkgs/eww/template | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 srcpkgs/eww/template

diff --git a/srcpkgs/eww/template b/srcpkgs/eww/template
new file mode 100644
index 0000000000000..8437f05d66338
--- /dev/null
+++ b/srcpkgs/eww/template
@@ -0,0 +1,18 @@
+# Template file for 'eww'
+pkgname=eww
+version=0.5.0
+revision=1
+build_style=cargo
+make_install_args="--path=crates/eww"
+hostmakedepends="pkg-config glibc-devel libgcc-devel"
+makedepends="gtk+3-devel gtk-layer-shell-devel pango-devel gdk-pixbuf-devel cairo-devel glib-devel"
+short_desc="ElKowar's Wacky Widgets - a toolkit for creating custom bars and widgets"
+maintainer="Christopher K. 'Shmish' Schmitt <me@shmish.dev>"
+license="MIT"
+homepage="https://github.com/elkowar/eww"
+distfiles="https://github.com/elkowar/eww/archive/refs/tags/v${version}.tar.gz"
+checksum=ea4f62e48e3750a361e0f359933d7d840d158592ff5b3683ba1f3ccf42bda819
+
+post_install() {
+	vlicense LICENSE
+}

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

* Re: New package: eww
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
@ 2024-02-20  2:04 ` shmishtopher
  2024-02-20  4:50 ` classabbyamp
                   ` (18 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: shmishtopher @ 2024-02-20  2:04 UTC (permalink / raw)
  To: ml

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

New comment by shmishtopher on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#issuecomment-1953372266

Comment:
As of version `0.5.0`, eww no longer depends on nightly rust, a blocker for #37234

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

* Re: New package: eww
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
  2024-02-20  2:04 ` shmishtopher
@ 2024-02-20  4:50 ` classabbyamp
  2024-02-20  4:54 ` [PR PATCH] [Updated] " shmishtopher
                   ` (17 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: classabbyamp @ 2024-02-20  4:50 UTC (permalink / raw)
  To: ml

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#issuecomment-1953484585

Comment:
commit message should be `New package: eww-0.5.0`

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

* Re: [PR PATCH] [Updated] New package: eww
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
  2024-02-20  2:04 ` shmishtopher
  2024-02-20  4:50 ` classabbyamp
@ 2024-02-20  4:54 ` shmishtopher
  2024-02-20  4:55 ` New package: eww-0.5.0 shmishtopher
                   ` (16 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: shmishtopher @ 2024-02-20  4:54 UTC (permalink / raw)
  To: ml

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

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

https://github.com/shmishtopher/void-packages package/eww
https://github.com/void-linux/void-packages/pull/48844

New package: eww
#### Testing the changes
- I tested the changes in this PR: **briefly**

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

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
- I built this PR locally for these architectures:
  - aarch64-musl (crossbuild)
  - armv7l (crossbuild)
  - armv6l-musl (crossbuild)

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

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

From 831ca32fe1d6306ff478a3ef3f0a97a524da43b6 Mon Sep 17 00:00:00 2001
From: "Christopher K. \"Shmish\" Schmitt" <me@shmish.dev>
Date: Mon, 19 Feb 2024 20:29:43 -0500
Subject: [PATCH] New package: eww-0.5.0

---
 srcpkgs/eww/template | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 srcpkgs/eww/template

diff --git a/srcpkgs/eww/template b/srcpkgs/eww/template
new file mode 100644
index 0000000000000..8437f05d66338
--- /dev/null
+++ b/srcpkgs/eww/template
@@ -0,0 +1,18 @@
+# Template file for 'eww'
+pkgname=eww
+version=0.5.0
+revision=1
+build_style=cargo
+make_install_args="--path=crates/eww"
+hostmakedepends="pkg-config glibc-devel libgcc-devel"
+makedepends="gtk+3-devel gtk-layer-shell-devel pango-devel gdk-pixbuf-devel cairo-devel glib-devel"
+short_desc="ElKowar's Wacky Widgets - a toolkit for creating custom bars and widgets"
+maintainer="Christopher K. 'Shmish' Schmitt <me@shmish.dev>"
+license="MIT"
+homepage="https://github.com/elkowar/eww"
+distfiles="https://github.com/elkowar/eww/archive/refs/tags/v${version}.tar.gz"
+checksum=ea4f62e48e3750a361e0f359933d7d840d158592ff5b3683ba1f3ccf42bda819
+
+post_install() {
+	vlicense LICENSE
+}

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

* Re: New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (2 preceding siblings ...)
  2024-02-20  4:54 ` [PR PATCH] [Updated] " shmishtopher
@ 2024-02-20  4:55 ` shmishtopher
  2024-02-20  5:51 ` [PR REVIEW] " classabbyamp
                   ` (15 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: shmishtopher @ 2024-02-20  4:55 UTC (permalink / raw)
  To: ml

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

New comment by shmishtopher on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#issuecomment-1953487422

Comment:
My apologies, should be corrected now

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

* Re: [PR REVIEW] New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (4 preceding siblings ...)
  2024-02-20  5:51 ` [PR REVIEW] " classabbyamp
@ 2024-02-20  5:51 ` classabbyamp
  2024-02-20  5:51 ` classabbyamp
                   ` (13 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: classabbyamp @ 2024-02-20  5:51 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#discussion_r1495272713

Comment:
wrap at ~80 chars
```suggestion
makedepends="gtk+3-devel gtk-layer-shell-devel pango-devel gdk-pixbuf-devel 
 cairo-devel glib-devel"
```

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

* Re: [PR REVIEW] New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (5 preceding siblings ...)
  2024-02-20  5:51 ` classabbyamp
@ 2024-02-20  5:51 ` classabbyamp
  2024-02-20  5:51 ` classabbyamp
                   ` (12 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: classabbyamp @ 2024-02-20  5:51 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#discussion_r1495260060

Comment:
```suggestion
build_style=cargo
build_helper="qemu"
```
allows us to use `vtargetrun` below

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

* Re: [PR REVIEW] New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (3 preceding siblings ...)
  2024-02-20  4:55 ` New package: eww-0.5.0 shmishtopher
@ 2024-02-20  5:51 ` classabbyamp
  2024-02-20  5:51 ` classabbyamp
                   ` (14 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: classabbyamp @ 2024-02-20  5:51 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#discussion_r1495274012

Comment:
this also requires [this patch](https://github.com/elkowar/eww/pull/1029)

```
mkdir srcpkgs/eww/patches
curl -L https://github.com/elkowar/eww/pull/1029.patch > srcpkgs/eww/patches/fix-completions.patch
```

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

* Re: [PR REVIEW] New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (9 preceding siblings ...)
  2024-02-20  5:51 ` classabbyamp
@ 2024-02-20  5:51 ` classabbyamp
  2024-02-20 14:42 ` [PR PATCH] [Updated] " shmishtopher
                   ` (8 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: classabbyamp @ 2024-02-20  5:51 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#discussion_r1495260168

Comment:
```suggestion
short_desc="Toolkit for creating custom bars and widgets"
```

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

* Re: [PR REVIEW] New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (8 preceding siblings ...)
  2024-02-20  5:51 ` classabbyamp
@ 2024-02-20  5:51 ` classabbyamp
  2024-02-20  5:51 ` classabbyamp
                   ` (9 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: classabbyamp @ 2024-02-20  5:51 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#discussion_r1495264093

Comment:
```suggestion
hostmakedepends="pkg-config"
```
do not appear to be needed

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

* Re: [PR REVIEW] New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (6 preceding siblings ...)
  2024-02-20  5:51 ` classabbyamp
@ 2024-02-20  5:51 ` classabbyamp
  2024-02-20  5:51 ` classabbyamp
                   ` (11 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: classabbyamp @ 2024-02-20  5:51 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#discussion_r1495259963

Comment:
```suggestion
	vlicense LICENSE
	for sh in bash fish zsh; do
		vtargetrun "${DESTDIR}/usr/bin/eww" shell-completions -s "$sh" > "eww.$sh"
		vcompletion "eww.$sh" "$sh"
	done
```


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

* Re: [PR REVIEW] New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (7 preceding siblings ...)
  2024-02-20  5:51 ` classabbyamp
@ 2024-02-20  5:51 ` classabbyamp
  2024-02-20  5:51 ` classabbyamp
                   ` (10 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: classabbyamp @ 2024-02-20  5:51 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#discussion_r1495275294

Comment:
```suggestion
homepage="https://elkowar.github.io/eww"
changelog="https://raw.githubusercontent.com/elkowar/eww/master/CHANGELOG.md"
```

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

* Re: [PR PATCH] [Updated] New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (10 preceding siblings ...)
  2024-02-20  5:51 ` classabbyamp
@ 2024-02-20 14:42 ` shmishtopher
  2024-02-20 15:30 ` shmishtopher
                   ` (7 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: shmishtopher @ 2024-02-20 14:42 UTC (permalink / raw)
  To: ml

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

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

https://github.com/shmishtopher/void-packages package/eww
https://github.com/void-linux/void-packages/pull/48844

New package: eww-0.5.0
#### Testing the changes
- I tested the changes in this PR: **briefly**

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

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
- I built this PR locally for these architectures:
  - aarch64-musl (crossbuild)
  - armv7l (crossbuild)
  - armv6l-musl (crossbuild)

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

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

From 831ca32fe1d6306ff478a3ef3f0a97a524da43b6 Mon Sep 17 00:00:00 2001
From: "Christopher K. \"Shmish\" Schmitt" <me@shmish.dev>
Date: Mon, 19 Feb 2024 20:29:43 -0500
Subject: [PATCH] New package: eww-0.5.0

---
 srcpkgs/eww/template | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 srcpkgs/eww/template

diff --git a/srcpkgs/eww/template b/srcpkgs/eww/template
new file mode 100644
index 0000000000000..8437f05d66338
--- /dev/null
+++ b/srcpkgs/eww/template
@@ -0,0 +1,18 @@
+# Template file for 'eww'
+pkgname=eww
+version=0.5.0
+revision=1
+build_style=cargo
+make_install_args="--path=crates/eww"
+hostmakedepends="pkg-config glibc-devel libgcc-devel"
+makedepends="gtk+3-devel gtk-layer-shell-devel pango-devel gdk-pixbuf-devel cairo-devel glib-devel"
+short_desc="ElKowar's Wacky Widgets - a toolkit for creating custom bars and widgets"
+maintainer="Christopher K. 'Shmish' Schmitt <me@shmish.dev>"
+license="MIT"
+homepage="https://github.com/elkowar/eww"
+distfiles="https://github.com/elkowar/eww/archive/refs/tags/v${version}.tar.gz"
+checksum=ea4f62e48e3750a361e0f359933d7d840d158592ff5b3683ba1f3ccf42bda819
+
+post_install() {
+	vlicense LICENSE
+}

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

* Re: [PR PATCH] [Updated] New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (11 preceding siblings ...)
  2024-02-20 14:42 ` [PR PATCH] [Updated] " shmishtopher
@ 2024-02-20 15:30 ` shmishtopher
  2024-02-20 15:31 ` [PR REVIEW] " shmishtopher
                   ` (6 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: shmishtopher @ 2024-02-20 15:30 UTC (permalink / raw)
  To: ml

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

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

https://github.com/shmishtopher/void-packages package/eww
https://github.com/void-linux/void-packages/pull/48844

New package: eww-0.5.0
#### Testing the changes
- I tested the changes in this PR: **briefly**

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

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
- I built this PR locally for these architectures:
  - aarch64-musl (crossbuild)
  - armv7l (crossbuild)
  - armv6l-musl (crossbuild)

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

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

From 25d28e268c8ac68b34bdb36a84fe4e7ede9592f7 Mon Sep 17 00:00:00 2001
From: "Christopher K. \"Shmish\" Schmitt" <me@shmish.dev>
Date: Mon, 19 Feb 2024 20:29:43 -0500
Subject: [PATCH] New package: eww-0.5.0

---
 srcpkgs/eww/patches/fix-completions.patch | 129 ++++++++++++++++++++++
 srcpkgs/eww/template                      |  25 +++++
 2 files changed, 154 insertions(+)
 create mode 100644 srcpkgs/eww/patches/fix-completions.patch
 create mode 100644 srcpkgs/eww/template

diff --git a/srcpkgs/eww/patches/fix-completions.patch b/srcpkgs/eww/patches/fix-completions.patch
new file mode 100644
index 0000000000000..224f676c5addc
--- /dev/null
+++ b/srcpkgs/eww/patches/fix-completions.patch
@@ -0,0 +1,129 @@
+From 525fb069b7eb3213dcbbd1e12bb476b546f63b82 Mon Sep 17 00:00:00 2001
+From: Zacharie Dubrulle <dubrullezacharie@gmail.com>
+Date: Mon, 19 Feb 2024 22:02:57 +0100
+Subject: [PATCH 1/2] Handle `shell-completions` before anything else
+
+---
+ crates/eww/src/client.rs |  4 ----
+ crates/eww/src/main.rs   |  8 ++++++++
+ crates/eww/src/opts.rs   | 14 +++++++-------
+ 3 files changed, 15 insertions(+), 11 deletions(-)
+
+diff --git a/crates/eww/src/client.rs b/crates/eww/src/client.rs
+index c9f14ab2..616c3872 100644
+--- a/crates/eww/src/client.rs
++++ b/crates/eww/src/client.rs
+@@ -6,7 +6,6 @@ use crate::{
+     paths::EwwPaths,
+ };
+ use anyhow::{Context, Result};
+-use clap::CommandFactory as _;
+ use std::{
+     io::{Read, Write},
+     os::unix::net::UnixStream,
+@@ -21,9 +20,6 @@ pub fn handle_client_only_action(paths: &EwwPaths, action: ActionClientOnly) ->
+                 .spawn()?
+                 .wait()?;
+         }
+-        ActionClientOnly::ShellCompletions { shell } => {
+-            clap_complete::generate(shell, &mut opts::RawOpt::command(), "eww", &mut std::io::stdout());
+-        }
+     }
+     Ok(())
+ }
+diff --git a/crates/eww/src/main.rs b/crates/eww/src/main.rs
+index aac52a6f..848c055b 100644
+--- a/crates/eww/src/main.rs
++++ b/crates/eww/src/main.rs
+@@ -5,6 +5,7 @@ extern crate gtk;
+ extern crate gtk_layer_shell as gtk_layer_shell;
+ 
+ use anyhow::{Context, Result};
++use clap::CommandFactory as _;
+ use daemon_response::{DaemonResponse, DaemonResponseReceiver};
+ use display_backend::DisplayBackend;
+ use opts::ActionWithServer;
+@@ -44,6 +45,11 @@ fn main() {
+         pretty_env_logger::formatted_timed_builder().filter(Some("eww"), log_level_filter).init();
+     }
+ 
++    if let opts::Action::ShellCompletions { shell } = opts.action {
++        clap_complete::generate(shell, &mut opts::RawOpt::command(), "eww", &mut std::io::stdout());
++        return;
++    }
++
+     #[allow(unused)]
+     let use_wayland = opts.force_wayland || detect_wayland();
+     #[cfg(all(feature = "wayland", feature = "x11"))]
+@@ -87,6 +93,7 @@ fn run<B: DisplayBackend>(opts: opts::Opt, eww_binary_name: String, display_back
+         .context("Failed to initialize eww paths")?;
+ 
+     let should_restart = match &opts.action {
++        opts::Action::ShellCompletions { .. } => unreachable!(),
+         opts::Action::Daemon => opts.restart,
+         opts::Action::WithServer(action) => opts.restart && action.can_start_daemon(),
+         opts::Action::ClientOnly(_) => false,
+@@ -100,6 +107,7 @@ fn run<B: DisplayBackend>(opts: opts::Opt, eww_binary_name: String, display_back
+     }
+ 
+     let would_show_logs = match opts.action {
++        opts::Action::ShellCompletions { .. } => unreachable!(),
+         opts::Action::ClientOnly(action) => {
+             client::handle_client_only_action(&paths, action)?;
+             false
+diff --git a/crates/eww/src/opts.rs b/crates/eww/src/opts.rs
+index 2446d8c4..b635a28b 100644
+--- a/crates/eww/src/opts.rs
++++ b/crates/eww/src/opts.rs
+@@ -59,6 +59,13 @@ pub(super) struct RawOpt {
+ 
+ #[derive(Subcommand, Debug, Serialize, Deserialize, PartialEq)]
+ pub enum Action {
++    /// Generate a shell completion script
++    ShellCompletions {
++        #[arg(short, long)]
++        #[serde(with = "serde_shell")]
++        shell: clap_complete::shells::Shell,
++    },
++
+     /// Start the Eww daemon.
+     #[command(name = "daemon", alias = "d")]
+     Daemon,
+@@ -75,13 +82,6 @@ pub enum ActionClientOnly {
+     /// Print and watch the eww logs
+     #[command(name = "logs")]
+     Logs,
+-
+-    /// Generate a shell completion script
+-    ShellCompletions {
+-        #[arg(short, long)]
+-        #[serde(with = "serde_shell")]
+-        shell: clap_complete::shells::Shell,
+-    },
+ }
+ 
+ #[derive(Subcommand, Debug, Serialize, Deserialize, PartialEq)]
+
+From 0b22ea7ef053fa8afebf5f6e107f689f5516ac38 Mon Sep 17 00:00:00 2001
+From: Zacharie Dubrulle <dubrullezacharie@gmail.com>
+Date: Mon, 19 Feb 2024 22:05:36 +0100
+Subject: [PATCH 2/2] Update CHANGELOG.md
+
+---
+ CHANGELOG.md | 3 +++
+ 1 file changed, 3 insertions(+)
+
+diff --git a/CHANGELOG.md b/CHANGELOG.md
+index 09914410..f34c078a 100644
+--- a/CHANGELOG.md
++++ b/CHANGELOG.md
+@@ -4,6 +4,9 @@ All notable changes to eww will be listed here, starting at changes since versio
+ 
+ ## Unreleased
+ 
++### Fixes
++- The `shell-completions` subcommand is now run before anything is set up
++
+ ## [0.5.0] (17.02.2024)
+ 
+ ### BREAKING CHANGES
diff --git a/srcpkgs/eww/template b/srcpkgs/eww/template
new file mode 100644
index 0000000000000..1eb64d3a25679
--- /dev/null
+++ b/srcpkgs/eww/template
@@ -0,0 +1,25 @@
+# Template file for 'eww'
+pkgname=eww
+version=0.5.0
+revision=1
+build_style=cargo
+build_helper="qemu"
+make_install_args="--path=crates/eww"
+hostmakedepends="pkg-config"
+makedepends="gtk+3-devel gtk-layer-shell-devel pango-devel gdk-pixbuf-devel
+ cairo-devel glib-devel"
+short_desc="Toolkit for creating custom bars and widgets"
+maintainer="Christopher K. 'Shmish' Schmitt <me@shmish.dev>"
+license="MIT"
+homepage="https://elkowar.github.io/eww"
+changelog="https://raw.githubusercontent.com/elkowar/eww/master/CHANGELOG.md"
+distfiles="https://github.com/elkowar/eww/archive/refs/tags/v${version}.tar.gz"
+checksum=ea4f62e48e3750a361e0f359933d7d840d158592ff5b3683ba1f3ccf42bda819
+
+post_install() {
+	vlicense LICENSE
+	for sh in bash fish zsh; do
+		vtargetrun "${DESTDIR}/usr/bin/eww" shell-completions -s "$sh" > "eww.$sh"
+		vcompletion "eww.$sh" "$sh"
+	done
+}

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

* Re: [PR REVIEW] New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (12 preceding siblings ...)
  2024-02-20 15:30 ` shmishtopher
@ 2024-02-20 15:31 ` shmishtopher
  2024-02-20 15:32 ` shmishtopher
                   ` (5 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: shmishtopher @ 2024-02-20 15:31 UTC (permalink / raw)
  To: ml

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

New review comment by shmishtopher on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#discussion_r1496030273

Comment:
Perfect!  I was ran into issues with completions and was going to submit a patch.  I didn't realize there was already one available.

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

* Re: New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (13 preceding siblings ...)
  2024-02-20 15:31 ` [PR REVIEW] " shmishtopher
@ 2024-02-20 15:32 ` shmishtopher
  2024-02-21  2:54 ` [PR PATCH] [Updated] " shmishtopher
                   ` (4 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: shmishtopher @ 2024-02-20 15:32 UTC (permalink / raw)
  To: ml

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

New comment by shmishtopher on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#issuecomment-1954474148

Comment:
Requested changes have been applied

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

* Re: [PR PATCH] [Updated] New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (14 preceding siblings ...)
  2024-02-20 15:32 ` shmishtopher
@ 2024-02-21  2:54 ` shmishtopher
  2024-04-12 12:49 ` justinesmithies
                   ` (3 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: shmishtopher @ 2024-02-21  2:54 UTC (permalink / raw)
  To: ml

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

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

https://github.com/shmishtopher/void-packages package/eww
https://github.com/void-linux/void-packages/pull/48844

New package: eww-0.5.0
#### Testing the changes
- I tested the changes in this PR: **briefly**

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

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
- I built this PR locally for these architectures:
  - aarch64-musl (crossbuild)
  - armv7l (crossbuild)
  - armv6l-musl (crossbuild)

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

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

From 7a05828f59d8680b5b3c6f392bd24d67ec80e871 Mon Sep 17 00:00:00 2001
From: "Christopher K. \"Shmish\" Schmitt" <me@shmish.dev>
Date: Mon, 19 Feb 2024 20:29:43 -0500
Subject: [PATCH] New package: eww-0.5.0

---
 srcpkgs/eww/patches/fix-completions.patch | 129 ++++++++++++++++++++++
 srcpkgs/eww/template                      |  25 +++++
 2 files changed, 154 insertions(+)
 create mode 100644 srcpkgs/eww/patches/fix-completions.patch
 create mode 100644 srcpkgs/eww/template

diff --git a/srcpkgs/eww/patches/fix-completions.patch b/srcpkgs/eww/patches/fix-completions.patch
new file mode 100644
index 00000000000000..224f676c5addc6
--- /dev/null
+++ b/srcpkgs/eww/patches/fix-completions.patch
@@ -0,0 +1,129 @@
+From 525fb069b7eb3213dcbbd1e12bb476b546f63b82 Mon Sep 17 00:00:00 2001
+From: Zacharie Dubrulle <dubrullezacharie@gmail.com>
+Date: Mon, 19 Feb 2024 22:02:57 +0100
+Subject: [PATCH 1/2] Handle `shell-completions` before anything else
+
+---
+ crates/eww/src/client.rs |  4 ----
+ crates/eww/src/main.rs   |  8 ++++++++
+ crates/eww/src/opts.rs   | 14 +++++++-------
+ 3 files changed, 15 insertions(+), 11 deletions(-)
+
+diff --git a/crates/eww/src/client.rs b/crates/eww/src/client.rs
+index c9f14ab2..616c3872 100644
+--- a/crates/eww/src/client.rs
++++ b/crates/eww/src/client.rs
+@@ -6,7 +6,6 @@ use crate::{
+     paths::EwwPaths,
+ };
+ use anyhow::{Context, Result};
+-use clap::CommandFactory as _;
+ use std::{
+     io::{Read, Write},
+     os::unix::net::UnixStream,
+@@ -21,9 +20,6 @@ pub fn handle_client_only_action(paths: &EwwPaths, action: ActionClientOnly) ->
+                 .spawn()?
+                 .wait()?;
+         }
+-        ActionClientOnly::ShellCompletions { shell } => {
+-            clap_complete::generate(shell, &mut opts::RawOpt::command(), "eww", &mut std::io::stdout());
+-        }
+     }
+     Ok(())
+ }
+diff --git a/crates/eww/src/main.rs b/crates/eww/src/main.rs
+index aac52a6f..848c055b 100644
+--- a/crates/eww/src/main.rs
++++ b/crates/eww/src/main.rs
+@@ -5,6 +5,7 @@ extern crate gtk;
+ extern crate gtk_layer_shell as gtk_layer_shell;
+ 
+ use anyhow::{Context, Result};
++use clap::CommandFactory as _;
+ use daemon_response::{DaemonResponse, DaemonResponseReceiver};
+ use display_backend::DisplayBackend;
+ use opts::ActionWithServer;
+@@ -44,6 +45,11 @@ fn main() {
+         pretty_env_logger::formatted_timed_builder().filter(Some("eww"), log_level_filter).init();
+     }
+ 
++    if let opts::Action::ShellCompletions { shell } = opts.action {
++        clap_complete::generate(shell, &mut opts::RawOpt::command(), "eww", &mut std::io::stdout());
++        return;
++    }
++
+     #[allow(unused)]
+     let use_wayland = opts.force_wayland || detect_wayland();
+     #[cfg(all(feature = "wayland", feature = "x11"))]
+@@ -87,6 +93,7 @@ fn run<B: DisplayBackend>(opts: opts::Opt, eww_binary_name: String, display_back
+         .context("Failed to initialize eww paths")?;
+ 
+     let should_restart = match &opts.action {
++        opts::Action::ShellCompletions { .. } => unreachable!(),
+         opts::Action::Daemon => opts.restart,
+         opts::Action::WithServer(action) => opts.restart && action.can_start_daemon(),
+         opts::Action::ClientOnly(_) => false,
+@@ -100,6 +107,7 @@ fn run<B: DisplayBackend>(opts: opts::Opt, eww_binary_name: String, display_back
+     }
+ 
+     let would_show_logs = match opts.action {
++        opts::Action::ShellCompletions { .. } => unreachable!(),
+         opts::Action::ClientOnly(action) => {
+             client::handle_client_only_action(&paths, action)?;
+             false
+diff --git a/crates/eww/src/opts.rs b/crates/eww/src/opts.rs
+index 2446d8c4..b635a28b 100644
+--- a/crates/eww/src/opts.rs
++++ b/crates/eww/src/opts.rs
+@@ -59,6 +59,13 @@ pub(super) struct RawOpt {
+ 
+ #[derive(Subcommand, Debug, Serialize, Deserialize, PartialEq)]
+ pub enum Action {
++    /// Generate a shell completion script
++    ShellCompletions {
++        #[arg(short, long)]
++        #[serde(with = "serde_shell")]
++        shell: clap_complete::shells::Shell,
++    },
++
+     /// Start the Eww daemon.
+     #[command(name = "daemon", alias = "d")]
+     Daemon,
+@@ -75,13 +82,6 @@ pub enum ActionClientOnly {
+     /// Print and watch the eww logs
+     #[command(name = "logs")]
+     Logs,
+-
+-    /// Generate a shell completion script
+-    ShellCompletions {
+-        #[arg(short, long)]
+-        #[serde(with = "serde_shell")]
+-        shell: clap_complete::shells::Shell,
+-    },
+ }
+ 
+ #[derive(Subcommand, Debug, Serialize, Deserialize, PartialEq)]
+
+From 0b22ea7ef053fa8afebf5f6e107f689f5516ac38 Mon Sep 17 00:00:00 2001
+From: Zacharie Dubrulle <dubrullezacharie@gmail.com>
+Date: Mon, 19 Feb 2024 22:05:36 +0100
+Subject: [PATCH 2/2] Update CHANGELOG.md
+
+---
+ CHANGELOG.md | 3 +++
+ 1 file changed, 3 insertions(+)
+
+diff --git a/CHANGELOG.md b/CHANGELOG.md
+index 09914410..f34c078a 100644
+--- a/CHANGELOG.md
++++ b/CHANGELOG.md
+@@ -4,6 +4,9 @@ All notable changes to eww will be listed here, starting at changes since versio
+ 
+ ## Unreleased
+ 
++### Fixes
++- The `shell-completions` subcommand is now run before anything is set up
++
+ ## [0.5.0] (17.02.2024)
+ 
+ ### BREAKING CHANGES
diff --git a/srcpkgs/eww/template b/srcpkgs/eww/template
new file mode 100644
index 00000000000000..1eb64d3a256796
--- /dev/null
+++ b/srcpkgs/eww/template
@@ -0,0 +1,25 @@
+# Template file for 'eww'
+pkgname=eww
+version=0.5.0
+revision=1
+build_style=cargo
+build_helper="qemu"
+make_install_args="--path=crates/eww"
+hostmakedepends="pkg-config"
+makedepends="gtk+3-devel gtk-layer-shell-devel pango-devel gdk-pixbuf-devel
+ cairo-devel glib-devel"
+short_desc="Toolkit for creating custom bars and widgets"
+maintainer="Christopher K. 'Shmish' Schmitt <me@shmish.dev>"
+license="MIT"
+homepage="https://elkowar.github.io/eww"
+changelog="https://raw.githubusercontent.com/elkowar/eww/master/CHANGELOG.md"
+distfiles="https://github.com/elkowar/eww/archive/refs/tags/v${version}.tar.gz"
+checksum=ea4f62e48e3750a361e0f359933d7d840d158592ff5b3683ba1f3ccf42bda819
+
+post_install() {
+	vlicense LICENSE
+	for sh in bash fish zsh; do
+		vtargetrun "${DESTDIR}/usr/bin/eww" shell-completions -s "$sh" > "eww.$sh"
+		vcompletion "eww.$sh" "$sh"
+	done
+}

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

* Re: New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (15 preceding siblings ...)
  2024-02-21  2:54 ` [PR PATCH] [Updated] " shmishtopher
@ 2024-04-12 12:49 ` justinesmithies
  2024-04-23 15:01 ` [PR PATCH] [Updated] " shmishtopher
                   ` (2 subsequent siblings)
  19 siblings, 0 replies; 21+ messages in thread
From: justinesmithies @ 2024-04-12 12:49 UTC (permalink / raw)
  To: ml

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

New comment by justinesmithies on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#issuecomment-2051701184

Comment:
Looking forward to having this available in the void repo. Nice work thanks.

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

* Re: [PR PATCH] [Updated] New package: eww-0.5.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (16 preceding siblings ...)
  2024-04-12 12:49 ` justinesmithies
@ 2024-04-23 15:01 ` shmishtopher
  2024-04-23 15:04 ` New package: eww-0.6.0 shmishtopher
  2024-04-28  9:01 ` AnInternetTroll
  19 siblings, 0 replies; 21+ messages in thread
From: shmishtopher @ 2024-04-23 15:01 UTC (permalink / raw)
  To: ml

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

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

https://github.com/shmishtopher/void-packages package/eww
https://github.com/void-linux/void-packages/pull/48844

New package: eww-0.5.0
#### Testing the changes
- I tested the changes in this PR: **briefly**

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

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
- I built this PR locally for these architectures:
  - aarch64-musl (crossbuild)
  - armv7l (crossbuild)
  - armv6l-musl (crossbuild)

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

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

From 86c6f82793087536066bb32fe3b1c6ad1940d1d2 Mon Sep 17 00:00:00 2001
From: "Christopher K. \"Shmish\" Schmitt" <me@shmish.dev>
Date: Mon, 19 Feb 2024 20:29:43 -0500
Subject: [PATCH] New package: eww-0.6.0

---
 srcpkgs/eww/template | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)
 create mode 100644 srcpkgs/eww/template

diff --git a/srcpkgs/eww/template b/srcpkgs/eww/template
new file mode 100644
index 00000000000000..6359f4d514c27b
--- /dev/null
+++ b/srcpkgs/eww/template
@@ -0,0 +1,25 @@
+# Template file for 'eww'
+pkgname=eww
+version=0.6.0
+revision=1
+build_style=cargo
+build_helper="qemu"
+make_install_args="--path=crates/eww"
+hostmakedepends="pkg-config"
+makedepends="gtk+3-devel gtk-layer-shell-devel pango-devel gdk-pixbuf-devel
+ cairo-devel glib-devel libdbusmenu-gtk3-devel"
+short_desc="Toolkit for creating custom bars and widgets"
+maintainer="Christopher K. 'Shmish' Schmitt <me@shmish.dev>"
+license="MIT"
+homepage="https://elkowar.github.io/eww"
+changelog="https://raw.githubusercontent.com/elkowar/eww/master/CHANGELOG.md"
+distfiles="https://github.com/elkowar/eww/archive/refs/tags/v${version}.tar.gz"
+checksum=cef361946946c566b79f8ddc6208d1a3f16b4ff9961439a3f86935e1cfa174a1
+
+post_install() {
+	vlicense LICENSE
+	for sh in bash fish zsh; do
+		vtargetrun "${DESTDIR}/usr/bin/eww" shell-completions -s "$sh" > "eww.$sh"
+		vcompletion "eww.$sh" "$sh"
+	done
+}

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

* Re: New package: eww-0.6.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (17 preceding siblings ...)
  2024-04-23 15:01 ` [PR PATCH] [Updated] " shmishtopher
@ 2024-04-23 15:04 ` shmishtopher
  2024-04-28  9:01 ` AnInternetTroll
  19 siblings, 0 replies; 21+ messages in thread
From: shmishtopher @ 2024-04-23 15:04 UTC (permalink / raw)
  To: ml

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

New comment by shmishtopher on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#issuecomment-2072614196

Comment:
A new release has been made, I've updated the commit message and title to match.  `0.6.0` no longer requires a patch to generate shell completions and introduces a new `libdbusmenu-gtk3-devel` dependency.

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

* Re: New package: eww-0.6.0
  2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
                   ` (18 preceding siblings ...)
  2024-04-23 15:04 ` New package: eww-0.6.0 shmishtopher
@ 2024-04-28  9:01 ` AnInternetTroll
  19 siblings, 0 replies; 21+ messages in thread
From: AnInternetTroll @ 2024-04-28  9:01 UTC (permalink / raw)
  To: ml

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

New comment by AnInternetTroll on void-packages repository

https://github.com/void-linux/void-packages/pull/48844#issuecomment-2081399838

Comment:
Tested on sway with the default configuration and works as expected.
Fancy looking bar

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

end of thread, other threads:[~2024-04-28  9:01 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-20  2:03 [PR PATCH] New package: eww shmishtopher
2024-02-20  2:04 ` shmishtopher
2024-02-20  4:50 ` classabbyamp
2024-02-20  4:54 ` [PR PATCH] [Updated] " shmishtopher
2024-02-20  4:55 ` New package: eww-0.5.0 shmishtopher
2024-02-20  5:51 ` [PR REVIEW] " classabbyamp
2024-02-20  5:51 ` classabbyamp
2024-02-20  5:51 ` classabbyamp
2024-02-20  5:51 ` classabbyamp
2024-02-20  5:51 ` classabbyamp
2024-02-20  5:51 ` classabbyamp
2024-02-20  5:51 ` classabbyamp
2024-02-20 14:42 ` [PR PATCH] [Updated] " shmishtopher
2024-02-20 15:30 ` shmishtopher
2024-02-20 15:31 ` [PR REVIEW] " shmishtopher
2024-02-20 15:32 ` shmishtopher
2024-02-21  2:54 ` [PR PATCH] [Updated] " shmishtopher
2024-04-12 12:49 ` justinesmithies
2024-04-23 15:01 ` [PR PATCH] [Updated] " shmishtopher
2024-04-23 15:04 ` New package: eww-0.6.0 shmishtopher
2024-04-28  9:01 ` AnInternetTroll

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).