search

mate-desktop / atril

A document viewer for MATE
http://www.mate-desktop.org
GNU General Public License v2.0
203 stars 62 forks source link

Build issue with 1.26.0 and libsynctex #535

Closed jlocash2 closed 3 years ago

jlocash2 commented 3 years ago

Expected behaviour

Build from source without synctex installed and it builds.

Actual behaviour

configure fails because synctex isn't installed.

Steps to reproduce the behaviour

Try building it without synctex installed

MATE general version

1.26.0

Package version

Linux Distribution

My own

Link to bugreport of your Distribution (requirement)

No bug report.

configure.ac should set enable_synctex=no by default to truly make it optional.

mbkma commented 3 years ago

it is optional via --enable-synctex=false (see https://github.com/mate-desktop/atril/blob/master/configure.ac#L374)

jlocash2 commented 3 years ago

But if synctex is "optional" shouldn't the default be to disable it and enable if requested?