Print a better error message if $DISPLAY is not set

NEW
Unassigned

Status

6 years ago
a month ago

People

(Reporter: vd, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mach])

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
Created attachment 737880 [details]
Full output from ./mach without $DISPLAY being set

If ./mach is run without $DISPLAY being set (e.g. over ssh without X forwarding), then the errors being printed do not make it obvious what the problem is.

See the full output in the attached file.

Comment 1

6 years ago
I'm going to punt this over to the realm of testing. Only commands are in the best position to determine whether they are capable of being executed. In this particular example, I think the test runners should be responsible for validating the environment. IMO they should fail fast with a clear error message. If we want to expose the "is environment OK to run this test harness" to mach and get even better UX, we can do that. But the first step is having the test runners detect a bad environment.
Component: mach → Mochitest
Product: Core → Testing
Summary: Print a better error message from ./mach if $DISPLAY is not set → Print a better error message if $DISPLAY is not set
Whiteboard: [mach]
(Reporter)

Updated

5 years ago
Assignee: vd → nobody
(Reporter)

Comment 2

5 years ago
I have no time to work on Mozilla bugs for now, should have unassigned this from me some time ago already to give chance to others to crack it.
No assignee, updating the status.
Status: ASSIGNED → NEW
You need to log in before you can comment on or make changes to this bug.