Closed Bug 579504 Opened 14 years ago Closed 12 years ago

remove CC from linux cltbld .bash_profile

Categories

(Release Engineering :: General, defect, P5)

All
Linux
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Assigned: bhearsum)

Details

(Whiteboard: [buildslaves][puppet][simple])

and CXX; these are interfering with things like Android builds, where we explicitly do not want a host compiler set as the default.

We already override this on linux Firefox builds via mozconfig.

We probably need CVS_RSH, and TBOX_CLIENT_CVS_DIR anymore.
Not sure if we need JAVA_HOME, though it seems to be causing issues in Android land (we have a workaround atm).
Nobody has volunteered to take this so I'm marking it P5. Please feel free to grab this if you want to work on this
Priority: -- → P5
Whiteboard: [buildslaves][puppet] → [buildslaves][puppet][simple]
Assignee: nobody → bhearsum
For CC/CXX, found these:
[root@mpt-production-puppet N]# find . -iregex .*centos.*bash_profile.*
./production/centos5-i686/stage/home/cltbld/.bash_profile
./production/centos5-i686/build/home/cltbld/.bash_profile
./production/centos5-x86_64/build/home/cltbld/.bash_profile
./production/centos5-x86_64/stage/home/cltbld/.bash_profile
./staging/centos5-x86_64/stage/home/cltbld/.bash_profile
./staging/centos5-x86_64/build/home/cltbld/.bash_profile
./staging/centos5-i686/stage/home/cltbld/.bash_profile
./staging/centos5-i686/build/home/cltbld/.bash_profile

I edited each "build" one by hand, and removed CC, CXX, TBOX_CLIENT_CVS_DIR, and JAVA_HOME.

CVS_RSH is needed for patcher-config and a few other things we do in releases.

I also made sure the changes were synced to the mv, scl1, and scl3 puppet masters.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.