736647 – app-text/atril: make synctex (tex dependency) optional

Bug 736647 - app-text/atril: make synctex (tex dependency) optional

Summary: app-text/atril: make synctex (tex dependency) optional

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Gentoo MATE Desktop

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2020-08-10 15:48 UTC by Grant Edwards
Modified:	2023-01-24 01:26 UTC (History)
CC List:	2 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Patch to make synctex support optional (atril-1.24.0-make-synctex-optional.patch,13.77 KB, patch) 2020-08-10 15:49 UTC, Grant Edwards	Details \| Diff
ebuild to make synctex support optional (atril-1.24.0-r2.ebuild,2.33 KB, text/plain) 2020-08-10 15:50 UTC, Grant Edwards	Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

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 .