Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pdfpc: disable REST server #48959

Merged
merged 1 commit into from
Apr 9, 2024
Merged

pdfpc: disable REST server #48959

merged 1 commit into from
Apr 9, 2024

Conversation

oreo639
Copy link
Member

@oreo639 oreo639 commented Feb 27, 2024

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. pdfpc/pdfpc#671 (comment)

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

@leahneukirchen
Copy link
Member

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

Also changelog= is 404.

@oreo639
Copy link
Member Author

oreo639 commented Feb 27, 2024

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.

@sgn
Copy link
Member

sgn commented Feb 29, 2024

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)

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.

pdfpc/pdfpc#671
@oreo639 oreo639 merged commit da0ce6b into void-linux:master Apr 9, 2024
8 checks passed
@oreo639 oreo639 deleted the pdfpc branch April 9, 2024 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants