Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] urlwatch: update to 2.26
@ 2023-04-11 21:10 RunningDroid
  2023-04-11 23:04 ` [PR REVIEW] " mhmdanas
                   ` (9 more replies)
  0 siblings, 10 replies; 11+ messages in thread
From: RunningDroid @ 2023-04-11 21:10 UTC (permalink / raw)
  To: ml

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

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

https://github.com/RunningDroid/void-packages update_urlwatch
https://github.com/void-linux/void-packages/pull/43398

urlwatch: update to 2.26
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### 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**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


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

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

From 02045001ae38a05e6cda347f0bac583c89e400f0 Mon Sep 17 00:00:00 2001
From: RunningDroid <runningdroid@zoho.com>
Date: Tue, 11 Apr 2023 17:09:02 -0400
Subject: [PATCH] urlwatch: update to 2.26

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

diff --git a/srcpkgs/urlwatch/template b/srcpkgs/urlwatch/template
index bdc8421176c2..f0a0ed4877a1 100644
--- a/srcpkgs/urlwatch/template
+++ b/srcpkgs/urlwatch/template
@@ -1,7 +1,7 @@
 # Template file for 'urlwatch'
 pkgname=urlwatch
-version=2.25
-revision=2
+version=2.26
+revision=1
 build_style=python3-module
 hostmakedepends="python3-setuptools"
 depends="python3-appdirs python3-keyring python3-minidb python3-requests
@@ -14,8 +14,9 @@ short_desc="Tool for monitoring webpages for updates"
 maintainer="RunningDroid <runningdroid@zoho.com>"
 license="BSD-3-Clause"
 homepage="https://thp.io/2008/urlwatch/"
+changelog="https://raw.githubusercontent.com/thp/urlwatch/master/CHANGELOG.md"
 distfiles="${PYPI_SITE}/u/urlwatch/urlwatch-${version}.tar.gz"
-checksum=6802297d3318286e7f3d36b9a4567a2fb09b0ae779d4b76811dd29a7281c1f8a
+checksum=b14bbe23d87b7d4ac8b4e71a51fd2066908a947ccf7b557c3e4f87b09f455e8f
 
 do_check() {
 	# skip the tests that require python modules that aren't packaged

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

* Re: [PR REVIEW] urlwatch: update to 2.26
  2023-04-11 21:10 [PR PATCH] urlwatch: update to 2.26 RunningDroid
@ 2023-04-11 23:04 ` mhmdanas
  2023-04-12  0:38 ` [PR PATCH] [Updated] " RunningDroid
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: mhmdanas @ 2023-04-11 23:04 UTC (permalink / raw)
  To: ml

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

New review comment by mhmdanas on void-packages repository

https://github.com/void-linux/void-packages/pull/43398#discussion_r1163410591

Comment:
Remove `do_check` below and do this:
```suggestion
# skip the tests that require python modules that aren't packaged
# (pdftotext & pytesseract)
make_check_args="-k not((pdf)or(ocr))"

```

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

* Re: [PR PATCH] [Updated] urlwatch: update to 2.26
  2023-04-11 21:10 [PR PATCH] urlwatch: update to 2.26 RunningDroid
  2023-04-11 23:04 ` [PR REVIEW] " mhmdanas
@ 2023-04-12  0:38 ` RunningDroid
  2023-04-12  0:41 ` RunningDroid
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: RunningDroid @ 2023-04-12  0:38 UTC (permalink / raw)
  To: ml

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

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

https://github.com/RunningDroid/void-packages update_urlwatch
https://github.com/void-linux/void-packages/pull/43398

urlwatch: update to 2.26
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### 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**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


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

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

From bf55f7b419cce88de0b71af1af97f1a8476f1e0a Mon Sep 17 00:00:00 2001
From: RunningDroid <runningdroid@zoho.com>
Date: Tue, 11 Apr 2023 17:09:02 -0400
Subject: [PATCH] urlwatch: update to 2.26

---
 srcpkgs/urlwatch/patches/test_handler.patch | 14 --------------
 srcpkgs/urlwatch/template                   | 18 ++++++++----------
 2 files changed, 8 insertions(+), 24 deletions(-)
 delete mode 100644 srcpkgs/urlwatch/patches/test_handler.patch

diff --git a/srcpkgs/urlwatch/patches/test_handler.patch b/srcpkgs/urlwatch/patches/test_handler.patch
deleted file mode 100644
index 73f301e49f9f..000000000000
--- a/srcpkgs/urlwatch/patches/test_handler.patch
+++ /dev/null
@@ -1,14 +0,0 @@
-Work around https://github.com/thp/urlwatch/issues/677
-
-diff --git a/lib/urlwatch/tests/test_handler.py b/lib/urlwatch/tests/test_handler.py
-index 736536a..fac9449 100644
---- a/lib/urlwatch/tests/test_handler.py
-+++ b/lib/urlwatch/tests/test_handler.py
-@@ -89,6 +89,7 @@ def test_pep8_conformance():
- 
- class ConfigForTest(CommandConfig):
-     def __init__(self, config, urls, cache, hooks, verbose):
-+        sys.argv = sys.argv[:1]
-         (prefix, bindir) = os.path.split(os.path.dirname(os.path.abspath(sys.argv[0])))
-         super().__init__('urlwatch', os.path.dirname(__file__), bindir, prefix, config, urls, hooks, cache, verbose)
- 
diff --git a/srcpkgs/urlwatch/template b/srcpkgs/urlwatch/template
index bdc8421176c2..88441dcd1674 100644
--- a/srcpkgs/urlwatch/template
+++ b/srcpkgs/urlwatch/template
@@ -1,7 +1,7 @@
 # Template file for 'urlwatch'
 pkgname=urlwatch
-version=2.25
-revision=2
+version=2.26
+revision=1
 build_style=python3-module
 hostmakedepends="python3-setuptools"
 depends="python3-appdirs python3-keyring python3-minidb python3-requests
@@ -9,19 +9,17 @@ depends="python3-appdirs python3-keyring python3-minidb python3-requests
 # Check the Docs for optional packages:
 # https://urlwatch.readthedocs.io/en/latest/dependencies.html#optional-packages
 checkdepends="python3-pytest python3-pycodestyle python3-docutils
- python3-Pygments python3-jq ${depends}"
+ python3-jq ${depends}"
 short_desc="Tool for monitoring webpages for updates"
 maintainer="RunningDroid <runningdroid@zoho.com>"
 license="BSD-3-Clause"
 homepage="https://thp.io/2008/urlwatch/"
+changelog="https://raw.githubusercontent.com/thp/urlwatch/master/CHANGELOG.md"
 distfiles="${PYPI_SITE}/u/urlwatch/urlwatch-${version}.tar.gz"
-checksum=6802297d3318286e7f3d36b9a4567a2fb09b0ae779d4b76811dd29a7281c1f8a
-
-do_check() {
-	# skip the tests that require python modules that aren't packaged
-	# (pdftotext & pytesseract)
-	pytest -k 'not (pdf or ocr)' -v
-}
+checksum=b14bbe23d87b7d4ac8b4e71a51fd2066908a947ccf7b557c3e4f87b09f455e8f
+# skip the tests that require python modules that aren't packaged
+# (pdftotext & pytesseract)
+make_check_args="-k not((pdf)or(ocr))"
 
 post_install() {
 	vlicense COPYING

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

* Re: urlwatch: update to 2.26
  2023-04-11 21:10 [PR PATCH] urlwatch: update to 2.26 RunningDroid
  2023-04-11 23:04 ` [PR REVIEW] " mhmdanas
  2023-04-12  0:38 ` [PR PATCH] [Updated] " RunningDroid
@ 2023-04-12  0:41 ` RunningDroid
  2023-04-12  0:42 ` [PR PATCH] [Updated] " RunningDroid
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: RunningDroid @ 2023-04-12  0:41 UTC (permalink / raw)
  To: ml

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

New comment by RunningDroid on void-packages repository

https://github.com/void-linux/void-packages/pull/43398#issuecomment-1504340963

Comment:
> The checks worked for me locally without `python3-Pygments` in checkdepends, so remove it from there.

Apparently 2.22 removed the dependency on Pygments and I didn't notice; I've removed it now.

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

* Re: [PR PATCH] [Updated] urlwatch: update to 2.26
  2023-04-11 21:10 [PR PATCH] urlwatch: update to 2.26 RunningDroid
                   ` (2 preceding siblings ...)
  2023-04-12  0:41 ` RunningDroid
@ 2023-04-12  0:42 ` RunningDroid
  2023-04-12  0:43 ` RunningDroid
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: RunningDroid @ 2023-04-12  0:42 UTC (permalink / raw)
  To: ml

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

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

https://github.com/RunningDroid/void-packages update_urlwatch
https://github.com/void-linux/void-packages/pull/43398

urlwatch: update to 2.26
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### 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**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


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

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

From d06ab45bdecb359be1fab4d3bc6dd378043b7c5a Mon Sep 17 00:00:00 2001
From: RunningDroid <runningdroid@zoho.com>
Date: Tue, 11 Apr 2023 17:09:02 -0400
Subject: [PATCH] urlwatch: update to 2.26

---
 srcpkgs/urlwatch/patches/test_handler.patch | 14 --------------
 srcpkgs/urlwatch/template                   | 18 ++++++++----------
 2 files changed, 8 insertions(+), 24 deletions(-)
 delete mode 100644 srcpkgs/urlwatch/patches/test_handler.patch

diff --git a/srcpkgs/urlwatch/patches/test_handler.patch b/srcpkgs/urlwatch/patches/test_handler.patch
deleted file mode 100644
index 73f301e49f9f..000000000000
--- a/srcpkgs/urlwatch/patches/test_handler.patch
+++ /dev/null
@@ -1,14 +0,0 @@
-Work around https://github.com/thp/urlwatch/issues/677
-
-diff --git a/lib/urlwatch/tests/test_handler.py b/lib/urlwatch/tests/test_handler.py
-index 736536a..fac9449 100644
---- a/lib/urlwatch/tests/test_handler.py
-+++ b/lib/urlwatch/tests/test_handler.py
-@@ -89,6 +89,7 @@ def test_pep8_conformance():
- 
- class ConfigForTest(CommandConfig):
-     def __init__(self, config, urls, cache, hooks, verbose):
-+        sys.argv = sys.argv[:1]
-         (prefix, bindir) = os.path.split(os.path.dirname(os.path.abspath(sys.argv[0])))
-         super().__init__('urlwatch', os.path.dirname(__file__), bindir, prefix, config, urls, hooks, cache, verbose)
- 
diff --git a/srcpkgs/urlwatch/template b/srcpkgs/urlwatch/template
index bdc8421176c2..58287278fe90 100644
--- a/srcpkgs/urlwatch/template
+++ b/srcpkgs/urlwatch/template
@@ -1,27 +1,25 @@
 # Template file for 'urlwatch'
 pkgname=urlwatch
-version=2.25
-revision=2
+version=2.26
+revision=1
 build_style=python3-module
+# skip the tests that require python modules that aren't packaged
+# (pdftotext & pytesseract)
+make_check_args="-k not((pdf)or(ocr))"
 hostmakedepends="python3-setuptools"
 depends="python3-appdirs python3-keyring python3-minidb python3-requests
  python3-yaml python3-lxml python3-cssselect"
 # Check the Docs for optional packages:
 # https://urlwatch.readthedocs.io/en/latest/dependencies.html#optional-packages
 checkdepends="python3-pytest python3-pycodestyle python3-docutils
- python3-Pygments python3-jq ${depends}"
+ python3-jq ${depends}"
 short_desc="Tool for monitoring webpages for updates"
 maintainer="RunningDroid <runningdroid@zoho.com>"
 license="BSD-3-Clause"
 homepage="https://thp.io/2008/urlwatch/"
+changelog="https://raw.githubusercontent.com/thp/urlwatch/master/CHANGELOG.md"
 distfiles="${PYPI_SITE}/u/urlwatch/urlwatch-${version}.tar.gz"
-checksum=6802297d3318286e7f3d36b9a4567a2fb09b0ae779d4b76811dd29a7281c1f8a
-
-do_check() {
-	# skip the tests that require python modules that aren't packaged
-	# (pdftotext & pytesseract)
-	pytest -k 'not (pdf or ocr)' -v
-}
+checksum=b14bbe23d87b7d4ac8b4e71a51fd2066908a947ccf7b557c3e4f87b09f455e8f
 
 post_install() {
 	vlicense COPYING

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

* Re: urlwatch: update to 2.26
  2023-04-11 21:10 [PR PATCH] urlwatch: update to 2.26 RunningDroid
                   ` (3 preceding siblings ...)
  2023-04-12  0:42 ` [PR PATCH] [Updated] " RunningDroid
@ 2023-04-12  0:43 ` RunningDroid
  2023-04-12  0:59 ` RunningDroid
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: RunningDroid @ 2023-04-12  0:43 UTC (permalink / raw)
  To: ml

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

New comment by RunningDroid on void-packages repository

https://github.com/void-linux/void-packages/pull/43398#issuecomment-1504340963

Comment:
> The checks worked for me locally without `python3-Pygments` in checkdepends, so remove it from there.

Apparently 2.22 removed the dependency on Pygments and I didn't notice; I've removed it now.

Edit: re-push because I forgot to `xlint` the template on my machine

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

* Re: urlwatch: update to 2.26
  2023-04-11 21:10 [PR PATCH] urlwatch: update to 2.26 RunningDroid
                   ` (4 preceding siblings ...)
  2023-04-12  0:43 ` RunningDroid
@ 2023-04-12  0:59 ` RunningDroid
  2023-04-13  1:46 ` RunningDroid
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: RunningDroid @ 2023-04-12  0:59 UTC (permalink / raw)
  To: ml

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

New comment by RunningDroid on void-packages repository

https://github.com/void-linux/void-packages/pull/43398#issuecomment-1504359101

Comment:
I stumbled across a [regression](https://github.com/thp/urlwatch/issues/755) when I tried to add upstream's dependency docs to my local urlwatch*, so I'd prefer to wait and see what upstream does before merging but I'm not opposed to merging as-is.

*: I have an [alias set up](https://gitlab.com/RunningDroid/minidots/-/blob/master/.zsh/aliases#L29-31) for some sites, I only came across this because I tried to add it to urlwatch without an alias.

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

* Re: urlwatch: update to 2.26
  2023-04-11 21:10 [PR PATCH] urlwatch: update to 2.26 RunningDroid
                   ` (5 preceding siblings ...)
  2023-04-12  0:59 ` RunningDroid
@ 2023-04-13  1:46 ` RunningDroid
  2023-04-13  2:08 ` [PR PATCH] [Updated] " RunningDroid
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: RunningDroid @ 2023-04-13  1:46 UTC (permalink / raw)
  To: ml

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

New comment by RunningDroid on void-packages repository

https://github.com/void-linux/void-packages/pull/43398#issuecomment-1504359101

Comment:
I stumbled across a ~~regression~~ [bug](https://github.com/thp/urlwatch/issues/755) when I tried to add upstream's dependency docs to my local urlwatch*, so I'd prefer to wait and see what upstream does before merging but I'm not opposed to merging as-is.

*: I have an [alias set up](https://gitlab.com/RunningDroid/minidots/-/blob/master/.zsh/aliases#L29-31) for some sites, I only came across this because I tried to add it to urlwatch without an alias.

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

* Re: [PR PATCH] [Updated] urlwatch: update to 2.26
  2023-04-11 21:10 [PR PATCH] urlwatch: update to 2.26 RunningDroid
                   ` (6 preceding siblings ...)
  2023-04-13  1:46 ` RunningDroid
@ 2023-04-13  2:08 ` RunningDroid
  2023-04-13  2:10 ` RunningDroid
  2023-04-23 22:25 ` [PR PATCH] [Merged]: " Duncaen
  9 siblings, 0 replies; 11+ messages in thread
From: RunningDroid @ 2023-04-13  2:08 UTC (permalink / raw)
  To: ml

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

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

https://github.com/RunningDroid/void-packages update_urlwatch
https://github.com/void-linux/void-packages/pull/43398

urlwatch: update to 2.26
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### 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**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


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

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

From fbb0ab01178a883fb5bf68bb1a60c216a0bd8117 Mon Sep 17 00:00:00 2001
From: RunningDroid <runningdroid@zoho.com>
Date: Tue, 11 Apr 2023 17:09:02 -0400
Subject: [PATCH] urlwatch: update to 2.26

---
 ...afbb047060277b00f570345e7e1969266712.patch | 88 +++++++++++++++++++
 srcpkgs/urlwatch/patches/test_handler.patch   | 14 ---
 srcpkgs/urlwatch/template                     | 18 ++--
 3 files changed, 96 insertions(+), 24 deletions(-)
 create mode 100644 srcpkgs/urlwatch/patches/6b68afbb047060277b00f570345e7e1969266712.patch
 delete mode 100644 srcpkgs/urlwatch/patches/test_handler.patch

diff --git a/srcpkgs/urlwatch/patches/6b68afbb047060277b00f570345e7e1969266712.patch b/srcpkgs/urlwatch/patches/6b68afbb047060277b00f570345e7e1969266712.patch
new file mode 100644
index 000000000000..832b963517e0
--- /dev/null
+++ b/srcpkgs/urlwatch/patches/6b68afbb047060277b00f570345e7e1969266712.patch
@@ -0,0 +1,88 @@
+From 6b68afbb047060277b00f570345e7e1969266712 Mon Sep 17 00:00:00 2001
+From: Thomas Perl <m@thp.io>
+Date: Wed, 12 Apr 2023 15:22:02 +0200
+Subject: [PATCH] Rework handling of running from a source checkout (fixes
+ #755)
+
+diff --git a/CHANGELOG.md b/CHANGELOG.md
+index 657e0494..fccf1f1d 100644
+--- a/CHANGELOG.md
++++ b/CHANGELOG.md
+@@ -4,6 +4,13 @@ All notable changes to this project will be documented in this file.
+ 
+ The format mostly follows [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
+ 
++## UNRELEASED
++
++### Fixed
++
++- Rework handling of running from a source checkout, fixes issues with example files
++  when `urlwatch` was run as `/usr/sbin/urlwatch`, e.g. on Void Linux (fixes #755)
++
+ ## [2.26] -- 2023-04-11
+ 
+ ### Added
+diff --git a/lib/urlwatch/cli.py b/lib/urlwatch/cli.py
+index 75332f09..f6a5802a 100755
+--- a/lib/urlwatch/cli.py
++++ b/lib/urlwatch/cli.py
+@@ -47,9 +47,6 @@
+ # Check if we are installed in the system already
+ (prefix, bindir) = os.path.split(os.path.dirname(os.path.abspath(sys.argv[0])))
+ 
+-if bindir != 'bin':
+-    sys.path.insert(0, os.path.join(prefix, bindir, 'lib'))
+-
+ from urlwatch.command import UrlwatchCommand
+ from urlwatch.config import CommandConfig
+ from urlwatch.main import Urlwatch
+@@ -90,7 +87,7 @@ def main():
+     if os.path.exists(old_cache_file) and not os.path.exists(new_cache_file):
+         cache_file = old_cache_file
+ 
+-    command_config = CommandConfig(sys.argv[1:], pkgname, urlwatch_dir, bindir, prefix,
++    command_config = CommandConfig(sys.argv[1:], pkgname, urlwatch_dir, prefix,
+                                    config_file, urls_file, hooks_file, cache_file, False)
+     setup_logger(command_config.verbose)
+ 
+diff --git a/lib/urlwatch/config.py b/lib/urlwatch/config.py
+index c1075460..40d206ec 100644
+--- a/lib/urlwatch/config.py
++++ b/lib/urlwatch/config.py
+@@ -52,20 +52,12 @@ def __init__(self, pkgname, urlwatch_dir, config, urls, cache, hooks, verbose):
+ 
+ class CommandConfig(BaseConfig):
+ 
+-    def __init__(self, args, pkgname, urlwatch_dir, bindir, prefix, config, urls, hooks, cache, verbose):
++    def __init__(self, args, pkgname, urlwatch_dir, prefix, config, urls, hooks, cache, verbose):
+         super().__init__(pkgname, urlwatch_dir, config, urls, cache, hooks, verbose)
+-        self.bindir = bindir
+-        self.prefix = prefix
+         self.migrate_cache = migrate_cache
+         self.migrate_urls = migrate_urls
+ 
+-        if self.bindir == 'bin':
+-            # Installed system-wide
+-            self.examples_dir = os.path.join(prefix, 'share', self.pkgname, 'examples')
+-        else:
+-            # Assume we are not yet installed
+-            self.examples_dir = os.path.join(prefix, bindir, 'share', self.pkgname, 'examples')
+-
++        self.examples_dir = os.path.join(prefix, 'share', self.pkgname, 'examples')
+         self.urls_yaml_example = os.path.join(self.examples_dir, 'urls.yaml.example')
+         self.hooks_py_example = os.path.join(self.examples_dir, 'hooks.py.example')
+ 
+diff --git a/lib/urlwatch/tests/test_handler.py b/lib/urlwatch/tests/test_handler.py
+index 1d5d22e6..7886acc9 100644
+--- a/lib/urlwatch/tests/test_handler.py
++++ b/lib/urlwatch/tests/test_handler.py
+@@ -89,8 +89,7 @@ def test_pep8_conformance():
+ 
+ class ConfigForTest(CommandConfig):
+     def __init__(self, config, urls, cache, hooks, verbose):
+-        (prefix, bindir) = os.path.split(os.path.dirname(os.path.abspath(sys.argv[0])))
+-        super().__init__([], 'urlwatch', os.path.dirname(__file__), bindir, prefix, config, urls, hooks, cache, verbose)
++        super().__init__([], 'urlwatch', os.path.dirname(__file__), root, config, urls, hooks, cache, verbose)
+ 
+ 
+ @contextlib.contextmanager
diff --git a/srcpkgs/urlwatch/patches/test_handler.patch b/srcpkgs/urlwatch/patches/test_handler.patch
deleted file mode 100644
index 73f301e49f9f..000000000000
--- a/srcpkgs/urlwatch/patches/test_handler.patch
+++ /dev/null
@@ -1,14 +0,0 @@
-Work around https://github.com/thp/urlwatch/issues/677
-
-diff --git a/lib/urlwatch/tests/test_handler.py b/lib/urlwatch/tests/test_handler.py
-index 736536a..fac9449 100644
---- a/lib/urlwatch/tests/test_handler.py
-+++ b/lib/urlwatch/tests/test_handler.py
-@@ -89,6 +89,7 @@ def test_pep8_conformance():
- 
- class ConfigForTest(CommandConfig):
-     def __init__(self, config, urls, cache, hooks, verbose):
-+        sys.argv = sys.argv[:1]
-         (prefix, bindir) = os.path.split(os.path.dirname(os.path.abspath(sys.argv[0])))
-         super().__init__('urlwatch', os.path.dirname(__file__), bindir, prefix, config, urls, hooks, cache, verbose)
- 
diff --git a/srcpkgs/urlwatch/template b/srcpkgs/urlwatch/template
index bdc8421176c2..58287278fe90 100644
--- a/srcpkgs/urlwatch/template
+++ b/srcpkgs/urlwatch/template
@@ -1,27 +1,25 @@
 # Template file for 'urlwatch'
 pkgname=urlwatch
-version=2.25
-revision=2
+version=2.26
+revision=1
 build_style=python3-module
+# skip the tests that require python modules that aren't packaged
+# (pdftotext & pytesseract)
+make_check_args="-k not((pdf)or(ocr))"
 hostmakedepends="python3-setuptools"
 depends="python3-appdirs python3-keyring python3-minidb python3-requests
  python3-yaml python3-lxml python3-cssselect"
 # Check the Docs for optional packages:
 # https://urlwatch.readthedocs.io/en/latest/dependencies.html#optional-packages
 checkdepends="python3-pytest python3-pycodestyle python3-docutils
- python3-Pygments python3-jq ${depends}"
+ python3-jq ${depends}"
 short_desc="Tool for monitoring webpages for updates"
 maintainer="RunningDroid <runningdroid@zoho.com>"
 license="BSD-3-Clause"
 homepage="https://thp.io/2008/urlwatch/"
+changelog="https://raw.githubusercontent.com/thp/urlwatch/master/CHANGELOG.md"
 distfiles="${PYPI_SITE}/u/urlwatch/urlwatch-${version}.tar.gz"
-checksum=6802297d3318286e7f3d36b9a4567a2fb09b0ae779d4b76811dd29a7281c1f8a
-
-do_check() {
-	# skip the tests that require python modules that aren't packaged
-	# (pdftotext & pytesseract)
-	pytest -k 'not (pdf or ocr)' -v
-}
+checksum=b14bbe23d87b7d4ac8b4e71a51fd2066908a947ccf7b557c3e4f87b09f455e8f
 
 post_install() {
 	vlicense COPYING

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

* Re: urlwatch: update to 2.26
  2023-04-11 21:10 [PR PATCH] urlwatch: update to 2.26 RunningDroid
                   ` (7 preceding siblings ...)
  2023-04-13  2:08 ` [PR PATCH] [Updated] " RunningDroid
@ 2023-04-13  2:10 ` RunningDroid
  2023-04-23 22:25 ` [PR PATCH] [Merged]: " Duncaen
  9 siblings, 0 replies; 11+ messages in thread
From: RunningDroid @ 2023-04-13  2:10 UTC (permalink / raw)
  To: ml

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

New comment by RunningDroid on void-packages repository

https://github.com/void-linux/void-packages/pull/43398#issuecomment-1506225225

Comment:
I've pulled in upstream's patch for the issue I came across, this is ready to merge whenever anyone has time

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

* Re: [PR PATCH] [Merged]: urlwatch: update to 2.26
  2023-04-11 21:10 [PR PATCH] urlwatch: update to 2.26 RunningDroid
                   ` (8 preceding siblings ...)
  2023-04-13  2:10 ` RunningDroid
@ 2023-04-23 22:25 ` Duncaen
  9 siblings, 0 replies; 11+ messages in thread
From: Duncaen @ 2023-04-23 22:25 UTC (permalink / raw)
  To: ml

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

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

urlwatch: update to 2.26
https://github.com/void-linux/void-packages/pull/43398

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

#### 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**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


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

end of thread, other threads:[~2023-04-23 22:25 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-11 21:10 [PR PATCH] urlwatch: update to 2.26 RunningDroid
2023-04-11 23:04 ` [PR REVIEW] " mhmdanas
2023-04-12  0:38 ` [PR PATCH] [Updated] " RunningDroid
2023-04-12  0:41 ` RunningDroid
2023-04-12  0:42 ` [PR PATCH] [Updated] " RunningDroid
2023-04-12  0:43 ` RunningDroid
2023-04-12  0:59 ` RunningDroid
2023-04-13  1:46 ` RunningDroid
2023-04-13  2:08 ` [PR PATCH] [Updated] " RunningDroid
2023-04-13  2:10 ` RunningDroid
2023-04-23 22:25 ` [PR PATCH] [Merged]: " Duncaen

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