[PR PATCH] New package: sxwm-1.5

[PR PATCH] New package: sxwm-1.5

[Rooyca]

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

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

https://github.com/Rooyca/void-packages sxwm
https://github.com/void-linux/void-packages/pull/55351

New package: sxwm-1.5
<!-- 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**

<!-- 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-glibc)



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

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

From ed5ea3d8bd05bd2585c08b240e10af7f98c30138 Mon Sep 17 00:00:00 2001
From: rooyca <rooyca@gmail.com>
Date: Fri, 16 May 2025 11:30:47 -0500
Subject: [PATCH] New package: sxwm-1.5

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

diff --git a/srcpkgs/sxwm/template b/srcpkgs/sxwm/template
new file mode 100644
index 00000000000000..6bb4eeb910027b
--- /dev/null
+++ b/srcpkgs/sxwm/template
@@ -0,0 +1,25 @@
+# Template file for 'sxwm'
+pkgname=sxwm
+version=1.5
+revision=1
+build_style=gnu-makefile
+makedepends="libX11-devel libXinerama-devel libXcursor-devel"
+depends="libX11 libXinerama libXcursor"
+short_desc="Tiling window managers dont have to be difficult"
+maintainer="rooyca <rooyca@gmail.com>"
+license="MIT"
+homepage="https://github.com/uint23/sxwm"
+distfiles="https://github.com/uint23/sxwm/archive/refs/tags/v${version}.tar.gz"
+checksum=5443ae3955c50527c22253f20e1f6d5c96e6cc842ff7e3370d0f6920167148b7
+
+LDFLAGS="-lX11 -lXinerama -lXcursor"
+
+do_install() {
+	make PREFIX=/usr DESTDIR=$DESTDIR install
+	vinstall default_sxrc 644 $HOME/.config/sxwmrc
+	vlicense LICENSE
+}
+
+post_install() {
+    rm -rf "${DESTDIR}/tmp"
+}

Re: [PR PATCH] [Updated] New package: sxwm-1.5

[Rooyca]

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

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

https://github.com/Rooyca/void-packages sxwm
https://github.com/void-linux/void-packages/pull/55351

New package: sxwm-1.5
<!-- 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**

<!-- 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-glibc)



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

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

From ed5ea3d8bd05bd2585c08b240e10af7f98c30138 Mon Sep 17 00:00:00 2001
From: rooyca <rooyca@gmail.com>
Date: Fri, 16 May 2025 11:30:47 -0500
Subject: [PATCH 1/2] New package: sxwm-1.5

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

diff --git a/srcpkgs/sxwm/template b/srcpkgs/sxwm/template
new file mode 100644
index 00000000000000..6bb4eeb910027b
--- /dev/null
+++ b/srcpkgs/sxwm/template
@@ -0,0 +1,25 @@
+# Template file for 'sxwm'
+pkgname=sxwm
+version=1.5
+revision=1
+build_style=gnu-makefile
+makedepends="libX11-devel libXinerama-devel libXcursor-devel"
+depends="libX11 libXinerama libXcursor"
+short_desc="Tiling window managers dont have to be difficult"
+maintainer="rooyca <rooyca@gmail.com>"
+license="MIT"
+homepage="https://github.com/uint23/sxwm"
+distfiles="https://github.com/uint23/sxwm/archive/refs/tags/v${version}.tar.gz"
+checksum=5443ae3955c50527c22253f20e1f6d5c96e6cc842ff7e3370d0f6920167148b7
+
+LDFLAGS="-lX11 -lXinerama -lXcursor"
+
+do_install() {
+	make PREFIX=/usr DESTDIR=$DESTDIR install
+	vinstall default_sxrc 644 $HOME/.config/sxwmrc
+	vlicense LICENSE
+}
+
+post_install() {
+    rm -rf "${DESTDIR}/tmp"
+}

From 54deaafb56aa61e0fade6645f3374752790f76f2 Mon Sep 17 00:00:00 2001
From: rooyca <rooyca@gmail.com>
Date: Fri, 16 May 2025 11:39:51 -0500
Subject: [PATCH 2/2] New package: sxwm-1.5

---
 srcpkgs/sxwm/template | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srcpkgs/sxwm/template b/srcpkgs/sxwm/template
index 6bb4eeb910027b..8291d84b50bacc 100644
--- a/srcpkgs/sxwm/template
+++ b/srcpkgs/sxwm/template
@@ -21,5 +21,5 @@ do_install() {
 }
 
 post_install() {
-    rm -rf "${DESTDIR}/tmp"
+	rm -rf "${DESTDIR}/tmp"
 }

Re: [PR REVIEW] New package: sxwm-1.5

[Bnyro]

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

New review comment by Bnyro on void-packages repository

https://github.com/void-linux/void-packages/pull/55351#discussion_r2106266545

Comment:
That's not really an appropriate description imo, it's rather a catch phrase / advertisement.

Re: [PR REVIEW] New package: sxwm-1.5

[Rooyca]

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

New review comment by Rooyca on void-packages repository

https://github.com/void-linux/void-packages/pull/55351#discussion_r2107815337

Comment:
True. 
I took it from the `About` section on the Github repo, should I write my own description? 

Re: [PR REVIEW] New package: sxwm-1.5

[Bnyro]

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

New review comment by Bnyro on void-packages repository

https://github.com/void-linux/void-packages/pull/55351#discussion_r2107840366

Comment:
Yeah.
Their README says
> Minimal. Fast. Configurable. Tiling Window Manager for X11
>
so rephrasing this probably gives us a decent enough description.

Re: [PR REVIEW] New package: sxwm-1.5

[Rooyca]

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

New review comment by Rooyca on void-packages repository

https://github.com/void-linux/void-packages/pull/55351#discussion_r2107851208

Comment:
I didn't see that :sweat_smile: Thanks!

Something like:
`A minimal, fast, and highly configurable tiling window manager for X11.` 

Re: [PR REVIEW] New package: sxwm-1.5

[Bnyro]

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

New review comment by Bnyro on void-packages repository

https://github.com/void-linux/void-packages/pull/55351#discussion_r2108346343

Comment:
Sounds good 👍 

Re: [PR PATCH] [Updated] New package: sxwm-1.5

[Rooyca]

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

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

https://github.com/Rooyca/void-packages sxwm
https://github.com/void-linux/void-packages/pull/55351

New package: sxwm-1.5
<!-- 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**

<!-- 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-glibc)



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

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

From ed5ea3d8bd05bd2585c08b240e10af7f98c30138 Mon Sep 17 00:00:00 2001
From: rooyca <rooyca@gmail.com>
Date: Fri, 16 May 2025 11:30:47 -0500
Subject: [PATCH 1/2] New package: sxwm-1.5

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

diff --git a/srcpkgs/sxwm/template b/srcpkgs/sxwm/template
new file mode 100644
index 00000000000000..6bb4eeb910027b
--- /dev/null
+++ b/srcpkgs/sxwm/template
@@ -0,0 +1,25 @@
+# Template file for 'sxwm'
+pkgname=sxwm
+version=1.5
+revision=1
+build_style=gnu-makefile
+makedepends="libX11-devel libXinerama-devel libXcursor-devel"
+depends="libX11 libXinerama libXcursor"
+short_desc="Tiling window managers dont have to be difficult"
+maintainer="rooyca <rooyca@gmail.com>"
+license="MIT"
+homepage="https://github.com/uint23/sxwm"
+distfiles="https://github.com/uint23/sxwm/archive/refs/tags/v${version}.tar.gz"
+checksum=5443ae3955c50527c22253f20e1f6d5c96e6cc842ff7e3370d0f6920167148b7
+
+LDFLAGS="-lX11 -lXinerama -lXcursor"
+
+do_install() {
+	make PREFIX=/usr DESTDIR=$DESTDIR install
+	vinstall default_sxrc 644 $HOME/.config/sxwmrc
+	vlicense LICENSE
+}
+
+post_install() {
+    rm -rf "${DESTDIR}/tmp"
+}

From f3191f3f3a807ea3379427986f81c06b190070e7 Mon Sep 17 00:00:00 2001
From: rooyca <rooyca@gmail.com>
Date: Fri, 16 May 2025 11:39:51 -0500
Subject: [PATCH 2/2] New package: sxwm-1.5

---
 srcpkgs/sxwm/template | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/sxwm/template b/srcpkgs/sxwm/template
index 6bb4eeb910027b..4569753493f28a 100644
--- a/srcpkgs/sxwm/template
+++ b/srcpkgs/sxwm/template
@@ -5,7 +5,7 @@ revision=1
 build_style=gnu-makefile
 makedepends="libX11-devel libXinerama-devel libXcursor-devel"
 depends="libX11 libXinerama libXcursor"
-short_desc="Tiling window managers dont have to be difficult"
+short_desc="Minimal, fast, and highly configurable tiling window manager for X11"
 maintainer="rooyca <rooyca@gmail.com>"
 license="MIT"
 homepage="https://github.com/uint23/sxwm"
@@ -21,5 +21,5 @@ do_install() {
 }
 
 post_install() {
-    rm -rf "${DESTDIR}/tmp"
+	rm -rf "${DESTDIR}/tmp"
 }

Re: [PR REVIEW] New package: sxwm-1.5

[Duncaen]

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

New review comment by Duncaen on void-packages repository

https://github.com/void-linux/void-packages/pull/55351#discussion_r2119640952

Comment:
move the manually installed things to `post_install` and let the `build_style` call `make install`.

Re: [PR PATCH] [Updated] New package: sxwm-1.5

[Rooyca]

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

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

https://github.com/Rooyca/void-packages sxwm
https://github.com/void-linux/void-packages/pull/55351

New package: sxwm-1.5
<!-- 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**

<!-- 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-glibc)



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

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

From ed5ea3d8bd05bd2585c08b240e10af7f98c30138 Mon Sep 17 00:00:00 2001
From: rooyca <rooyca@gmail.com>
Date: Fri, 16 May 2025 11:30:47 -0500
Subject: [PATCH 1/2] New package: sxwm-1.5

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

diff --git a/srcpkgs/sxwm/template b/srcpkgs/sxwm/template
new file mode 100644
index 00000000000000..6bb4eeb910027b
--- /dev/null
+++ b/srcpkgs/sxwm/template
@@ -0,0 +1,25 @@
+# Template file for 'sxwm'
+pkgname=sxwm
+version=1.5
+revision=1
+build_style=gnu-makefile
+makedepends="libX11-devel libXinerama-devel libXcursor-devel"
+depends="libX11 libXinerama libXcursor"
+short_desc="Tiling window managers dont have to be difficult"
+maintainer="rooyca <rooyca@gmail.com>"
+license="MIT"
+homepage="https://github.com/uint23/sxwm"
+distfiles="https://github.com/uint23/sxwm/archive/refs/tags/v${version}.tar.gz"
+checksum=5443ae3955c50527c22253f20e1f6d5c96e6cc842ff7e3370d0f6920167148b7
+
+LDFLAGS="-lX11 -lXinerama -lXcursor"
+
+do_install() {
+	make PREFIX=/usr DESTDIR=$DESTDIR install
+	vinstall default_sxrc 644 $HOME/.config/sxwmrc
+	vlicense LICENSE
+}
+
+post_install() {
+    rm -rf "${DESTDIR}/tmp"
+}

From 25b90060e3510846ee64351094d841a2a557b60c Mon Sep 17 00:00:00 2001
From: rooyca <rooyca@gmail.com>
Date: Fri, 16 May 2025 11:39:51 -0500
Subject: [PATCH 2/2] New package: sxwm-1.5

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

diff --git a/srcpkgs/sxwm/template b/srcpkgs/sxwm/template
index 6bb4eeb910027b..01030fd89c748a 100644
--- a/srcpkgs/sxwm/template
+++ b/srcpkgs/sxwm/template
@@ -5,7 +5,7 @@ revision=1
 build_style=gnu-makefile
 makedepends="libX11-devel libXinerama-devel libXcursor-devel"
 depends="libX11 libXinerama libXcursor"
-short_desc="Tiling window managers dont have to be difficult"
+short_desc="Minimal, fast, and highly configurable tiling window manager for X11"
 maintainer="rooyca <rooyca@gmail.com>"
 license="MIT"
 homepage="https://github.com/uint23/sxwm"
@@ -14,12 +14,8 @@ checksum=5443ae3955c50527c22253f20e1f6d5c96e6cc842ff7e3370d0f6920167148b7
 
 LDFLAGS="-lX11 -lXinerama -lXcursor"
 
-do_install() {
-	make PREFIX=/usr DESTDIR=$DESTDIR install
+post_install() {
 	vinstall default_sxrc 644 $HOME/.config/sxwmrc
 	vlicense LICENSE
-}
-
-post_install() {
-    rm -rf "${DESTDIR}/tmp"
+	rm -rf "${DESTDIR}/tmp"
 }

Re: [PR REVIEW] New package: sxwm-1.5

[Rooyca]

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

New review comment by Rooyca on void-packages repository

https://github.com/void-linux/void-packages/pull/55351#discussion_r2130286597

Comment:
Ok, Thanks. :smiley: 

Re: [PR PATCH] [Merged]: New package: sxwm-1.5

[Duncaen]

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

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

New package: sxwm-1.5
https://github.com/void-linux/void-packages/pull/55351

Description:
<!-- 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**

<!-- 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-glibc)