You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildr.apache.org by "Alex Boisvert (JIRA)" <ji...@apache.org> on 2010/08/04 00:18:16 UTC

[jira] Created: (BUILDR-488) artifact poms not reinstalled

artifact poms not reinstalled
-----------------------------

                 Key: BUILDR-488
                 URL: https://issues.apache.org/jira/browse/BUILDR-488
             Project: Buildr
          Issue Type: Bug
            Reporter: Alex Boisvert


Currently, artifact poms are never reinstalled if they already exist in local repository, as illustrated by this failing spec,

Index: packaging/artifact_spec.rb
===================================================================
--- packaging/artifact_spec.rb	(revision 966696)
+++ packaging/artifact_spec.rb	(working copy)
@@ -608,6 +608,16 @@
     install artifact(@spec).from(@file)
     lambda { install.invoke }.should change { File.exist?(artifact(@spec).pom.to_s) }.to(true)
   end
+
+  it 'should reinstall POM alongside artifact' do
+    pom = artifact(@spec).pom.to_s
+    write @file
+    write pom
+    sleep 1
+
+    install artifact(@spec).from(@file)
+    lambda { install.invoke }.should change { File.mtime(pom) }
+  end
 end


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (BUILDR-488) artifact poms not reinstalled

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/BUILDR-488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12902226#action_12902226 ] 

Hudson commented on BUILDR-488:
-------------------------------

Integrated in Buildr-ci-build #104 (See [https://hudson.apache.org/hudson/job/Buildr-ci-build/104/])
    

> artifact poms not reinstalled
> -----------------------------
>
>                 Key: BUILDR-488
>                 URL: https://issues.apache.org/jira/browse/BUILDR-488
>             Project: Buildr
>          Issue Type: Bug
>            Reporter: Alex Boisvert
>            Assignee: Alex Boisvert
>
> Currently, artifact poms are never reinstalled if they already exist in local repository, as illustrated by this failing spec,
> Index: packaging/artifact_spec.rb
> ===================================================================
> --- packaging/artifact_spec.rb	(revision 966696)
> +++ packaging/artifact_spec.rb	(working copy)
> @@ -608,6 +608,16 @@
>      install artifact(@spec).from(@file)
>      lambda { install.invoke }.should change { File.exist?(artifact(@spec).pom.to_s) }.to(true)
>    end
> +
> +  it 'should reinstall POM alongside artifact' do
> +    pom = artifact(@spec).pom.to_s
> +    write @file
> +    write pom
> +    sleep 1
> +
> +    install artifact(@spec).from(@file)
> +    lambda { install.invoke }.should change { File.mtime(pom) }
> +  end
>  end

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (BUILDR-488) artifact poms not reinstalled

Posted by "Alex Boisvert (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/BUILDR-488?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alex Boisvert reassigned BUILDR-488:
------------------------------------

    Assignee: Alex Boisvert

> artifact poms not reinstalled
> -----------------------------
>
>                 Key: BUILDR-488
>                 URL: https://issues.apache.org/jira/browse/BUILDR-488
>             Project: Buildr
>          Issue Type: Bug
>            Reporter: Alex Boisvert
>            Assignee: Alex Boisvert
>
> Currently, artifact poms are never reinstalled if they already exist in local repository, as illustrated by this failing spec,
> Index: packaging/artifact_spec.rb
> ===================================================================
> --- packaging/artifact_spec.rb	(revision 966696)
> +++ packaging/artifact_spec.rb	(working copy)
> @@ -608,6 +608,16 @@
>      install artifact(@spec).from(@file)
>      lambda { install.invoke }.should change { File.exist?(artifact(@spec).pom.to_s) }.to(true)
>    end
> +
> +  it 'should reinstall POM alongside artifact' do
> +    pom = artifact(@spec).pom.to_s
> +    write @file
> +    write pom
> +    sleep 1
> +
> +    install artifact(@spec).from(@file)
> +    lambda { install.invoke }.should change { File.mtime(pom) }
> +  end
>  end

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (BUILDR-488) artifact poms not reinstalled

Posted by "Alex Boisvert (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/BUILDR-488?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alex Boisvert resolved BUILDR-488.
----------------------------------

    Resolution: Fixed

Fixed, along with other artifact-related improvements.

boisvert@smudge:~/svn/buildr-pom$ svn commit -m "
BUILDR-488 artifact poms not reinstalled
                     
Also:  it's now possible to provide artifact content with,
        artifact(...).content some_string

Also:  artifact(...).pom.from(file) works as expected.

Also:  poms associated with packages are no longer created directly   
       in the local repository.  they are created alongside packages  
       under 'target' which is more consistent.  the local repository 
       is therefore only written when installing/downloading artifacts
       -- not during packaging.
"
Sending        CHANGELOG
Sending        lib/buildr/core/util.rb
Sending        lib/buildr/packaging/artifact.rb
Sending        lib/buildr/packaging/package.rb
Sending        spec/core/util_spec.rb
Sending        spec/packaging/artifact_spec.rb
Sending        spec/packaging/packaging_spec.rb
Transmitting file data .......
Committed revision 982126.


> artifact poms not reinstalled
> -----------------------------
>
>                 Key: BUILDR-488
>                 URL: https://issues.apache.org/jira/browse/BUILDR-488
>             Project: Buildr
>          Issue Type: Bug
>            Reporter: Alex Boisvert
>            Assignee: Alex Boisvert
>
> Currently, artifact poms are never reinstalled if they already exist in local repository, as illustrated by this failing spec,
> Index: packaging/artifact_spec.rb
> ===================================================================
> --- packaging/artifact_spec.rb	(revision 966696)
> +++ packaging/artifact_spec.rb	(working copy)
> @@ -608,6 +608,16 @@
>      install artifact(@spec).from(@file)
>      lambda { install.invoke }.should change { File.exist?(artifact(@spec).pom.to_s) }.to(true)
>    end
> +
> +  it 'should reinstall POM alongside artifact' do
> +    pom = artifact(@spec).pom.to_s
> +    write @file
> +    write pom
> +    sleep 1
> +
> +    install artifact(@spec).from(@file)
> +    lambda { install.invoke }.should change { File.mtime(pom) }
> +  end
>  end

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.