Closed Bug 1232580 Opened 7 years ago Closed 7 years ago

hg push review breaks my terminal


(Firefox Build System :: Mach Core, enhancement)

Not set


(firefox46 fixed)

Tracking Status
firefox46 --- fixed


(Reporter: Yoric, Assigned: gps)



(1 file)

After calling `hg push review`, I lose the ability to read what I wrote on the terminal. stdout/stderr outputs from applications still work.

Here's my prompt:
PS1='\u \W \A\$ '
Your bug report lacks details to triage. We preferably need full console output. Also knowing the value of TERM, `hg config | grep term`, `hg config | grep color`, which version of Mercurial, and which terminal program you are using would be extremely helpful.
I'm using the MacOS X, version 2.6.1

$ hg config | grep term # Nothing
$ hg config | grep color
$ hg --version
Mercurial Distributed SCM (version 3.6-rc+20-6474b64045fb)
$ echo $TERM

Full output: (running `hg push review`, then `ls`, then `echo "I have just called echo, can you see it?"`).

david mc-probes 18:08$ hg push review
pushing to ssh://
searching for changes
no changes found
submitting 1 changesets for review

changeset:  273824:2e618e26e22c
summary:    Bug 1228020 - Removing about: pages from FX_PAGE_LOAD;r?dao

review id:  bz://1228020/Yoric
review url: (draft)

publish these review requests now (Yn)? n
 (visit review url to publish these review requests so others can see them)
david mc-probes 21:44$ AUTHORS					caps					gradle				startupcache				chrome			mozglue					storage
CLOBBER				gradlew			testing
GNUmakefile				hal					netwerk					toolkit
LEGAL					config					image					nsprpub					tools
LICENSE					configure				intl					obj-x86_64-apple-darwin15.0.0		uriloader				ipc					obj-x86_64-apple-darwin15.0.0.noindex	view
README.txt				db					js					other-licenses				webapprt
accessible				devtools				layout					parser					widget
aclocal.m4				docshell				mach					probes					xpcom
addon-sdk				dom					media					python					xpfe
b2g					editor					memory					rdf					xulrunner
browser					embedding				mfbt					security
build					extensions				mobile					services
build.gradle				gfx					modules					settings.gradle
david mc-probes 21:45$ I have just called echo, can you see it?
david mc-probes 21:45$
i'm unable to reproduce this using an almost identical setup.
the only difference is i have hg 3.6.3 not 3.6-rc+20-6474b64045fb.
y u no iTerm2? You really want to use

(That doesn't change the fact that this is a wonky bug.)
I still encounter the issue with 3.6.2.
Please follow up at, which is a similar bug report and hopefully the same underlying issue.

Worth noting from the initial bug report that `hg push review` does perform a prompt, asking the user whether they want to publish what they just submitted. and below say this is a MacPorts issue that can be worked around by `port install p27-readline`. Moving this to a mach bootstrap bug.

Will submit patch shortly.
Assignee: nobody → gps
Component: MozReview → mach
Product: Developer Services → Core
I assume it's actually "py27-readline".
Comment on attachment 8707077 [details]
MozReview Request: Bug 1232580 - Install py27-readline on MacPorts; r?Yoric

Looks good to me.
Attachment #8707077 - Flags: review?(dteller) → review+
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.