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

ASF GitHub Bot commented on SQOOP-3415:
---------------------------------------

Github user maugly24 commented on a diff in the pull request:

    https://github.com/apache/sqoop/pull/62#discussion_r237962514
  
    --- Diff: build.gradle ---
    @@ -356,6 +359,15 @@ tasks.withType(Test) {
         ignoreFailures ignoreTestFailures
     }
     
    +project.tasks.each {
    +  if ( it.name.toLowerCase().endsWith('test') ) {
    +    it.doFirst({
    --- End diff --
    
    Hi Szabi,
    
    Thanks for the suggestion!
    
    I've modified the build.gradle file accordingly!


> Fix gradle test+build when clean applied as the first command + warning issue 
> fixes
> -----------------------------------------------------------------------------------
>
>                 Key: SQOOP-3415
>                 URL: https://issues.apache.org/jira/browse/SQOOP-3415
>             Project: Sqoop
>          Issue Type: Bug
>    Affects Versions: 1.5.0
>            Reporter: Attila Szabo
>            Assignee: Attila Szabo
>            Priority: Major
>             Fix For: 1.5.0
>
>
> If the user wants to build like the following command:
> gradlew clean unittest
> the gradle process ends up in an exception and the whole process left there 
> hanging forever. The root cause of this is the following:
> tasks.withType runs in the configuration part of the build, where we ensure 
> the neccessary directories exist.
> after that clean is executed and all of the dirs got deleted.
> Proposed fix:
> Apply directory creation as the first step of test tasks.
> on the top:
> there are some missing options b/c of Junit annotation processors, and also 
> Xlint information are swallowed currently. We aim to fix these things as well



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to