[PR PATCH] pdfpc: disable REST server

[PR PATCH] pdfpc: disable REST server

[oreo639]

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

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

https://github.com/oreo639/void-packages pdfpc
https://github.com/void-linux/void-packages/pull/48959

pdfpc: disable REST server
Build with libwebkit2gtk41

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

#### Testing the changes
- I tested the changes in this PR: **YES**|**briefly**|**NO**

<!--
#### 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/48959.patch is attached

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

From 640219b5cd646fd9e251ef1c0ef1dbc73e54199d Mon Sep 17 00:00:00 2001
From: oreo639 <oreo6391@gmail.com>
Date: Mon, 26 Feb 2024 16:29:40 -0800
Subject: [PATCH] pdfpc: disable REST server

Build with libwebkit2gtk41
---
 srcpkgs/pdfpc/template | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/pdfpc/template b/srcpkgs/pdfpc/template
index 450a9f300730e..0dcab66fb0fc7 100644
--- a/srcpkgs/pdfpc/template
+++ b/srcpkgs/pdfpc/template
@@ -1,12 +1,13 @@
 # Template file for 'pdfpc'
 pkgname=pdfpc
 version=4.6.0
-revision=1
+revision=2
 build_style=cmake
+configure_args="-DREST=OFF"
 hostmakedepends="pkg-config vala libgee08-devel"
 makedepends="gst-plugins-base1-devel gtk+3-devel libgee08-devel
  libzstd-devel poppler-glib-devel json-glib-devel vala-devel
- webkit2gtk-devel discount-devel qrencode-devel"
+ libwebkit2gtk41-devel discount-devel"
 short_desc="Presenter console with multi-monitor support for PDF files"
 maintainer="Aaron Marcher <info@nulltime.net>"
 license="GPL-2.0-or-later"
@@ -14,3 +15,7 @@ homepage="https://pdfpc.github.io/"
 changelog="https://raw.githubusercontent.com/pdfpc/pdfpc/master/CHANGELOG.txt"
 distfiles="https://github.com/pdfpc/pdfpc/archive/v${version}.tar.gz"
 checksum=3b1a393f36a1b0ddc29a3d5111d8707f25fb2dd2d93b0401ff1c66fa95f50294
+
+post_patch() {
+	vsed -e 's/webkit2gtk-4.0/webkit2gtk-4.1/' -i src/CMakeLists.txt
+}

Re: pdfpc: disable REST server

[leahneukirchen]

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

New comment by leahneukirchen on void-packages repository

https://github.com/void-linux/void-packages/pull/48959#issuecomment-1966724067

Comment:
Please write one sentence why. There's probably a good reason. ;)

Also `changelog=` is 404.

Re: pdfpc: disable REST server

[oreo639]

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

New comment by oreo639 on void-packages repository

https://github.com/void-linux/void-packages/pull/48959#issuecomment-1967540487

Comment:
> Please write one sentence why. There's probably a good reason. ;)

Sorry about that. I thought I did, but I think it got lost and I was too tired to notice.
I added it now.

Re: pdfpc: disable REST server

[oreo639]

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

New comment by oreo639 on void-packages repository

https://github.com/void-linux/void-packages/pull/48959#issuecomment-1967540487

Comment:
> Please write one sentence why. There's probably a good reason. ;)

Sorry about that. I thought I did, but I think it got lost and I didn't notice.
I added it now.

Re: [PR PATCH] [Updated] pdfpc: disable REST server

[oreo639]

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

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

https://github.com/oreo639/void-packages pdfpc
https://github.com/void-linux/void-packages/pull/48959

pdfpc: disable REST server
<!-- Uncomment relevant sections and delete options which are not applicable -->

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

Build with webkit2gtk-4.1, which provides an identical api to webkit2gtk-4.0 except it uses libsoup3.
The REST server functionality requires libsoup2. libsoup2 and libsoup3 cannot be loaded in the same process at the same time.

One of the maintainers recommended just disabling the rest server and building the markdown presentation notes viewer with webkit2gtk-4.1 stating that as far as he knows, he is the only one that actually uses the rest server function. https://github.com/pdfpc/pdfpc/issues/671#issuecomment-1552713117

I was wondering if anyone here that uses pdfpc uses the rest server function?

<!--
#### 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/48959.patch is attached

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

From f00c2e41dd9a59a9a3b75f6efd9872485b512b5e Mon Sep 17 00:00:00 2001
From: oreo639 <oreo6391@gmail.com>
Date: Mon, 26 Feb 2024 16:29:40 -0800
Subject: [PATCH] pdfpc: disable REST server

Build with libwebkit2gtk41
REST functionality cannot be used with libwebkit2gtk41 since it uses libsoup2.
---
 srcpkgs/pdfpc/template | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/pdfpc/template b/srcpkgs/pdfpc/template
index 450a9f300730ea..c5eff73cd6fabd 100644
--- a/srcpkgs/pdfpc/template
+++ b/srcpkgs/pdfpc/template
@@ -1,16 +1,21 @@
 # Template file for 'pdfpc'
 pkgname=pdfpc
 version=4.6.0
-revision=1
+revision=2
 build_style=cmake
+configure_args="-DREST=OFF"
 hostmakedepends="pkg-config vala libgee08-devel"
 makedepends="gst-plugins-base1-devel gtk+3-devel libgee08-devel
  libzstd-devel poppler-glib-devel json-glib-devel vala-devel
- webkit2gtk-devel discount-devel qrencode-devel"
+ libwebkit2gtk41-devel discount-devel"
 short_desc="Presenter console with multi-monitor support for PDF files"
 maintainer="Aaron Marcher <info@nulltime.net>"
 license="GPL-2.0-or-later"
 homepage="https://pdfpc.github.io/"
-changelog="https://raw.githubusercontent.com/pdfpc/pdfpc/master/CHANGELOG.txt"
+changelog="https://raw.githubusercontent.com/pdfpc/pdfpc/master/CHANGELOG.rst"
 distfiles="https://github.com/pdfpc/pdfpc/archive/v${version}.tar.gz"
 checksum=3b1a393f36a1b0ddc29a3d5111d8707f25fb2dd2d93b0401ff1c66fa95f50294
+
+post_patch() {
+	vsed -e 's/webkit2gtk-4.0/webkit2gtk-4.1/' -i src/CMakeLists.txt
+}

Re: pdfpc: disable REST server

[sgn]

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

New comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/pull/48959#issuecomment-1970836829

Comment:
I would write the whole paragraph into commit msg:
> The REST server functionality requires libsoup2. libsoup2 and libsoup3 cannot be loaded in the same process at the same time.
>
> One of the maintainers recommended just disabling the rest server and building the markdown presentation notes viewer with webkit2gtk-4.1 stating that as far as he knows, he is the only one that actually uses the rest server function. pdfpc/pdfpc#671 (comment)

Re: [PR PATCH] [Updated] pdfpc: disable REST server

[oreo639]

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

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

https://github.com/oreo639/void-packages pdfpc
https://github.com/void-linux/void-packages/pull/48959

pdfpc: disable REST server
<!-- Uncomment relevant sections and delete options which are not applicable -->

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

Build with webkit2gtk-4.1, which provides an identical api to webkit2gtk-4.0 except it uses libsoup3.
The REST server functionality requires libsoup2. libsoup2 and libsoup3 cannot be loaded in the same process at the same time.

One of the maintainers recommended just disabling the rest server and building the markdown presentation notes viewer with webkit2gtk-4.1 stating that as far as he knows, he is the only one that actually uses the rest server function. https://github.com/pdfpc/pdfpc/issues/671#issuecomment-1552713117

I was wondering if anyone here that uses pdfpc uses the rest server function?

<!--
#### 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/48959.patch is attached

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

From b3e0e5bbe9ecb65d1902f6178a52de42ca05c5aa Mon Sep 17 00:00:00 2001
From: oreo639 <oreo6391@gmail.com>
Date: Mon, 26 Feb 2024 16:29:40 -0800
Subject: [PATCH] pdfpc: disable REST server

Build with libwebkit2gtk41

REST server functionality requires libsoup2, however libsoup2 cannot be loaded
int the same process as libsoup3.

One of the maintainers reccomended disabling REST and building presentation
notes with webkit2gtk-4.1, stating that as far as he knows, he is the only one
using the REST server functionality.

https://github.com/pdfpc/pdfpc/issues/671
---
 srcpkgs/pdfpc/template | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/pdfpc/template b/srcpkgs/pdfpc/template
index 450a9f300730ea..c5eff73cd6fabd 100644
--- a/srcpkgs/pdfpc/template
+++ b/srcpkgs/pdfpc/template
@@ -1,16 +1,21 @@
 # Template file for 'pdfpc'
 pkgname=pdfpc
 version=4.6.0
-revision=1
+revision=2
 build_style=cmake
+configure_args="-DREST=OFF"
 hostmakedepends="pkg-config vala libgee08-devel"
 makedepends="gst-plugins-base1-devel gtk+3-devel libgee08-devel
  libzstd-devel poppler-glib-devel json-glib-devel vala-devel
- webkit2gtk-devel discount-devel qrencode-devel"
+ libwebkit2gtk41-devel discount-devel"
 short_desc="Presenter console with multi-monitor support for PDF files"
 maintainer="Aaron Marcher <info@nulltime.net>"
 license="GPL-2.0-or-later"
 homepage="https://pdfpc.github.io/"
-changelog="https://raw.githubusercontent.com/pdfpc/pdfpc/master/CHANGELOG.txt"
+changelog="https://raw.githubusercontent.com/pdfpc/pdfpc/master/CHANGELOG.rst"
 distfiles="https://github.com/pdfpc/pdfpc/archive/v${version}.tar.gz"
 checksum=3b1a393f36a1b0ddc29a3d5111d8707f25fb2dd2d93b0401ff1c66fa95f50294
+
+post_patch() {
+	vsed -e 's/webkit2gtk-4.0/webkit2gtk-4.1/' -i src/CMakeLists.txt
+}

Re: [PR PATCH] [Updated] pdfpc: disable REST server

[oreo639]

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

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

https://github.com/oreo639/void-packages pdfpc
https://github.com/void-linux/void-packages/pull/48959

pdfpc: disable REST server
<!-- Uncomment relevant sections and delete options which are not applicable -->

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

Build with webkit2gtk-4.1, which provides an identical api to webkit2gtk-4.0 except it uses libsoup3.
The REST server functionality requires libsoup2. libsoup2 and libsoup3 cannot be loaded in the same process at the same time.

One of the maintainers recommended just disabling the rest server and building the markdown presentation notes viewer with webkit2gtk-4.1 stating that as far as he knows, he is the only one that actually uses the rest server function. https://github.com/pdfpc/pdfpc/issues/671#issuecomment-1552713117

I was wondering if anyone here that uses pdfpc uses the rest server function?

<!--
#### 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/48959.patch is attached

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

From 1ee657faa881ff9e345367d6fbaf1a106a430975 Mon Sep 17 00:00:00 2001
From: oreo639 <oreo6391@gmail.com>
Date: Mon, 26 Feb 2024 16:29:40 -0800
Subject: [PATCH] pdfpc: disable REST server

Build with libwebkit2gtk41

REST server functionality requires libsoup2, however libsoup2 cannot be loaded
in to the same process as libsoup3.

One of the maintainers reccomended disabling REST and building presentation
notes with webkit2gtk-4.1, stating that as far as he knows, he is the only one
using the REST server functionality.

https://github.com/pdfpc/pdfpc/issues/671
---
 srcpkgs/pdfpc/template | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/pdfpc/template b/srcpkgs/pdfpc/template
index 450a9f300730ea..c5eff73cd6fabd 100644
--- a/srcpkgs/pdfpc/template
+++ b/srcpkgs/pdfpc/template
@@ -1,16 +1,21 @@
 # Template file for 'pdfpc'
 pkgname=pdfpc
 version=4.6.0
-revision=1
+revision=2
 build_style=cmake
+configure_args="-DREST=OFF"
 hostmakedepends="pkg-config vala libgee08-devel"
 makedepends="gst-plugins-base1-devel gtk+3-devel libgee08-devel
  libzstd-devel poppler-glib-devel json-glib-devel vala-devel
- webkit2gtk-devel discount-devel qrencode-devel"
+ libwebkit2gtk41-devel discount-devel"
 short_desc="Presenter console with multi-monitor support for PDF files"
 maintainer="Aaron Marcher <info@nulltime.net>"
 license="GPL-2.0-or-later"
 homepage="https://pdfpc.github.io/"
-changelog="https://raw.githubusercontent.com/pdfpc/pdfpc/master/CHANGELOG.txt"
+changelog="https://raw.githubusercontent.com/pdfpc/pdfpc/master/CHANGELOG.rst"
 distfiles="https://github.com/pdfpc/pdfpc/archive/v${version}.tar.gz"
 checksum=3b1a393f36a1b0ddc29a3d5111d8707f25fb2dd2d93b0401ff1c66fa95f50294
+
+post_patch() {
+	vsed -e 's/webkit2gtk-4.0/webkit2gtk-4.1/' -i src/CMakeLists.txt
+}

Re: [PR PATCH] [Merged]: pdfpc: disable REST server

[oreo639]

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

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

pdfpc: disable REST server
https://github.com/void-linux/void-packages/pull/48959

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

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

Build with webkit2gtk-4.1, which provides an identical api to webkit2gtk-4.0 except it uses libsoup3.
The REST server functionality requires libsoup2. libsoup2 and libsoup3 cannot be loaded in the same process at the same time.

One of the maintainers recommended just disabling the rest server and building the markdown presentation notes viewer with webkit2gtk-4.1 stating that as far as he knows, he is the only one that actually uses the rest server function. https://github.com/pdfpc/pdfpc/issues/671#issuecomment-1552713117

I was wondering if anyone here that uses pdfpc uses the rest server function?

<!--
#### 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
-->