You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildr.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2011/09/14 18:02:12 UTC
[jira] [Commented] (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: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