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

Stefan Miklosovic edited comment on CASSANDRA-18603 at 6/23/23 9:37 AM:
------------------------------------------------------------------------

Should not ant clean remove checkstyle cache?

Currently, ant clean does this:
{code:java}
$ ant clean
Buildfile: 
/home/fermat/dev/cassandra/cassandra-instaclustr/cassandra/build.xmlclean:
   [delete] Deleting directory 
/home/fermat/dev/cassandra/cassandra-instaclustr/cassandra/build/test
   [delete] Deleting directory 
/home/fermat/dev/cassandra/cassandra-instaclustr/cassandra/build/classes
   [delete] Deleting directory 
/home/fermat/dev/cassandra/cassandra-instaclustr/cassandra/src/gen-java
   [delete] Deleting directory 
/home/fermat/dev/cassandra/cassandra-instaclustr/cassandra/src/resources/org/apache/cassandra/config
   [delete] Deleting directory 
/home/fermat/dev/cassandra/cassandra-instaclustr/cassandra/build/jacoco
 {code}
So if we clean classes, we have orphaned checkstyle, no?

I am aware this is not directly related to this ticked but as we are on this, 
could not we fix that too? There is no reason to do yet another ticket for such 
triviality.


was (Author: smiklosovic):
Should not ant clean remove checkstyle cache?

Currently, ant clean does this:
{code:java}
$ ant clean
Buildfile: 
/home/fermat/dev/cassandra/cassandra-instaclustr/cassandra/build.xmlclean:
   [delete] Deleting directory 
/home/fermat/dev/cassandra/cassandra-instaclustr/cassandra/build/test
   [delete] Deleting directory 
/home/fermat/dev/cassandra/cassandra-instaclustr/cassandra/build/classes
   [delete] Deleting directory 
/home/fermat/dev/cassandra/cassandra-instaclustr/cassandra/src/gen-java
   [delete] Deleting directory 
/home/fermat/dev/cassandra/cassandra-instaclustr/cassandra/src/resources/org/apache/cassandra/config
   [delete] Deleting directory 
/home/fermat/dev/cassandra/cassandra-instaclustr/cassandra/build/jacoco
 {code}
So if we clean classes, we have orphaned checkstyle, no?

> Move checkstyle files into .build
> ---------------------------------
>
>                 Key: CASSANDRA-18603
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-18603
>             Project: Cassandra
>          Issue Type: Task
>          Components: Build
>            Reporter: Berenguer Blasi
>            Assignee: Berenguer Blasi
>            Priority: Normal
>             Fix For: 4.1.x, 5.x
>
>
> Checkstyle files are currently in the root folder whereas the rest of similar 
> purposed files live under .build. Let's move them for consistency.
> CC [~mck]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to