Closed Bug 386992 Opened 14 years ago Closed 13 years ago

Sisyphus - sync misc changes

Categories

(Testing Graveyard :: Sisyphus, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bc, Assigned: bc)

References

Details

Attachments

(6 files)

Attached patch sync.patchSplinter Review
This bug will track the changes to Sisyphus required to get buildbot driven js test automation working. This first patch syncs the tree with my working copy.
Attached file checkin log
Attached patch patch 2Splinter Review
remove arch from mac OSID, 
change signal reporting in timed_run.py to test for signals before the exit code, remove gnu regexp extensions due to mac sed limitations. 

robcee: I am thinking I am liking the idea of not only building bash, but also several of the base gnu utilities into /usr/local/bin so I don't have to worry about the non-gnuness of mac. Thoughts?

sisyphus/bin/build.sh,v  <--  build.sh
new revision: 1.3; previous revision: 1.2

sisyphus/bin/edit-talkback.sh,v  <--  edit-talkback.sh
new revision: 1.3; previous revision: 1.2

sisyphus/bin/install-build.sh,v  <--  install-build.sh
new revision: 1.3; previous revision: 1.2

sisyphus/bin/library.sh,v  <--  library.sh
new revision: 1.3; previous revision: 1.2

sisyphus/bin/test-setup.sh,v  <--  test-setup.sh
new revision: 1.3; previous revision: 1.2

sisyphus/bin/tester.sh,v  <--  tester.sh
new revision: 1.3; previous revision: 1.2

sisyphus/bin/timed_run.py,v  <--  timed_run.py
new revision: 1.2; previous revision: 1.1
I think that's a bit of an extreme approach when something as simple as ports exists. You could include the versions you expect and the commands to install them in a script, but building from scratch yourself seems like overkill.
Depends on: 396028
morphing this bug to be able getting the cvs version in sync with mine. We can file additional bugs for new issues. I'll file bugs on any new issues which arise and block bug 396028.
Summary: Sisyphus - support buildbot js test automation → Sisyphus - sync misc changes
Attached patch sync-w.patchSplinter Review
diff -w -p -U 10 patch for readability.

cleans up indenting using emacs shell mode with Sh Indent Comment set to 't' to indent comments as normal lines.

replaces hard coded paths beginning with </work/mozilla/mozilla.com/test.mozilla.com/www> with environment variables.

update scripts to use consistent usage exit code of 1.

bin/checkout.sh updated to use eval cvs ... in order to pass optional command line parameters related to branch|date pulls.

bin/build.sh updated to add ability to force a clobber and rebuild if the build fails and TEST_FORCE_CLOBBER_ON_ERROR is defined.

bin/clean.sh updated to use clean instead of distclean as a target.

bin/library.sh updated to allow override of default XPCOM_DEBUG_BREAK=warn environment variable, and to (temporarily) disable crash reporter.

updates data/* files with current paths to builds on ftp.mozilla.org.
adds data/ files for too-much-gc, jprof builds.
Attached patch sync.patchSplinter Review
full version of patch.
Attached file commit.log
-> fixed.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Blocks: 396028
No longer depends on: 396028
Mass move of Sisyphus bugs to Testing : Sisyphus. Filter on SisyphusMassMove to ignore.
Component: Testing → Sisyphus
Product: Core → Testing
QA Contact: testing → sisyphus
Version: unspecified → Trunk
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.