Bug 252304 - [build] Build Timestamps are inconsistent
Summary: [build] Build Timestamps are inconsistent
Status: RESOLVED FIXED
Alias: None
Product: AJDT
Classification: Tools
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.6.1   Edit
Assignee: AJDT-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-27 23:43 EDT by Andrew Eisenberg CLA
Modified: 2012-04-03 14:24 EDT (History)
1 user (show)

See Also:


Attachments
patch to update the build process (73.79 KB, patch)
2008-10-27 23:51 EDT, Andrew Eisenberg CLA
andrew.eisenberg: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Eisenberg CLA 2008-10-27 23:43:18 EDT
On some  builds of AJDT, the timestamp of some plugins are different from the timestamp of the update site itself.  This means that these builds cannot be installed because the timestamps need to be identical.

I looked into the reason for this and it seems that the timestamp is not being properly passed into the pde build process, even though cruisecontrol sets it for the update site.  Each feature therefore gets a new timestamp based on the current time that the build started (since it can't find the cruisecontrol timestamp).

I have located this problem and will be submitting a patch soon.
Comment 1 Andrew Eisenberg CLA 2008-10-27 23:51:48 EDT
Created attachment 116250 [details]
patch to update the build process

This patch updates the build process to include the new changes.

Also included in this patch are other changes to the ajdt pde build to make it more consistent with the standard pde build.
Comment 2 Andrew Eisenberg CLA 2008-10-27 23:53:53 EDT
Note that the main thing that had to change to get this to work was to ensure that the forceContextQualifier property was set before starting the build.

Now it is set both in the build.properties and in the build-ajdt1.6.xml ant script on the build machine.
Comment 3 Andrew Clement CLA 2008-10-28 12:09:09 EDT
patch from comment 2 is in.
Comment 4 Andrew Eisenberg CLA 2008-11-18 13:58:10 EST
this problem has been fixed.