Bug 404593 - misc. minor improvements in production scripts
Summary: misc. minor improvements in production scripts
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Releng (show other bugs)
Version: 4.2.1   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.3 M7   Edit
Assignee: David Williams CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-28 16:09 EDT by David Williams CLA
Modified: 2013-05-30 16:44 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Williams CLA 2013-03-28 16:09:18 EDT
This bug is mostly to document changes I'm making to "clean up" some of the production scripts. 

Its primarily motivated by noticing we are currently getting two copies of "basebuilder" (for the few times we need it). Once in 

{root}/org.eclipse.releng.basebuilder_tagname

and again in 

{root}/sitedir/downloads/drops4/{buildId}/org.eclipse.releng.basebuilder

The latter one was done so specifically so that what ever basebuilder we use 
during the build is the exact same one we use when processing the test results, 
which might not be available for several hours (days?). 

While one approach or the other would likely work, we definitely don't need two, and I prefer the "drop specific" (or "build specific") version. 

This will be more important when we move "off" the base builder, and just install a mini-eclipse, and add tools with p2, etc. 

Plus, back to the "clean up" topic, as I was preparing to make this change, noticed some functions with (currently) unused variables ... small clumps of commented out code, etc.
Comment 2 David Williams CLA 2013-03-29 01:23:40 EDT
With this last commit, I'm done with this for now. This once contains a little more cleanup, some renames, tried to standardize formatting of bash scripts, and ... off topic of this bug (more bug 400633 comment 10)... added a function to test the number of arguments passed to a function matched what was expected (for bullet-proofing). 

http://git.eclipse.org/c/platform/eclipse.platform.releng.aggregator.git/commit/?id=5599fdcf2c85b71e5ccd2a44d0ff50f5861e8471
Comment 3 David Williams CLA 2013-03-29 09:09:23 EDT
Follow up commit to make common 'basebuilder' eclipse executable. 

http://git.eclipse.org/c/platform/eclipse.platform.releng.aggregator.git/commit/?id=f6f21dae6474544eab879f69f720b708f6a4bd22
Comment 4 David Williams CLA 2013-05-30 16:44:26 EDT
mass change to 'verified', as these bugs are either routine or obviously fixed build breaks.