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

Add filter option #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

DerGuteMoritz
Copy link

This patch adds an option to filter the dependencies to be copied into
the libdir. Possible values are a regular expression which is matched on
the dependency jar file names or a predicate which is applied to the
java.io.File object to check for whether it is to be copied.

The config structure is changed so that all libdir specific options now
reside below the :libdir key in the project defintion. :libdir-path is
kept for backwards compatibility, though.

Moritz Heidkamp added 4 commits January 30, 2013 11:32
This patch adds an option to filter the dependencies to be copied into
the libdir. Possible values are a regular expression which is matched on
the dependency jar file names or a predicate which is applied to the
java.io.File object to check for whether it is to be copied.

The config structure is changed so that all libdir specific options now
reside below the :libdir key in the project defintion. :libdir-path is
kept for backwards compatibility, though.
@djpowell
Copy link
Owner

Hi,

Thanks for the patch. Good idea. Though I was just thinking - it might be better to accept a list of exclusion patterns, to match how :jar-exclusions and :uberjar-exclusions work...

I'll try and have a look at the weekend.

Thanks,
Dave

@DerGuteMoritz
Copy link
Author

Hey Dave,

at least in my case a list of exclusion patterns would not have helped me as I only needed two very specific jars which are then processed specially in the deployment process so I would vote for a whitelist approach or at least leave the option open to specify an arbitrary function so one can choose freely.

Anyway, thanks for the useful plugin!
Moritz

@anthcourtney
Copy link

Any chance of this being resurrected? This would be very useful.

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.

3 participants