You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildr.apache.org by "Peter Donald (JIRA)" <ji...@apache.org> on 2012/11/15 07:22:14 UTC

[jira] [Closed] (BUILDR-611) Buildr should not unnecessarily recompile Java files explicitly added to compile.from

     [ https://issues.apache.org/jira/browse/BUILDR-611?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Peter Donald closed BUILDR-611.
-------------------------------

    Resolution: Fixed
    
> 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: Peter Donald
>             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.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira