Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added support for external launch4j configuration file #32

Merged
merged 3 commits into from
Apr 11, 2016

Conversation

adamdec
Copy link

@adamdec adamdec commented Mar 15, 2016

Usage:
<configuration> <infile>${project.basedir}/etc/bin/config.xml</infile> </configuration>

By default it will take from ${project.basedir}/src/main/launch4j/${project.artifactId}-launch4j.xml

<infile>${project.basedir}/etc/bin/config.xml</infile>
</configuration>
```
By default it will take from ${project.basedir}/src/main/launch4j/${project.artifactId}-launch4j.xml
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's better to use src/main/resources as files from that location are automatically filtered and there is no need for additional setup to do it and such approach will follow Maven's convention

@lukaszlenart
Copy link
Collaborator

Upss... totally forgot about this PR :(

@lukaszlenart
Copy link
Collaborator

Looks good 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants