Bug 370357 - Add Ant 1.8.3 to build
Summary: Add Ant 1.8.3 to build
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Ant (show other bugs)
Version: 3.8   Edit
Hardware: All All
: P3 enhancement with 1 vote (vote)
Target Milestone: 3.8 M7   Edit
Assignee: Platform-Ant-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on: 370356
Blocks: 346730
  Show dependency tree
 
Reported: 2012-02-01 12:52 EST by DJ Houghton CLA
Modified: 2012-04-30 04:59 EDT (History)
8 users (show)

See Also:


Attachments
Ant 1.8.3 plugin zip (4.47 MB, application/octet-stream)
2012-03-05 05:07 EST, Satyam Kandula CLA
no flags Details
Ant 1.8.3 Eclipse plugin zip (4.47 MB, application/octet-stream)
2012-03-06 03:48 EST, Satyam Kandula CLA
no flags Details
Ant 1.8.3 bundle (6.52 MB, application/octet-stream)
2012-03-07 23:26 EST, Michael Rennie CLA
no flags Details
Ant tests patch (2.56 KB, patch)
2012-03-07 23:26 EST, Michael Rennie CLA
no flags Details | Diff
pre-build Ant binary bundle (2.27 MB, application/octet-stream)
2012-03-12 15:23 EDT, Michael Rennie CLA
no flags Details
pre-built Ant source bundle (2.14 MB, application/octet-stream)
2012-03-12 15:24 EDT, Michael Rennie CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description DJ Houghton CLA 2012-02-01 12:52:46 EST
Once Ant 1.8.3 is released we will put it in Orbit (bug 370356) and then it should be released to the Platform's builds.
Comment 1 Satyam Kandula CLA 2012-03-05 05:07:22 EST
Created attachment 212054 [details]
Ant 1.8.3 plugin zip

I have packaged the new bits into this plugin. There are some files which I don't know if we should take. Here are some of them. 

Mike, Please look at these and let me know your thoughts on this? 

1. From 1.8.3, I just copied bin, etc, lib, LICENSE.TXT and NOTICE.TXT.
I didn't copy manual, fetch.xml, get-m2.xml, INSTALL.txt, KEYS.txt, README.txt and WHATSNEW.txt as the previous versions of this file were not there. 

2. The lib directory has pom files. Do we need to include them? I have removed them.

3. In the directory about_files, I didn't get the files ASL-LICENSE-2.0.txt, DOM-LICENSE.html and SAX-LICENSE.html. 

4. Where to get about.html file from?
Comment 2 Satyam Kandula CLA 2012-03-05 05:42:14 EST
Mike, Can you please look at my questions in comment 1 and the zip and see what else need to be done? Thanks.
Comment 3 Michael Rennie CLA 2012-03-05 10:12:15 EST
(In reply to comment #1) 
> 
> 1. From 1.8.3, I just copied bin, etc, lib, LICENSE.TXT and NOTICE.TXT.
> I didn't copy manual, fetch.xml, get-m2.xml, INSTALL.txt, KEYS.txt, README.txt
> and WHATSNEW.txt as the previous versions of this file were not there. 
> 

This is fine.

> 2. The lib directory has pom files. Do we need to include them? I have removed
> them.

The pom files are used by Apache for building Ant, they can stay removed. 

> 3. In the directory about_files, I didn't get the files ASL-LICENSE-2.0.txt,
> DOM-LICENSE.html and SAX-LICENSE.html. 

You can get them from:

http://apache.org/licenses/LICENSE-2.0 <- this is the ASL-LICENSE.txt
http://www.saxproject.org/copying.html <- this is the SAX 'license'
http://www.w3.org/Consortium/Legal/2002/copyright-software-20021231 <- the DOM license

> 4. Where to get about.html file from?

We created that ourselves and you can just copy it over. Just make sure to update any of the links in it if you change the names of anything in the /about_files directory.
Comment 4 Satyam Kandula CLA 2012-03-06 03:43:50 EST
(In reply to comment #3)
Thanks Mike. 

> > 3. In the directory about_files, I didn't get the files ASL-LICENSE-2.0.txt,
> > DOM-LICENSE.html and SAX-LICENSE.html. 
> 
> You can get them from:
> 
> http://apache.org/licenses/LICENSE-2.0 <- this is the ASL-LICENSE.txt
> http://www.saxproject.org/copying.html <- this is the SAX 'license'
> http://www.w3.org/Consortium/Legal/2002/copyright-software-20021231 <- the DOM
> license
I don't see anything modified. So, leaving them around. 

> > 4. Where to get about.html file from?
> 
> We created that ourselves and you can just copy it over. Just make sure to
> update any of the links in it if you change the names of anything in the
> /about_files directory.
I had to just update the Ant version. Everything looks good.
Comment 5 Satyam Kandula CLA 2012-03-06 03:48:56 EST
Created attachment 212118 [details]
Ant 1.8.3 Eclipse plugin zip

Updated the zip as for comment 3.
Comment 6 Michael Rennie CLA 2012-03-07 23:26:01 EST
Created attachment 212269 [details]
Ant 1.8.3 bundle

Here is a new version of Ant 1.8.3 that can be exported into your host for testing. 

So far all of the JUnit tests pass (with the patch applied I will post here) and smoke testing has not revealed any issues. I have no yet run the performance tests.
Comment 7 Michael Rennie CLA 2012-03-07 23:26:45 EST
Created attachment 212270 [details]
Ant tests patch
Comment 8 Michael Rennie CLA 2012-03-07 23:33:22 EST
The bundle I posted has some shape changes - rather than having to mass copy / paste source into differing dirs per library I updated the classpath + build.properties to build the source bundle with only one source root rather than 15 or so. 

The benefit of this is three-fold: 

1. there are less source roots for the platform to work with when debugging - should make source lookup in the Apache classes a bit faster

2. the bundle can now be built like a 'normal' Eclipse bundle with source

3. when creating new versions of org.apache.ant in the future you only have to grab the source and dump it in the 'libsrc' dir and you are done, rather than the labour-intensive way of doing it now - creating a bunch of dirs per library, copying different source to different dirs, etc, etc
Comment 9 Michael Rennie CLA 2012-03-09 11:42:53 EST
Satyam, Curtis and I are testing the new Ant bundle installed in our host 4.2 installations, can I get you to test it in 3.8? 

To test:

1. import org.apache.ant into your workspace
2. context menu > Export > Deployable plugins and fragments
3. select install into host
4. Options tab > 'Export Source' set to "generate source bundles"
5. apply tests patch
Comment 10 Satyam Kandula CLA 2012-03-12 05:06:40 EDT
(In reply to comment #9)
Thanks Mike for the detailed instructions. I was able to install it and able to run. All tests look good. I didn't see any issues while testing, except for the source code as there is no source attachment. Will it be different in the real installation?
Comment 11 Curtis Windatt CLA 2012-03-12 10:42:07 EDT
(In reply to comment #10)
> (In reply to comment #9)
> Thanks Mike for the detailed instructions. I was able to install it and able to
> run. All tests look good. I didn't see any issues while testing, except for the
> source code as there is no source attachment. Will it be different in the real
> installation?

This will be fine in a real installation.  PDE's install into host action doesn't modify the source features when creating the feature patch.  I opened bug 373967 for this.
Comment 12 Michael Litwak CLA 2012-03-12 15:14:57 EDT
Ant 1.8.3 fixes a bug where JUnitReport fails under Java 7 when Ant is run in embedded environments such as Eclispe:

https://issues.apache.org/bugzilla/show_bug.cgi?id=51668

So I'm hoping the inclusion of Ant 1.8.3 in Eclipse Indigo will be soon.
Comment 13 Michael Rennie CLA 2012-03-12 15:23:19 EDT
Created attachment 212495 [details]
pre-build Ant binary bundle
Comment 14 Michael Rennie CLA 2012-03-12 15:24:18 EDT
Created attachment 212496 [details]
pre-built Ant source bundle
Comment 15 Dani Megert CLA 2012-03-13 02:57:24 EDT
The test pass for M6 is today. We should not risk to destabilize the builds and the drops and consume this with the first M7 I-build.
Comment 16 Dani Megert CLA 2012-03-13 02:58:25 EDT
(In reply to comment #12)
> So I'm hoping the inclusion of Ant 1.8.3 in Eclipse Indigo will be soon.

I guess you meant 'Juno', as there won't be any further Indigo (i.e. 3.7.x) builds.
Comment 17 Michael Rennie CLA 2012-03-13 09:56:27 EDT
(In reply to comment #15)
> The test pass for M6 is today. We should not risk to destabilize the builds and
> the drops and consume this with the first M7 I-build.

I agree completely, I just did not get around to moving the milestone yet.
Comment 19 Kim Moir CLA 2012-03-19 19:41:40 EDT
test build successful, released to integration
Comment 20 Michael Rennie CLA 2012-03-20 11:28:05 EDT
(In reply to comment #19)
> test build successful, released to integration

Except for the Ant tests patch, which I pushed here:

http://git.eclipse.org/c/platform/eclipse.platform.git/commit/?id=b3981a7fcd79832097b50206a1544b7a29d41363
Comment 21 Michael Rennie CLA 2012-03-21 10:39:07 EDT
(In reply to comment #20)
> (In reply to comment #19)
> > test build successful, released to integration
> 
> Except for the Ant tests patch, which I pushed here:
> 
> http://git.eclipse.org/c/platform/eclipse.platform.git/commit/?id=b3981a7fcd79832097b50206a1544b7a29d41363

One more failure in the I/N-build (I20120320-1400):

testAntHome	

Failure	Incorrect message. Should end with org.apache.ant. Message: [echo] C:\buildtest\N20120320-2000\eclipse-testing\test-eclipse\eclipse\plugins\org.apache.ant_1.8.3.v20120316-1400
junit.framework.AssertionFailedError: Incorrect message. Should end with org.apache.ant. Message: [echo] C:\buildtest\N20120320-2000\eclipse-testing\test-eclipse\eclipse\plugins\org.apache.ant_1.8.3.v20120316-1400
at org.eclipse.ant.tests.ui.separateVM.SeparateVMTests.testAntHome(SeparateVMTests.java:223)
at org.eclipse.ant.tests.ui.AbstractAntUIBuildTest.access$0(AbstractAntUIBuildTest.java:1)
at org.eclipse.ant.tests.ui.AbstractAntUIBuildTest$1.run(AbstractAntUIBuildTest.java:44)
at java.lang.Thread.run(Thread.java:722)
Comment 22 Michael Rennie CLA 2012-03-21 14:52:47 EDT
(In reply to comment #21)

> One more failure in the I/N-build (I20120320-1400):
> 
> testAntHome    

fixed in:

http://git.eclipse.org/c/platform/eclipse.platform.git/commit/?id=8eec0a6e1a22bff56b6e457a756291246ab1e321
Comment 23 Michael Rennie CLA 2012-03-28 13:08:16 EDT
Marking as fixed, Ant 1.8.3 is in builds > 21/03/2012 and no issues have been found so far.

Any problems should be opened in their own bugs.
Comment 24 Dani Megert CLA 2012-04-30 04:59:51 EDT
Verified in 3.8-I20120429-2000 and 4.2-I20120429-1800.