Adding better parameter handling

Question

Adding better parameter handling

Kogoro opened this issue 8 years ago · comments

We should improve the parameter handling through a config file or flags in the command line call.
This makes it even easier with optional parameters.

Christopher So · Answer 1 · Thu Sep 15 2016 18:53:48 GMT+0800 (China Standard Time)

.ini files are now supported with the following syntax:

[options]
bibtex-dir=
main-dir=
out-dir-rel=
pdf-dir=
pdf-dir-rel=
tags=tt-tags
clean=true
citationService=true
citation-dir=
out-format=HTML_NEW

The .ini file should lay at the top program folder.

Thomas Thüm · Answer 2 · Wed Oct 05 2016 02:17:06 GMT+0800 (China Standard Time)

If there is just one .ini file with a fixed location, how can I start several configurations without changing the .ini file(s)?

Christopher So · Answer 3 · Wed Oct 05 2016 02:21:34 GMT+0800 (China Standard Time)

At the moment it is not possible, but I can add different path support in the next days.

Thomas Thüm · Answer 4 · Wed Oct 05 2016 02:27:50 GMT+0800 (China Standard Time)

No hurry. I noticed that I can use it with the command-line options for now.

Christopher So · Answer 5 · Fri Oct 07 2016 05:57:09 GMT+0800 (China Standard Time)

Now it is possible to give the path to the .ini file as command line parameter. So you can create different links to the executable and run different configurations. If no parameter is set, it will try to load the options.ini file in the executable's main directory.

Thomas Thüm · Answer 6 · Wed Oct 19 2016 16:31:16 GMT+0800 (China Standard Time)

I got parsing errors with the following ini file:

bibtex-dir=C:\\Users\\tthuem\\git\\paper-BibTeX\\bibtex\\
main-dir=C:\\Users\\tthuem\\git\\dropbox\\Dropbox\\Literatur\\
out-dir-rel=HTML\\
pdf-dir=..\\Library\\
pdf-dir-rel=Library\\
tags=tt-tags
clean=true
citationService=true
citation-dir=C:\\Users\\tthuem\\git\\paper-BibTeX\\bibtex\\
out-format=HTML_NEW

org.ini4j.InvalidFileFormatException: parse error (at line: 1): bibtex-dir=C:\\Users\\tthuem\\git\\paper-BibTeX\\bibtex\\
        at org.ini4j.spi.AbstractParser.parseError(AbstractParser.java:53)
        at org.ini4j.spi.IniParser.parse(IniParser.java:97)
        at org.ini4j.spi.IniParser.parse(IniParser.java:67)
        at org.ini4j.Ini.load(Ini.java:119)
        at org.ini4j.Ini.load(Ini.java:114)
        at org.ini4j.Ini.load(Ini.java:99)
        at org.ini4j.Ini.<init>(Ini.java:69)
        at de.mibtex.BibtexViewer.main(BibtexViewer.java:72)
Exception in thread "main" java.lang.NullPointerException
        at de.mibtex.BibtexViewer.main(BibtexViewer.java:76)

Can you give an example how the file is supposed to look like or correct the above example?

Christopher So · Answer 7 · Thu Oct 20 2016 00:41:20 GMT+0800 (China Standard Time)

Should now work. Seems Windows needs a special format other than Linux. Should be fixed now on the dev branch.

My configuration ini looks like this on my Windows machine:

[options]
bibtex-dir=P:\\Github\\MibTeX\\MibTeX\\bibtex\\
main-dir=P:\\Github\\MibTeX\\MibTeX\\bibtex\\
out-dir-rel=..\\HTML\\
pdf-dir=..\\Library\\
pdf-dir-rel=PDF
tags=tt-tags
clean=true
citationService=true
citation-dir=
out-format=HTML_NEW

Thomas Thüm · Answer 8 · Thu Oct 20 2016 01:23:45 GMT+0800 (China Standard Time)

Took me a while to find out that [options] is required and that you changed the order of two parameters. ;)

Seems to work fine now. Good job! :)