Bob notes that dist-dev doesn't need (and thus shouldn't have) its
dependency on tools. I take that as tacit agreement that dist-dev can be
the "minimally build everything for test execution" name.
On Mon, Jun 15, 2009 at 11:53 AM, Freeland Abbott <[email protected]>wrote:
> Bob, John, can you review? This doesn't address the "buildonly or dist-dev
> as precursor to user/test" question, but I think addresses Thomas' and Bob's
> other issue.
>
>
>
>
> On Mon, Jun 15, 2009 at 11:47 AM, Freeland Abbott <[email protected]>wrote:
>
>> +GWTC
>>
>> With regard to the Mac looking for gwt-dev-linux, since I don't literally
>> see a build-dev target, what's the chain above the line you cited? I
>> thought my fixes to <property.ensure> in e.g. user/build.xml line 47
>> addressed that, and I see I missed some (in doc, samples, and soyc-vis...
>> only the last of which is likely to be stopping you, though).
>>
>>
>> With regard to using buildonly as precursor to tests, I think you want to
>> the target dist-dev, though I'm open to suggestions for how this "should"
>> work. It used to be that:
>>
>> 1. we had both "build" and "dist," as separate targets (unchanged),
>> 2. tests effectively require a distribution to run in (unchanged),
>> 3. we had "buildonly," but did not have "dist-dev," so the two were
>> blurred explicitly (changed),
>> 4. "build" depended, by way of "-do," on "dist," which I found
>> confusingly backwards (changed),
>> 5. top-level "test" depended on "build", which depended on "dist" as
>> above (changed to depend on "dist-one," but should probably be dist-dev
>> instead now that it exists),
>> 6. "build" was the default target, which as above worked because it
>> ran "dist" as a dependency (changed; "dist" is project default target, and
>> depends on build).
>>
>> Solutions:
>>
>> 1. Do we want buildonly to imply dist-dev? Do we want build to imply
>> dist? My intuition is no, that if separate, build builds and dist
>> packages
>> and dist needs build (but not vice-versa). That is semantic a change from
>> before, though.
>> 2. If not, then we want to retrain ourselves to use dist-dev at least
>> once, to get the distro set, even if you then use buildonly after that to
>> refresh sources?
>>
>>
>>
>>
>> On Mon, Jun 15, 2009 at 11:10 AM, Freeland Abbott <[email protected]>wrote:
>>
>>> Thanks, I'll revisit.
>>>
>>>
>>>
>>> On Mon, Jun 15, 2009 at 10:43 AM, BobV <[email protected]> wrote:
>>>
>>>>
>>>> This no longer correctly works on mac platform.
>>>>
>>>> ant buildonly && cd user && ant test
>>>> /Users/bob/gwt/trunk/common.ant.xml:299: GWT must be built before
>>>> performing any tests. This can be fixed by running ant in the
>>>> /Users/bob/gwt/trunk directory.
>>>>
>>>>
>>>> also, the ant build-dev target fails:
>>>> /Users/bob/gwt/trunk/common.ant.xml:299: Cannot find dependency
>>>> /Users/bob/gwt/trunk/build/lib/gwt-dev-linux.jar
>>>>
>>>>
>>>> --
>>>> Bob Vawter
>>>> Google Web Toolkit Team
>>>>
>>>
>>>
>>
>
--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---
Index: build-tools/ant-gwt/src/com/google/gwt/ant/taskdefs/LatestTimeJar.java
===================================================================
--- build-tools/ant-gwt/src/com/google/gwt/ant/taskdefs/LatestTimeJar.java (revision 5558)
+++ build-tools/ant-gwt/src/com/google/gwt/ant/taskdefs/LatestTimeJar.java (working copy)
@@ -116,8 +116,12 @@
@Override
public void addToZip(ZipOutputStream out, String path) throws IOException {
FileInputStream inStream = new FileInputStream(tmpFile);
- doZipFile(inStream, out, path, timestamp, archive, mode);
- tmpFile.delete();
+ try {
+ doZipFile(inStream, out, path, timestamp, archive, mode);
+ tmpFile.delete();
+ } finally {
+ inStream.close();
+ }
}
}
Index: build.xml
===================================================================
--- build.xml (revision 5558)
+++ build.xml (working copy)
@@ -35,7 +35,7 @@
<gwt.ant dir="distro-source" target="${build.host.platform}" />
</target>
- <target name="dist-dev" depends="buildonly, tools" description="Make this platform's distribution, minus doc and samples">
+ <target name="dist-dev" depends="buildonly" description="Make this platform's distribution, minus doc and samples">
<gwt.ant dir="distro-source" target="${build.host.platform}" />
</target>
@@ -93,7 +93,7 @@
<call-subproject subproject="samples" subtarget="checkstyle" />
</target>
- <target name="test" depends="dist-one" description="Runs all the GWT tests">
+ <target name="test" depends="dist-dev" description="Runs all the GWT tests">
<call-subproject subproject="buildtools" subtarget="test" />
<call-subproject subproject="dev" subtarget="test" />
<call-subproject subproject="user" subtarget="test" />
Index: tools/api-checker/build.xml
===================================================================
--- tools/api-checker/build.xml (revision 5558)
+++ tools/api-checker/build.xml (working copy)
@@ -7,7 +7,7 @@
<property name="tools.build" value="${gwt.build.out}/${project.tail}" />
<!-- Platform shouldn't matter here, just picking one -->
- <property.ensure name="gwt.dev.jar" location="${gwt.build.lib}/gwt-dev-linux.jar" />
+ <property.ensure name="gwt.dev.jar" location="${gwt.build.lib}/gwt-dev-${build.host.platform}.jar" />
<property.ensure name="gwt.user.jar" location="${gwt.build.lib}/gwt-user.jar" />
<target name="compile" description="Compile all class files">
Index: tools/soyc-vis/build.xml
===================================================================
--- tools/soyc-vis/build.xml (revision 5558)
+++ tools/soyc-vis/build.xml (working copy)
@@ -7,7 +7,7 @@
<import file="${gwt.root}/common.ant.xml" />
<!-- Platform shouldn't matter here, just picking one -->
- <property.ensure name="gwt.dev.jar" location="${gwt.build.lib}/gwt-dev-linux.jar" />
+ <property.ensure name="gwt.dev.jar" location="${gwt.build.lib}/gwt-dev-${build.host.platform}.jar" />
<target name="clean">
<delete dir="build"/>
@@ -41,4 +41,4 @@
<target name="checkstyle"/>
-</project>
\ No newline at end of file
+</project>
Index: samples/common.ant.xml
===================================================================
--- samples/common.ant.xml (revision 5558)
+++ samples/common.ant.xml (working copy)
@@ -16,7 +16,7 @@
<property.ensure name="gwt.user.jar" location="${gwt.build.lib}/gwt-user.jar" />
<!-- Platform shouldn't matter here, just picking one -->
- <property.ensure name="gwt.dev.jar" location="${gwt.build.lib}/gwt-dev-linux.jar" />
+ <property.ensure name="gwt.dev.jar" location="${gwt.build.lib}/gwt-dev-${build.host.platform}.jar" />
<!-- Mirror directory for scripts; makes building distro easier -->
<property name="samples.scripts" value="${gwt.build.out}/samples-scripts" />
Index: doc/build.xml
===================================================================
--- doc/build.xml (revision 5558)
+++ doc/build.xml (working copy)
@@ -7,7 +7,7 @@
<property.ensure name="gwt.user.jar" location="${gwt.build.lib}/gwt-user.jar" />
<!-- Platform shouldn't matter here, just picking one -->
- <property.ensure name="gwt.dev.jar" location="${gwt.build.lib}/gwt-dev-linux.jar" />
+ <property.ensure name="gwt.dev.jar" location="${gwt.build.lib}/gwt-dev-${build.host.platform}.jar" />
<property name="USER_PKGS"
value="com.google.gwt.animation.client;com.google.gwt.benchmarks.client;com.google.gwt.core.client;com.google.gwt.core.ext;com.google.gwt.core.ext.soyc;com.google.gwt.core.ext.linker;com.google.gwt.core.ext.typeinfo;com.google.gwt.debug.client;com.google.gwt.dom.client;com.google.gwt.event.dom.client;com.google.gwt.event.logical.shared;com.google.gwt.event.shared;com.google.gwt.http.client;com.google.gwt.i18n.client;com.google.gwt.i18n.client.constants;com.google.gwt.i18n.rebind.format;com.google.gwt.i18n.rebind.keygen;com.google.gwt.json.client;com.google.gwt.junit.client;com.google.gwt.benchmarks.client;com.google.gwt.user.client;com.google.gwt.user.client.rpc;com.google.gwt.user.client.ui;com.google.gwt.user.datepicker.client;com.google.gwt.user.server.rpc;com.google.gwt.xml.client"/>
