norbusan / rssguard-debian

Debian packaging of RSS Guard
GNU General Public License v3.0
6 stars 1 forks source link

Build the nowebengine version of this package #1

Open n-peugnet opened 3 years ago

n-peugnet commented 3 years ago

Hi, thank you for packaging rssguard for debian.

I am new to the debian workflow so i'm not sure if I should have done an RFP for this, but I was wondering if you could also build the nowebengine version of RSSGuard with this source package.

Here is how archlinux did it, it seems like it is just a matter of setting the USE_WEBENGINE=false env var.

norbusan commented 2 years ago

Hi Nicolas, I just saw that issue. Interesting idea. It would require splitting the package (documentation files, library/headers, etc etc). Surely possible, but what is the gain?

Why do you prefer a version without webengine?

n-peugnet commented 2 years ago

I very much prefer the viewer panel to match my system theme, and the nowebengine does this perfectly. Also, as the content of an RSS feed is usually primarily informative, I don't really care if some of the content (like images) is not displayed properly. If I want to read the article as it was intended to be seen, I open it in my web browser.

So I don't need the Web engine in the RSS reader, the app without it better matches my system theme which is perfect and as this version of the app is maintained upstream I hope that it won't be to much of a burden to maintain. All these reasons makes me think that it can be interesting to add this version to Debian.

USE_WEBENGINNE=true 2021-09-27-110629_561x551_scrot

USE_WEBENGINE=false 2021-09-27-110503_581x553_scrot

I know at least of one other similar case in Debian which is qbittorrent. For this program, there is also a package for the qbittorrent-nox version which is maintained upstream. This might be an example to follow.

n-peugnet commented 2 years ago

Also, as a lot of the dependencies are stripped, this makes for a much more lightweight install (almost twice as light according to the appimage releases)