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

Hoss Man commented on LUCENE-5831:
----------------------------------

maybe instead of modifying the ant deps, we should instead update the error 
message produced by precommit when it isn't happy with the jar checksums to 
help the user sanity check themselves

something like...

{noformat}
Missing checksum for foo.jar

If you recently modified any ivy.xml or ivy.properties files to add new jars, 
make sure you run "ant clean-jars jar-checksums" before running precommit.
{noformat}

> ant precommit should depend on clean-jars
> -----------------------------------------
>
>                 Key: LUCENE-5831
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5831
>             Project: Lucene - Core
>          Issue Type: Bug
>          Components: general/build
>            Reporter: Steve Rowe
>            Priority: Minor
>
> Ivy's bug that fails to remove differently versioned dependencies in 
> (test-)lib/ dirs even though we set {{sync="true"}} on {{<ivy:retrieve>}} (I 
> couldn't find a JIRA for this) continues to cause trouble/confusion (see 
> related LUCENE-5467).
> We should make the ant {{precommit}} target depend on {{clean-jars}}, so that 
> people won't think they need to run {{ant jar-checksums}} because of stale 
> jars Ivy leaves in {{lib/}} or {{test-lib/}} directories, which currently 
> causes {{ant precommit}} to bitch that there are missing checksums.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to