Foopy start_cp.sh should ensure an Attached screen session

RESOLVED FIXED

Status

Release Engineering
Buildduty
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: Callek, Assigned: Callek)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 612001 [details] [diff] [review]
[tools] v1

So, we have been hit before with start_cp.sh not being attached to a screen session!

As well as exiting the screen session when done.

This patch should make the human-side of updating foopies much saner!

x.ref with: http://stackoverflow.com/questions/3472287/how-do-you-tell-if-the-current-terminal-session-is-in-gnu-screen and http://www.gnu.org/software/screen/manual/screen.html#Term
Attachment #612001 - Flags: review?(bear)

Comment 1

6 years ago
\o/

Comment 2

6 years ago
Comment on attachment 612001 [details] [diff] [review]
[tools] v1

you may want to escape or quote the pipes, probably not required, but using IO redirection operators in an echo makes me all itchy like when they are not quoted
Attachment #612001 - Flags: review?(bear) → review+
(Assignee)

Comment 3

6 years ago
http://hg.mozilla.org/build/tools/rev/7ad60dba039a

At least for foopy05 this needs to be copied out of tools directly (not symlinked).
(Assignee)

Comment 4

6 years ago
Gyah, looks like I forgot to save the file before qref/finish/push.

Fixed in: http://hg.mozilla.org/build/tools/rev/db4ecf0524e8
(Assignee)

Comment 5

6 years ago
This is fixed, but still needs deploy to the foopies. Filing a followup bug for that.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
Depends on: 742499
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.