Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 490974 - Can no longer "republish" DL site
Summary: Can no longer "republish" DL site
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Releng (show other bugs)
Version: 4.6   Edit
Hardware: PC Linux
: P3 minor (vote)
Target Milestone: 4.6 M7   Edit
Assignee: David Williams CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-04-04 01:22 EDT by David Williams CLA
Modified: 2016-04-05 14:11 EDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description David Williams CLA 2016-04-04 01:22:15 EDT 
This does not, and should not, occur often, which is why I set to 'minor'. 

In the past, we could invoke "publish-eclipse.sh" with buildproperties.shsource as input, and it would "republish" after corrections made. 

One immediate problem, is that the BUILD_ID in buildproperties.shsource is no longer honored. That is because the scripts were changed to not allow "override", because if ran from Hudson, Hudson has its own BUILD_ID and we do not want it to be the one used in a normal case, so we simply "compute an set it" each time. 

But, for a "republish" that is all wrong. 

Some possible solutions: 

Make the Hudson case smarter, so we know it is from Hudson, and ignore its BUILD_ID in that case. 

Could possibly remember an "OVERRIDE_BUILD_ID" which, if it existed, we would allow it to be set to BUILD_ID.
Comment 1 David Williams CLA 2016-04-05 14:11:44 EDT 
http://git.eclipse.org/c/platform/eclipse.platform.releng.aggregator.git/commit/?id=57a9f0c58c3db995c2122997bceb187413ab0fd9

I had to "republish" Equinox DL site for today's I-build, so took the time to figure out a solution to this. 

Conceptually, if BUILD_ID is already set but does NOT match our typical pattern, then we compute a new one. If it does match our pattern, we go ahead and re-use that value.