New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 846546 link

Starred by 7 users

Issue metadata

Status: Assigned
Owner:
Last visit > 30 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Make Crostini Terminal app less hacky

Project Member Reported by timloh@chromium.org, May 25 2018

Issue description

The Crostini Terminal is current running crosh (a v1 app) with some hacks to make it not get picked up in the shelf as crosh. We should try and avoid these hacks, for example moving crosh off the legacy app system and making separate PWAs for crosh and the terminal.
 
For now I think we just want to have the terminal be a separate v1 legacy packaged app to crosh.
But to be honest after hearing your explanation of the pinning fix I'm not longer sure, we should talk about it early next week. Maybe the shelf is the real hackiness, not how we've setup the terminal :/

Comment 3 by vapier@chromium.org, May 29 2018

Components: OS>Systems>Containers
I noticed some other weirdness. The terminal also shows up in Chrome history, and will re-open with ctrl-shift-T from a Chrome window.
The terminal also has problems handling complex commands passed as commandline inputs i.e. running "x-terminal-emulator <some command>".

See https://bugs.chromium.org/p/chromium/issues/detail?id=898111 for one manifestation of that, and the currently proposed fix for that will probably break (if they're not already broken) commands containing characters with special interpretations in URLs. Something more comprehensive should be done about this at some point.
i don't think x-terminal-emulator is related to this bug
Labels: -Pri-2 Pri-3
Summary: Make Crostini Terminal app less hacky (was: Make Crostini Terminal less hacky)

Sign in to add a comment