Command execution via --execute='' is broken

Bug #1324577 reported by Sergey "Shnatsel" Davidoff
22
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Terminal
Fix Released
High
Sergey "Shnatsel" Davidoff

Bug Description

Command execution via --execute='' is broken!

Steps to reproduce:
$ pantheon-terminal --execute='/bin/cat'

Expected: blank terminal waiting for input.
Actual results: either a terminal window appears but closes instantly, or it appears with a blank shell prompt, as if no command execution was requested.

This is observed both on Luna, version 0.2.4.1-0~r528+pkg19~ubuntu12.04.1, and Isis, version 0.3.0.1, and even on Debian.

A screencast demonstrating the issue on pantheon-terminal and the correct behavior on xfce4-terminal is attached.

Related branches

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :
Changed in pantheon-terminal:
status: New → Confirmed
importance: Undecided → High
Cody Garver (codygarver)
Changed in pantheon-terminal:
milestone: none → isis-beta2
Revision history for this message
dkotrada (dkotrada) wrote :

Mayby that log output will help. Who knows.

Starte externes Werkzeug '/usr/bin/pantheon-terminal' -geom 123x28+300+316 -e vi "/tmp/PastebinCompyth_z13174.txt" +2
[_LOG_LEVEL_INFO 20:43:04.991235] Application.vala:77: Pantheon Terminal version: 0.2.4.1
[_LOG_LEVEL_INFO 20:43:04.991277] Application.vala:79: Kernel version: 3.2.0-60-generic
[_LOG_LEVEL_DEBUG 20:43:04.992865] Settings.vala:158: Loading settings from schema 'org.pantheon.terminal.saved-state'
[_LOG_LEVEL_DEBUG 20:43:04.993481] Settings.vala:158: Loading settings from schema 'org.pantheon.terminal.settings'
[_LOG_LEVEL_DEBUG 20:43:05.206568] [Gtk] Connecting to session manager
pantheon-terminal: ERROR: Unbekannte Option -geom

'/usr/bin/pantheon-terminal' beendet

Changed in pantheon-terminal:
assignee: nobody → Alexander Fomin (xanderfomin)
status: Confirmed → In Progress
Revision history for this message
Alexander Fomin (xanderfomin) wrote :

There was a problem in process_argv() method for commands that consist from one string.
That's why any command like "cat" or "ls" was doomed to fail.
Note that commands like "sudo cat" worked fine.

Fix proposed for merge.

Changed in pantheon-terminal:
status: In Progress → Fix Committed
Revision history for this message
Alexander Fomin (xanderfomin) wrote :

I've reassigned this bug to you as your merge finally fixed it.

Changed in pantheon-terminal:
assignee: Alexander Fomin (xanderfomin) → Sergey "Shnatsel" Davidoff (shnatsel)
Changed in pantheon-terminal:
status: Fix Committed → Fix Released
status: Fix Released → Fix Committed
David Gomes (davidgomes)
Changed in pantheon-terminal:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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