Skip to content

Add "-console" parameter for Windows launcher#3242

Merged
lenhard merged 2 commits into
masterfrom
addConsolePara
Sep 29, 2017
Merged

Add "-console" parameter for Windows launcher#3242
lenhard merged 2 commits into
masterfrom
addConsolePara

Conversation

@tobiasdiez

Copy link
Copy Markdown
Member

On Windows, the install4j launcher (JabRef.exe) actually can be used to start JabRef with command line options. By default no output is shown unless the -console parameter is specified. That option is handled by install4j but also passed to the underlying jar, where it screwed up our command line parser since it was not registered as an allowed option. This is fixed with this PR.
Upshot: .\JabRef.exe -console -h prints the command line usage help page (in comparison, the old way java -jar JabRef.jar -h).

I'll update the help as soon as the PR is merged.

  • Change in CHANGELOG.md described
  • Tests created for changes
  • Screenshots added (for bigger UI changes)
  • Manually tested changed features in running JabRef
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)
  • If you changed the localization: Did you run gradle localizationUpdate?

@tobiasdiez tobiasdiez added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Sep 29, 2017
@lenhard lenhard merged commit 6cca2ed into master Sep 29, 2017
@lenhard lenhard deleted the addConsolePara branch September 29, 2017 15:27
Siedlerchr added a commit that referenced this pull request Oct 7, 2017
* upstream/master: (113 commits)
  Open statistics dialog from correct thread (#3272)
  Fix for issue 2811: bibtexkey generator does not use crossref information (#3248)
  Fix for issue 3143: Import entry from clipboard in different formats (#3243)
  French translation correction (#3262)
  Wait to ask to collect anonymous statistics in JabRefExecutorService to allow jvm to terminate (#3266)
  Directory pattern bracketed expressions (#3238)
  Show development information
  Release v4.0
  add another author to mailmap
  moved changelog entry to the right category
  update new AUTHORS info
  Update log4j from 2.9.0 -> 2.9.1
  fix dblp fetcher
  Add missing Turkish translation
  Add "-console" parameter for Windows launcher (#3242)
  Path check converted to if statement
  Changelog updated
  Fixed renaming files which are not in main directory.
  Only use last name for auto completion in search bar. Fixes JabRef#253
  Implemented issue #3229 (#3233)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants