Bug 736647

Summary:	app-text/atril: make synctex (tex dependency) optional
Product:	Gentoo Linux	Reporter:	Grant Edwards <grant.b.edwards>
Component:	Current packages	Assignee:	Gentoo MATE Desktop <mate>
Status:	RESOLVED FIXED
Severity:	normal	CC:	jstein, sam
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	Patch to make synctex support optional ebuild to make synctex support optional

Description Grant Edwards 2020-08-10 15:48:38 UTC

The 1.24.0-r1 ebuid requires that TeX be installed.  This dependency
didn't exist in older versions of Atril and is rather onerous if you
don't use TeX/LaTeX. This dependency was due to a new 1.24 upstream
requirement for an external 'synctex' library instead of previous
"built-in" synctex support.

Upstream has fixed this at https://github.com/mate-desktop/atril/pull/436,
and synctex support is now optional.  Until that change appears
downstream, the attached ebuild and patch make synctex support
optional (via the existing 'synctex' USE flag).  The synctex support
is disabled by default, which I believe matches the new upstream
behavior.


Reproducible: Always

Comment 1 Grant Edwards 2020-08-10 15:49:40 UTC

Created attachment 654043 [details, diff]
Patch to make synctex support optional

Comment 2 Grant Edwards 2020-08-10 15:50:50 UTC

Created attachment 654045 [details]
ebuild to make synctex support optional

Comment 3 Grant Edwards 2020-08-10 17:47:11 UTC

Just to illustrate what I meant by "onerous": without the attached patch, emerge
demanded to install 49 new packages in order to upgrade atril from 1.23 to 1.24.

Comment 4 OzTiram 2020-08-11 06:48:06 UTC

I'm glad to see this issue. I have a similar ebuild in my mate-de-gentoo overlay. Also, a similar patch is already merged upstream. It is found in the 1.25 release of atril.

Comment 5 Adam Feldman gentoo-dev

2020-09-05 01:44:51 UTC

Thanks for submitting the patches.  FYI for the future, try to keep USE flags, DEPENDencies and configure args in alphabetical order.

Comment 6 Sam James archtester

2021-03-17 04:25:03 UTC

ping?

Comment 7 OzTiram 2021-03-17 07:50:29 UTC

The patch to remove depends on tex has been merged upstream.
Also, the latest ebuild in gentoo's tree has already a synctex flag which isn't 
enabled by default. Hence, the bug can be closed.

Comment 8 Adam Feldman gentoo-dev

2021-03-17 15:58:55 UTC

(In reply to Sam James from comment #6)
> ping?

My machine is back online.  Will be addressing all outstanding MATE issues tomorrow.

Comment 9 OzTiram 2023-01-02 23:52:15 UTC

This bug can be closed. A patch by myself was accepted upstream in version 1.26 which is currently stable in Gentoo .