Add a setting to set the shell to launch

Bug #909717 reported by Fabian Thoma
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Terminal
Fix Released
Wishlist
David Gomes

Bug Description

What it says in the description

Revision history for this message
David Gomes (davidgomes) wrote :

This needs design input. Should it be like Gnome Terminal? I'm not sure we need all those options, but I need to know how to organize them.

David Gomes (davidgomes)
Changed in pantheon-terminal:
importance: Undecided → Wishlist
status: New → Confirmed
Cody Garver (codygarver)
Changed in pantheon-terminal:
assignee: nobody → elementary UX Team (elementary-design)
Revision history for this message
Danielle Foré (danrabbit) wrote :

Can someone explain what this even means?

Revision history for this message
David Gomes (davidgomes) wrote :

By default, we launch Bash, but a terminal can be made to launch other stuff.

Revision history for this message
Harvey Cabaguio (harveycabaguio) wrote :

What is the point of launching something else other than bash? Just curious.

Revision history for this message
Cris Dywan (kalikiana) wrote :

When you say "bash" you surely mean "the user's default shell".

To change the user shell, you would usually use "chsh -s /bin/zsh". Otherwise it would not change in eg. embedded terminals or virtual consoles. However GNOME Terminal has a "custom command" option, maybe this is what's being referred to here - personally I've never had a use case for it.

Revision history for this message
Danielle Foré (danrabbit) wrote :

Yea, if there is a system wide setting for your preferred shell, I would think we should always respect that.

Revision history for this message
David Gomes (davidgomes) wrote :

But Daniel, this is a power application, and by default we'll always respect bash, just make it customizable. Even if not on the Preferences Dialog, but instead only via dconf-editor.

Changed in pantheon-terminal:
milestone: none → 0.2
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Re-targeting to 0.1 because of https://blueprints.launchpad.net/elementaryos/+spec/friendly-shell
Should be a dconf key, not exposed via GUI.

Changed in pantheon-terminal:
milestone: 0.2 → 0.1
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Christian: chsh is completely different. It changes the login shell, which handles things like system() calls etc. It's never directly exposed to the user in GUI mode. In Debian and Ubuntu it's DASH, while terminals use BASH by default. That may be either libvte or terminal setting; I'll check that out. If there's a libvte setting, this should not be done in pantheon-terminal.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Apparently it's set in an environment variable, $SHELL

Changed in pantheon-terminal:
status: Confirmed → Incomplete
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Oh, wait. Looks like Christian was right after all...

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

I'll try setting my default shell in /etc/passwd to FISH and see if it works.

Revision history for this message
David Gomes (davidgomes) wrote :

Shnatsel, how did it go?

Changed in pantheon-terminal:
status: Incomplete → Opinion
status: Opinion → Confirmed
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Setting fish in /etc/passwd worked. I doubt there's an easy way to do this with libvte, so screw it.

Changed in pantheon-terminal:
status: Confirmed → Invalid
Revision history for this message
David Gomes (davidgomes) wrote :

It's actually very easy to make it customizable, just change one argument in a certain function. If people ask for it, I'll add it, but for now, it's fine.

Revision history for this message
David Gomes (davidgomes) wrote :

Came to think about it, we should add this.

Changed in pantheon-terminal:
status: Invalid → Confirmed
Revision history for this message
Danielle Foré (danrabbit) wrote :

If it were solely my decision, I wouldn't put any preferences in pantheon-terminal at all. It seems like such a simple app that there's no need to do it. And if you know about changing shells, then you'll probably do like Christian suggested before.

I dunno, that's just my opinion.

Changed in pantheon-terminal:
assignee: elementary UX Team (elementary-design) → nobody
Changed in pantheon-terminal:
assignee: nobody → Mario Guerriero (mefrio-g)
Revision history for this message
Mario Guerriero (mefrio-g) wrote :

I made some tests trying to launch "chsh -s /bin/shell" and I have to figure that commands which needs root password can't be launched well with GLib.Process.spawn_command_line so I don't know other ways to get it working

Changed in pantheon-terminal:
assignee: Mario Guerriero (mefrio-g) → nobody
Revision history for this message
Danielle Foré (danrabbit) wrote :

I don't think this is really relevant to launching Luna beta since it's a wish list and not a bug. Removing the luna beta milestone.

Changed in pantheon-terminal:
milestone: luna-beta1 → none
David Gomes (davidgomes)
Changed in pantheon-terminal:
status: Confirmed → Won't Fix
status: Won't Fix → Confirmed
David Gomes (davidgomes)
Changed in pantheon-terminal:
assignee: nobody → David Gomes (davidgomes)
milestone: none → luna-beta2
status: Confirmed → Fix Committed
Changed in pantheon-terminal:
status: Fix Committed → Fix Released
Revision history for this message
Marnen Laibow-Koser (marnen) wrote :

I guess I'll be using GNOME Terminal, then. I do in fact want preference settings in my terminal application: not everything should be done by messing around with config files. See iTerm on Mac OS for an example of a terminal emulator that does this well.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.