Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] zathura: add patch to fix valid file detection
@ 2024-01-29 15:53 balejk
  2024-01-29 20:27 ` [PR PATCH] [Merged]: " cinerea0
  0 siblings, 1 reply; 2+ messages in thread
From: balejk @ 2024-01-29 15:53 UTC (permalink / raw)
  To: ml

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

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

https://github.com/balejk/void-packages zathura-patch
https://github.com/void-linux/void-packages/pull/48423

zathura: add patch to fix valid file detection
Without this patch, it is not possible to use completion for zathura's
`:open` command as it will only list files which zathura cannot open and
exclude all other, instead of doing it the other way around.

---

<!-- 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**|**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, (x86_64-musl)


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

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

From c228362d7e47d06fcad294b78d223c785654b542 Mon Sep 17 00:00:00 2001
From: Karel Balej <balejk@matfyz.cz>
Date: Mon, 29 Jan 2024 16:48:10 +0100
Subject: [PATCH] zathura: add patch to fix valid file detection

Without this patch, it is not possible to use completion for zathura's
`:open` command as it will only list files which zathura cannot open and
exclude all other, instead of doing it the other way around.
---
 ...e_valid_extension-should-return-True.patch | 26 +++++++++++++++++++
 srcpkgs/zathura/template                      |  2 +-
 2 files changed, 27 insertions(+), 1 deletion(-)
 create mode 100644 srcpkgs/zathura/patches/0001-The-function-file_valid_extension-should-return-True.patch

diff --git a/srcpkgs/zathura/patches/0001-The-function-file_valid_extension-should-return-True.patch b/srcpkgs/zathura/patches/0001-The-function-file_valid_extension-should-return-True.patch
new file mode 100644
index 0000000000000..517c0d49cb3db
--- /dev/null
+++ b/srcpkgs/zathura/patches/0001-The-function-file_valid_extension-should-return-True.patch
@@ -0,0 +1,26 @@
+From d74091a7452846112c0bb11e1412cd2af48ee7cd Mon Sep 17 00:00:00 2001
+From: Carlo Buccisano <slashdeipoveri@gmail.com>
+Date: Tue, 12 Dec 2023 00:24:20 +0100
+Subject: [PATCH] The function file_valid_extension should return True if
+ plugin is different then NULL, not if it is equal.
+
+---
+ zathura/utils.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/zathura/utils.c b/zathura/utils.c
+index 802757f..8f379be 100644
+--- a/zathura/utils.c
++++ b/zathura/utils.c
+@@ -60,7 +60,7 @@ file_valid_extension(zathura_t* zathura, const char* path)
+   zathura_plugin_t* plugin = zathura_plugin_manager_get_plugin(zathura->plugins.manager, content_type);
+   g_free(content_type);
+ 
+-  return plugin == NULL;
++  return plugin != NULL;
+ }
+ 
+ static void
+-- 
+2.43.0
+
diff --git a/srcpkgs/zathura/template b/srcpkgs/zathura/template
index 4ae25707e6b87..a7aafcd71f4d9 100644
--- a/srcpkgs/zathura/template
+++ b/srcpkgs/zathura/template
@@ -1,7 +1,7 @@
 # Template file for 'zathura'
 pkgname=zathura
 version=0.5.4
-revision=1
+revision=2
 build_style=meson
 configure_args="-Dsynctex=enabled"
 hostmakedepends="pkg-config intltool python3-Sphinx desktop-file-utils

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

* Re: [PR PATCH] [Merged]: zathura: add patch to fix valid file detection
  2024-01-29 15:53 [PR PATCH] zathura: add patch to fix valid file detection balejk
@ 2024-01-29 20:27 ` cinerea0
  0 siblings, 0 replies; 2+ messages in thread
From: cinerea0 @ 2024-01-29 20:27 UTC (permalink / raw)
  To: ml

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

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

zathura: add patch to fix valid file detection
https://github.com/void-linux/void-packages/pull/48423

Description:
Without this patch, it is not possible to use completion for zathura's
`:open` command as it will only list files which zathura cannot open and
exclude all other, instead of doing it the other way around.

---

<!-- 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**|**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, (x86_64-musl)


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

end of thread, other threads:[~2024-01-29 20:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-29 15:53 [PR PATCH] zathura: add patch to fix valid file detection balejk
2024-01-29 20:27 ` [PR PATCH] [Merged]: " cinerea0

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