[
https://issues.apache.org/jira/browse/LUCENE-3774?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13206378#comment-13206378
]
Dawid Weiss commented on LUCENE-3774:
-------------------------------------
I looked at DependencyChecker, we should rewrite it as an ANT task, really --
it would be cleaner to use (with a dirset or a fileset). I'm thinking it could
look like this:
{code}
<check-license>
<jars>
<fileset dir="base" includes="**/lib/*.jar" />
</jars>
</check-license>
{code}
Base JAR names could be considered a match (or better, a mapper could be added
to specify explicit mapping between the two). The upside of this is that we
could have a single top-level check license task that would simply scan
everything, once and wouldn't need to be updated as libs are added or removed.
I can help with the implementation, this is a trivial task, but I'd appreciate
if you could hook it in ANT scripts where appropriate, Steve. Let me know if
this is a good idea.
> check-legal isn't doing its job
> -------------------------------
>
> Key: LUCENE-3774
> URL: https://issues.apache.org/jira/browse/LUCENE-3774
> Project: Lucene - Java
> Issue Type: Improvement
> Components: general/build
> Affects Versions: 3.6, 4.0
> Reporter: Steven Rowe
> Assignee: Steven Rowe
> Attachments: LUCENE-3774.patch, LUCENE-3774.patch
>
>
> In trunk, the {{check-legal-lucene}} ant target is not checking any
> {{lucene/contrib/\*\*/lib/}} directories; the {{modules/**/lib/}} directories
> are not being checked; and {{check-legal-solr}} can't be checking
> {{solr/example/lib/\*\*/\*.jar}}, because there are currently {{.jar}} files
> in there that don't have a license.
> These targets are set up to take in a full list of {{lib/}} directories in
> which to check, but modules move around, and these lists are not being kept
> up-to-date.
> Instead, {{check-legal-\*}} should run for each module, if the module has a
> {{lib/}} directory, and it should be specialized for modules that have more
> than one ({{solr/core/}}) or that have a {{lib/}} directory in a non-standard
> place ({{lucene/core/}}).
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
