[ https://issues.apache.org/jira/browse/LUCENE-3774?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13206739#comment-13206739 ]
Dawid Weiss commented on LUCENE-3774: ------------------------------------- I assumed modules doesn't need to be self-contained like Lucene or Solr. I can fix that by enforcing tools' compilation in that macro... I'll do that. This reminds me of the "infrastructure tools" problem we had in Carrot2. We finally decided to simply have them as a stand-alone project living within the same repository space, with a stored, versioned binary artefact updated when tools had to be updated (rarely). This does version a binary file but you don't need to worry about recompiling things over and over. > 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: 3774.patch, LUCENE-3774.patch, LUCENE-3774.patch, > LUCENE-3774.patch, LUCENE-3774.patch, LUCENE-3774.patch, LUCENE3774.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: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org