After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 780154 - Add a "Open terminal in flatpak enviroment" menu item
Add a "Open terminal in flatpak enviroment" menu item
Status: RESOLVED OBSOLETE
Product: gnome-builder
Classification: Other
Component: general
3.23.x
Other Linux
: Normal normal
: ---
Assigned To: GNOME Builder Maintainers
GNOME Builder Maintainers
Depends on:
Blocks: 781010
 
 
Reported: 2017-03-16 15:24 UTC by Carlos Soriano
Modified: 2018-01-11 10:21 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
terminal: Add Runtime Terminal action to menu (1.04 KB, patch)
2017-04-08 21:14 UTC, Matthew Leeds
committed Details | Review

Description Carlos Soriano 2017-03-16 15:24:19 UTC
Right now we have in the hamburguer menu Open terminal and a item menu called Flatpak that have some specific options in it.
It would be cool to present also a menu item for opening a terminal in flatpak environment This is alreayd possible with the ctrl+alt+shift+t shortcut, so the code is already in there, only thing missing is the menu item.

For now it probably should be inside the Flatpak submenu, and if in the future we have some abstraction layer, we can put a more generic "Open terminal in sandbox build environment" or so.

This is a good task for newcomers once the maintainers agrees.
Comment 1 Christian Hergert 2017-03-18 06:43:36 UTC
This feature is not Flatpak specific, so it seems weird to put it under the flatpak menu.

It is just a terminal inside your build environment (could be on the host, could be inside of jhbuild, could be inside of flatpak).

But yes, it would be nice to have some UI somewhere for this.
Comment 2 Matthew Leeds 2017-04-08 21:14:49 UTC
Created attachment 349533 [details] [review]
terminal: Add Runtime Terminal action to menu
Comment 3 Christian Hergert 2017-04-08 22:30:08 UTC
Review of attachment 349533 [details] [review]:

LGTM

Should we specify build vs app terminal?

Also push to stable branch
Comment 4 Matthew Leeds 2017-04-09 00:28:54 UTC
Pushed as commit eb15290a3 and commit c7c7aec02. 

Do we have ways to open both?
Comment 5 Christian Hergert 2017-04-09 02:30:38 UTC
we do not yet. its possible to add, it just sort of complicates the spawning code in gb-terminal-view.c, and we probably need to use IdeRunner so that we get finish-args and such.
Comment 6 Christian Hergert 2017-07-21 20:28:12 UTC
We have a "New Stuff Button" in the header bar now, which has a new terminal menu item. I think we should probably move this there as well.

Since the runtime terminal currently is based around the build environment more than runtime, we should probably name it "New build terminal" or something like that.

We can/should probably have a way to do the run terminal but that still requires work in the terminal plugin to access the runner.
Comment 7 GNOME Infrastructure Team 2018-01-11 10:21:36 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/gnome-builder/issues/194.