buildr-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (BUILDR-611) Buildr should not unnecessarily recompile Java files explicitly added to compile.from
Date Wed, 14 Sep 2011 16:02:12 GMT

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

Hudson commented on BUILDR-611:
-------------------------------

Integrated in Buildr-ci-build #275 (See [https://builds.apache.org/job/Buildr-ci-build/275/])
    BUILDR-611 Buildr should not unnecessarily recompile Java files explicitly added to compile.from

boisvert : 
Files : 
* /buildr/trunk/CHANGELOG
* /buildr/trunk/lib/buildr/core/compile.rb
* /buildr/trunk/spec/core/compile_spec.rb


> Buildr should not unnecessarily recompile Java files explicitly added to compile.from
> -------------------------------------------------------------------------------------
>
>                 Key: BUILDR-611
>                 URL: https://issues.apache.org/jira/browse/BUILDR-611
>             Project: Buildr
>          Issue Type: Bug
>          Components: Compilers
>    Affects Versions: 1.4.6
>            Reporter: FĂ©lix Enrique Llorente Pastora
>            Assignee: Alex Boisvert
>             Fix For: 1.4.7
>
>
> We have something like that in our buildfile
> compile.from(FileList[ 
>                               FooI/jsrc/**/*.java',
>                               Bar/jsrc/']
>                        .exclude('FooJava.java'))
> Calling build task twice do two compilations.
> buildr doesn't remove the ";" character from the package name in the compile_map 
> We did have to use the following patch to fix it:
> --- lib/buildr/core/compile.rb	2011-09-14 11:55:23.377091000 +0200
> +++ lib/buildr/core/compile.rb.org	2011-09-14 11:56:08.619013000 +0200
> @@ -193,7 +193,7 @@
>              # try to extract package name from .java or .scala files
>              if ['.java', '.scala', '.groovy'].include? File.extname(source)
>                package = findFirst(source, /^\s*package\s+(\S+)\s*;?\s*$/)
> -              map[source] = package ? File.join(target, package[1].gsub('.', '/').gsub(';',''),
File.basename(source).ext(target_ext)) : target
> +              map[source] = package ? File.join(target, package[1].gsub('.', '/'), File.basename(source).ext(target_ext))
: target
>              elsif
>                map[source] = target
>              end

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

Mime
View raw message