You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@buildr.apache.org by Alexis Midon <al...@gmail.com> on 2008/01/31 20:31:38 UTC

Bug in trunk: incomplete test.compile classpath

Hi guys,

with the latest trunk, the test.compile classpath does NOT include the
compile classpath dependencies.
This is cause by an misplaced 'if' statement in core/test.rb

Here is a fix:

Index: core/test.rb
===================================================================
--- core/test.rb        (revision 617186)
+++ core/test.rb        (working copy)
@@ -538,7 +538,8 @@
     after_define do |project|
       test = project.test
       # Dependency on compiled code, its dependencies and resources.
-      test.with project.compile.dependencies, Array(project.compile.target)
if project.compile.target
+      test.with project.compile.dependencies
+      test.with Array(project.compile.target) if project.compile.target
       test.with Array(project.resources.target)
       # Dependency on compiled tests and resources.  Dependencies added
using with.
       test.dependencies.concat Array(test.compile.target) if
test.compile.target

Re: Bug in trunk: incomplete test.compile classpath

Posted by Alexis Midon <al...@gmail.com>.
I created a jira issue, and attached my fix.

https://issues.apache.org/jira/browse/BUILDR-33



On Jan 31, 2008 11:31 AM, Alexis Midon <al...@gmail.com> wrote:

>
> Hi guys,
>
> with the latest trunk, the test.compile classpath does NOT include the
> compile classpath dependencies.
> This is cause by an misplaced 'if' statement in core/test.rb
>
> Here is a fix:
>
> Index: core/test.rb
> ===================================================================
> --- core/test.rb        (revision 617186)
> +++ core/test.rb        (working copy)
> @@ -538,7 +538,8 @@
>      after_define do |project|
>        test = project.test
>        # Dependency on compiled code, its dependencies and resources.
> -      test.with project.compile.dependencies, Array(
> project.compile.target) if project.compile.target
> +      test.with project.compile.dependencies
> +      test.with Array(project.compile.target) if project.compile.target
>        test.with Array(project.resources.target)
>        # Dependency on compiled tests and resources.  Dependencies added
> using with.
>        test.dependencies.concat Array(test.compile.target) if
> test.compile.target
>
>