Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] New package: swaylock-effects-1.7.0.0
@ 2024-01-05 13:36 Clos3y
  2024-01-05 13:40 ` ahesford
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Clos3y @ 2024-01-05 13:36 UTC (permalink / raw)
  To: ml

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

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

https://github.com/Clos3y/void-packages swaylock-effects-1.7.0.0
https://github.com/void-linux/void-packages/pull/48075

New package: swaylock-effects-1.7.0.0
<!-- 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)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - i686 (X)
  - i686-musl (X)
  - x86_64-musl (X)
  - armv6l (X)
  - armv6l-musl (X)

Opening as draft to discuss possible security concerns previously mentioned by @ericonr (https://github.com/void-linux/void-packages/pull/26392#issuecomment-883685280, https://github.com/void-linux/void-packages/pull/27971#pullrequestreview-570102279). [This version is a fork](https://github.com/jirutka/swaylock-effects) of the now [unmaintained original](https://github.com/mortie/swaylock-effects). The maintainer is active, and the current version of `jirutka/swaylock-effects` is behind `swaywm/swaylock` by [23 commits](https://github.com/jirutka/swaylock-effects/compare/master...swaywm:swaylock:master). Happy to discuss :)

Also, regarding the template, I'm not sure if it's preferred to have `conflicts` or `replaces`? I've gone for the former because the latter would require changing the `swaylock` template too.

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-swaylock-effects-1.7.0.0-48075.patch --]
[-- Type: text/x-diff, Size: 1247 bytes --]

From 7207677eb58dd9cae1a0c86998adc5920b967bb8 Mon Sep 17 00:00:00 2001
From: closey <sam.w.close@gmail.com>
Date: Fri, 5 Jan 2024 00:15:35 +0000
Subject: [PATCH] New package: swaylock-effects-1.7.0.0

---
 srcpkgs/swaylock-effects/template | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 srcpkgs/swaylock-effects/template

diff --git a/srcpkgs/swaylock-effects/template b/srcpkgs/swaylock-effects/template
new file mode 100644
index 0000000000000..47c740e96da1a
--- /dev/null
+++ b/srcpkgs/swaylock-effects/template
@@ -0,0 +1,19 @@
+# Template file for 'swaylock-effects'
+pkgname=swaylock-effects
+version=1.7.0.0
+revision=1
+build_style=meson
+conf_files="/etc/pam.d/swaylock"
+hostmakedepends="pkg-config wayland-devel scdoc"
+makedepends="wayland-devel wayland-protocols libxkbcommon-devel gdk-pixbuf-devel cairo-devel libgomp-devel pam-devel"
+short_desc="Swaylock, with fancy effects"
+maintainer="Sam Close <sam.w.close@gmail.com>"
+license="MIT"
+homepage="https://github.com/jirutka/swaylock-effects"
+distfiles="${homepage}/archive/refs/tags/v${version}.tar.gz"
+checksum=e94d79e189602694bedfbafb553ce3c6c976426e16f76d93bf7e226dc2876eb6
+conflicts="swaylock>=0"
+
+post_install() {
+	vlicense LICENSE
+}

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

* Re: New package: swaylock-effects-1.7.0.0
  2024-01-05 13:36 [PR PATCH] New package: swaylock-effects-1.7.0.0 Clos3y
@ 2024-01-05 13:40 ` ahesford
  2024-01-05 13:40 ` ahesford
  2024-01-05 13:48 ` Clos3y
  2 siblings, 0 replies; 4+ messages in thread
From: ahesford @ 2024-01-05 13:40 UTC (permalink / raw)
  To: ml

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

New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/48075#issuecomment-1878676170

Comment:
This is a fork of a fork of swaylock. What benefits would this confer, and why chose this particular fork?

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

* Re: New package: swaylock-effects-1.7.0.0
  2024-01-05 13:36 [PR PATCH] New package: swaylock-effects-1.7.0.0 Clos3y
  2024-01-05 13:40 ` ahesford
@ 2024-01-05 13:40 ` ahesford
  2024-01-05 13:48 ` Clos3y
  2 siblings, 0 replies; 4+ messages in thread
From: ahesford @ 2024-01-05 13:40 UTC (permalink / raw)
  To: ml

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

New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/48075#issuecomment-1878676170

Comment:
This is a fork of a fork of swaylock. What benefits would this confer, and why chooose this particular fork?

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

* Re: New package: swaylock-effects-1.7.0.0
  2024-01-05 13:36 [PR PATCH] New package: swaylock-effects-1.7.0.0 Clos3y
  2024-01-05 13:40 ` ahesford
  2024-01-05 13:40 ` ahesford
@ 2024-01-05 13:48 ` Clos3y
  2 siblings, 0 replies; 4+ messages in thread
From: Clos3y @ 2024-01-05 13:48 UTC (permalink / raw)
  To: ml

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

New comment by Clos3y on void-packages repository

https://github.com/void-linux/void-packages/pull/48075#issuecomment-1878686269

Comment:
> This is a fork of a fork of swaylock. What benefits would this confer, and why chooose this particular fork?

The changes are aesthetic with subjectively 'nicer' screenlocking, so I think I'd be hard pressed to say truthfully there are any benefits, per-se. 

As for why this fork, unlike the repo it is forked from (from which people have made the previous PRs), it's being maintained and developed (albeit slowly).

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

end of thread, other threads:[~2024-01-05 13:48 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-05 13:36 [PR PATCH] New package: swaylock-effects-1.7.0.0 Clos3y
2024-01-05 13:40 ` ahesford
2024-01-05 13:40 ` ahesford
2024-01-05 13:48 ` Clos3y

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