add support for specifying an ssh private key in the symbol upload script

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
13 years ago
12 years ago

People

(Reporter: ted, Assigned: ted)

Tracking

unspecified
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 4 obsolete attachments)

Apparently the windows tinderbox needs to specify an SSH private key in the ssh command line.  I'll add another env var that the upload script will optionally use to point to a private key.
We may not need this, this should be doable in ~/.ssh/config.
Blocks: 376063
Add the env var, plus some env var renaming.  I also tweaked tinder-defaults.pl to match.
Attachment #260446 - Flags: first-review?(benjamin)
Posted patch same as before (obsolete) — Splinter Review
Just realized that some of my entries in tinder-defaults didn't have trailing semicolons.  At least they were commented out!
Attachment #260446 - Attachment is obsolete: true
Attachment #260447 - Flags: first-review?(benjamin)
Attachment #260446 - Flags: first-review?(benjamin)
Attachment #260447 - Flags: first-review?(benjamin) → first-review+
Checked in on trunk.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attachment #260605 - Flags: first-review?(ted.mielczarek)
Posted patch correct bash-fu (obsolete) — Splinter Review
Right, I totally tested with the correct bashism and then wrote the wrong thing in the patch.  D'oh!
Attachment #260447 - Attachment is obsolete: true
Attachment #260605 - Attachment is obsolete: true
Attachment #260614 - Flags: first-review?(benjamin)
Attachment #260605 - Flags: first-review?(ted.mielczarek)
Ugh, looking at the log it wouldn't have worked even with that fix, cause I missed a line in my renaming in an earlier patch.  This fixes both.
Attachment #260614 - Attachment is obsolete: true
Attachment #260615 - Flags: first-review?(benjamin)
Attachment #260614 - Flags: first-review?(benjamin)
Attachment #260615 - Flags: first-review?(benjamin) → first-review+
Changed to #!/bin/bash in the script and checked in.
Status: REOPENED → RESOLVED
Closed: 13 years ago13 years ago
Resolution: --- → FIXED
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.