FS#56049 - [qpdfview] texlive-bin should be optional

Attached to Project: Community Packages
Opened by Andrea Diamantini (adjam) - Thursday, 19 October 2017, 15:45 GMT
Last edited by Balló György (City-busz) - Wednesday, 17 January 2018, 14:09 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Alad Wenter (Alad)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

Description:
Texlive-bin has been moved from build dep to runtime dep. Is this really needed?
This should be a lightweight pdf viewer.

Additional info:
* package version(s)
* config and/or log files etc.

package version: 0.4.17beta1-4

Steps to reproduce:
This task depends upon

Closed by  Balló György (City-busz)
Wednesday, 17 January 2018, 14:09 GMT
Reason for closing:  Fixed
Additional comments about closing:  qpdfview 0.4.17beta1-5
Comment by vicencb (vicencb) - Thursday, 19 October 2017, 16:56 GMT
That extra required dependency (recursively) needs about 50 more MiB while qpdfview+poppler are both less than 8 MiB. That is overkill for a feature not as widely used as pdf files.
Please, make it an optional dependency.
Comment by Alad Wenter (Alad) - Thursday, 19 October 2017, 18:01 GMT
The old AUR package had a questionable system in place where dependencies would be dynamically generated depending on what packages are installed. This is obviously even less feasible for a repository package, so at minimum it is a makedep. However, from my testing qpdfview doesn't run unless texlive-bin is _also_ a runtime dependency. Complain to upstream to change their build system if those few dozen MBs bother you that much.
Comment by Balló György (City-busz) - Wednesday, 17 January 2018, 14:09 GMT
I checked the source code, and texlive-bin is not used at all by qpdfview. Only the libsynctex library is needed, so I removed the texlive-bin dependency in qpdfview 0.4.17beta1-5.

Loading...