Improvements to buildit

Details

  • Type: Improvement Improvement
  • Status: Reopened Reopened
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: 3.1.3
  • Component/s: Infrastructure: Build
  • Labels:
    None
  • Patch Submitted:
    Yes

Description

1. Filip Pizlo passed on the attached patch which improves ssh handling of the buildit script. Fil, can you please do a "statement of origin" so we can formally contribute it? (http://jikesrvm.org/Contributions)

2. I would also like to switch the -p switch from being perfcounter (which I use, but is esoteric and not used by many), to profile, which everyone should be using for performance studies. Please yell if you don't want me to do this.

3. I have started writing some notes on the wiki on how to use buildit, with a handful of examples.

  1. Text File
    buildit.patch
    06/Apr/08 5:01 PM
    15 kB
    Filip Pizlo
  2. Text File
    buildit.patch
    04/Apr/08 11:57 PM
    13 kB
    Steve Blackburn
  3. Text File
    buildit.patch.soo.txt
    05/Apr/08 12:27 AM
    0.5 kB
    Filip Pizlo
  4. Text File
    purdue_building.patch
    14/Feb/09 1:55 PM
    14 kB
    Filip Pizlo

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Filip Pizlo added a comment -

statement of origin

Show
Filip Pizlo added a comment - statement of origin
Hide
Permalink
Filip Pizlo added a comment -

statement of origin

Show
Filip Pizlo added a comment - statement of origin
Hide
Permalink
Filip Pizlo added a comment -

updated buildit patch

Show
Filip Pizlo added a comment - updated buildit patch
Hide
Permalink
Filip Pizlo added a comment -

Added an updated patch that adds the ability to have buildit source an initialization script on the remote machine. Useful for running on machines where JAVA_HOME is not the only environment variable that must be set before a JVM, or Ant, can be run.

Show
Filip Pizlo added a comment - Added an updated patch that adds the ability to have buildit source an initialization script on the remote machine. Useful for running on machines where JAVA_HOME is not the only environment variable that must be set before a JVM, or Ant, can be run.
Hide
Permalink
Steve Blackburn added a comment -

Fixed in 14108 ("-p" fix), 14109 (Filip Pizlo's patch), and some documentation added to the wiki

Show
Steve Blackburn added a comment - Fixed in 14108 ("-p" fix), 14109 (Filip Pizlo's patch), and some documentation added to the wiki
Hide
Permalink
David Grove added a comment -

This change completely broke buildit for me.

[dgrove@linchen rvm-trunk]$ ./bin/buildit --processors 2 lydian --test-run pre-commit
Usage: buildit [-dhgqpz] [-r root] [-s suffix] [-j java-home]
[--test-run xxx] build-host [-c target-host] [ Configs... ]

ERROR: Could not find configuration '0'

I suspect it has something to do with the use of flex_chomp instead of chomp, but just backing that piece out wasn't good enough.

I'm running perl 5.8.8 on RHEL 5.1

Show
David Grove added a comment - This change completely broke buildit for me. [dgrove@linchen rvm-trunk]$ ./bin/buildit --processors 2 lydian --test-run pre-commit Usage: buildit [-dhgqpz] [-r root] [-s suffix] [-j java-home] [--test-run xxx] build-host [-c target-host] [ Configs... ] ERROR: Could not find configuration '0' I suspect it has something to do with the use of flex_chomp instead of chomp, but just backing that piece out wasn't good enough. I'm running perl 5.8.8 on RHEL 5.1
Hide
Permalink
David Grove added a comment -

Sorry, by this change I mean 14109. If I back that out, everything is fine.

Show
David Grove added a comment - Sorry, by this change I mean 14109. If I back that out, everything is fine.
Hide
Permalink
Steve Blackburn added a comment -

OK. Sorry about that. I have backed out, and will explore it a little further today.

Show
Steve Blackburn added a comment - OK. Sorry about that. I have backed out, and will explore it a little further today.
Hide
Permalink
David Grove added a comment -

closed by mistake during bulk operations.

Show
David Grove added a comment - closed by mistake during bulk operations.
Hide
Permalink
Jan Sinschek added a comment -

The script uses its default value for the JAVA_HOME even when a system variable is available that differs, so it will not work if the default doesn't.

Show
Jan Sinschek added a comment - The script uses its default value for the JAVA_HOME even when a system variable is available that differs, so it will not work if the default doesn't.
Hide
Permalink
Ian Rogers added a comment -

I believe this is a problem with GNU Classpath

Show
Ian Rogers added a comment - I believe this is a problem with GNU Classpath
Hide
Permalink
Filip Pizlo added a comment -

New patch for the latest buildit (as of r15439)

Show
Filip Pizlo added a comment - New patch for the latest buildit (as of r15439)
Hide
Permalink
David Grove added a comment -

Defer to 3.1.1

Show
David Grove added a comment - Defer to 3.1.1
Hide
Permalink
David Grove added a comment -

bulk defer open issues to 3.1.2

Show
David Grove added a comment - bulk defer open issues to 3.1.2

People

  • Assignee:
    Unassigned
    Reporter:
    Steve Blackburn
Vote (0)
Watch (0)

Dates

  • Created:
    Updated: