Bug#769526: xlshlt: xslhlt should use configuration file provided by package by default

2018-03-03 Thread Vasiliy Shlykov
Package: xlshlt Version: xslhlt Severity: minor Tags: patch Dear Maintainer, xslhlt package provides the standard configuration file /usr/share/xslthl/highlighters/xslthl-config.xml within. But unfortunately I always have to set filename explicitly by java command-line option «-Dxslthl.config»

Bug#769526: xlshlt: xslhlt should use configuration file provided by package by default

2014-11-17 Thread Шлыков Василий
16.11.2014 12:54, Eugene Zhukov пишет: Hi and thank you for your bug report! I examined the patch and have a question about it. What is the purpose of } else if (new File(xslthl-config.xml).isFile()) {? As far as I understand it will always be true. What did I miss? I would only change

Bug#769526: xlshlt: xslhlt should use configuration file provided by package by default

2014-11-17 Thread Eugene Zhukov
Thanks for clarifications Niels and Vasiliy. I will upload the fix shortly. As of xslthl-config.xml being in /etc, I don't think this is a configuration file a user would want to change (arguably it's in a preferred form of modification) - it's either provided or not. Eugene __ This is the

Bug#769526: xlshlt: xslhlt should use configuration file provided by package by default

2014-11-16 Thread Eugene Zhukov
Hi and thank you for your bug report! I examined the patch and have a question about it. What is the purpose of } else if (new File(xslthl-config.xml).isFile()) {? As far as I understand it will always be true. What did I miss? I would only change configFilename = xslthl-config.xml; to

Bug#769526: xlshlt: xslhlt should use configuration file provided by package by default

2014-11-16 Thread Niels Thykier
On 2014-11-16 10:54, Eugene Zhukov wrote: Hi and thank you for your bug report! I examined the patch and have a question about it. What is the purpose of } else if (new File(xslthl-config.xml).isFile()) {? As far as I understand it will always be true. What did I miss? I would only change