Details

    • Type: Improvement Improvement
    • Status: Reopened Reopened
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 3.1.4
    • Component/s: Infrastructure: Build
    • Labels:
      None
    • Patch Submitted:
      Yes
    • Number of attachments :
      4

      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. buildit.patch
        15 kB
        Filip Pizlo
      2. buildit.patch
        13 kB
        Steve Blackburn
      3. buildit.patch.soo.txt
        0.5 kB
        Filip Pizlo
      4. purdue_building.patch
        14 kB
        Filip Pizlo

        Activity

        Hide
        Filip Pizlo added a comment -

        statement of origin

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

        statement of origin

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

        updated buildit patch

        Show
        Filip Pizlo added a comment - updated buildit patch
        Hide
        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
        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
        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
        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
        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
        David Grove added a comment -

        closed by mistake during bulk operations.

        Show
        David Grove added a comment - closed by mistake during bulk operations.
        Hide
        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
        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
        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
        David Grove added a comment -

        Defer to 3.1.1

        Show
        David Grove added a comment - Defer to 3.1.1
        Hide
        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
        Hide
        David Grove added a comment -

        bulk defer issues to 3.1.4

        Show
        David Grove added a comment - bulk defer issues to 3.1.4

          People

          • Assignee:
            Unassigned
            Reporter:
            Steve Blackburn
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated: