RFC: Drop embedded MathJax in favor of using distro-packaged one? #158

Closed
monsta opened this Issue Sep 7, 2015 · 9 comments

Projects

None yet

7 participants

@monsta
Member
monsta commented Sep 7, 2015

Well, it has been done in Debian here - as a patch. Can we do the same upstream and drop the excessive amount of source code?

@infirit @flexiondotorg @NiceandGently @willysr @posophe @obache
thoughts?

yes, some of you have already confirmed that MathJax is in your distros, I know 😄

@infirit
Contributor
infirit commented Sep 7, 2015

I am not against removing and relying on the distro's packaged version. I am only concerned about distro's that do not have this packaged, slackware was one but then I don't think @willysr enabled epub in atril..

@infirit infirit added the confirmed label Sep 7, 2015
@infirit infirit changed the title from Drop embedded MathJax in favor of using distro-packaged one? to RFC: Drop embedded MathJax in favor of using distro-packaged one? Sep 7, 2015
@flexiondotorg
Member

I agree with this proposal.

@willysr
willysr commented Sep 7, 2015

yes, go on with this idea :)
just let me know when it has become a mandatory dep

@obache
obache commented Sep 8, 2015

I agree with this idea too.

I don't know whether "/usr/share/javascript/mathjax" is the de facto standard location for MathJax installation or not, but it is better to add a configure option to specify the location (with existence and required version check). I will also avoid potential missing dependency from packaging.

@posophe
posophe commented Sep 8, 2015

I agree too. And don't worry, I'll package it by basing it on Fedora's

@monsta
Member
monsta commented Sep 10, 2015

Ok, there goes #160.

@raveit65
Member

@monsta
Do i have to include MathJax as runtime require in downstream package?

@raveit65
Member

ok. i see it is already in debian control

@monsta
Member
monsta commented Oct 14, 2015

Yes, runtime dep has to be there - in debian it's libjs-mathjax.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment