You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "Tomoko Uchida (Jira)" <ji...@apache.org> on 2020/02/18 16:06:00 UTC

[jira] [Commented] (LUCENE-9219) Port ECJ-based linter to gradle

    [ https://issues.apache.org/jira/browse/LUCENE-9219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17039190#comment-17039190 ] 

Tomoko Uchida commented on LUCENE-9219:
---------------------------------------

Hi [~dweiss],

I attached [^LUCENE-9219.patch], a modified version of the patch in LUCENE-9201. Could you take a look at this?
 (diffs from LUCENE-9201-ecj-2.patch)
 - use sourceSet.getTaskName()
 - task group was changed to "Verification"
 - the artifact version of "org.eclipse.jdt:ecj" was moved to "build.gradle" from "ecj-lint.gradle"

Also I excluded the changes for "lucene/common-build.xml", because I thought this was added to the patch accidentally. Please let me know if this should be included again.
{code:java}
diff --git a/lucene/common-build.xml b/lucene/common-build.xml
index 1e3da88250b..ca5887df550 100644
--- a/lucene/common-build.xml
+++ b/lucene/common-build.xml
@@ -2034,6 +2034,7 @@ ${ant.project.name}.test.dependencies=${test.classpath.list}
   </target>
 
   <target name="-ecj-javadoc-lint" depends="-ecj-javadoc-lint-unsupported,-ecj-javadoc-lint-src,-ecj-javadoc-lint-tests"/>
+  <target name="ecj" depends="-ecj-javadoc-lint" />
{code}

> Port ECJ-based linter to gradle
> -------------------------------
>
>                 Key: LUCENE-9219
>                 URL: https://issues.apache.org/jira/browse/LUCENE-9219
>             Project: Lucene - Core
>          Issue Type: Sub-task
>            Reporter: Dawid Weiss
>            Priority: Major
>         Attachments: LUCENE-9219.patch, LUCENE-9219.patch
>
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org