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 763888 - [PATCH] Add Command applet
[PATCH] Add Command applet
Status: RESOLVED FIXED
Product: gnome-applets
Classification: Other
Component: general
git master
Other Linux
: Normal enhancement
: ---
Assigned To: gnome-applets Maintainers
gnome-applets Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-03-18 17:47 UTC by Balló György
Modified: 2016-03-21 20:26 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add Command applet (22.89 KB, patch)
2016-03-18 17:47 UTC, Balló György
none Details | Review
Add Command applet v2 (23.00 KB, patch)
2016-03-20 16:33 UTC, Balló György
committed Details | Review
command, timerapplet: port from GtkTable to GtkGrid (9.16 KB, patch)
2016-03-21 19:57 UTC, Balló György
committed Details | Review

Description Balló György 2016-03-18 17:47:40 UTC
Created attachment 324296 [details] [review]
Add Command applet

It's a new applet originally developed for MATE Panel. Original source:
http://git.mate-desktop.org/mate-applets/tree/command
Comment 1 Alberts Muktupāvels 2016-03-19 11:48:39 UTC
I am not license expert, but:
GNOME Applets - GNU General Public License, version 2 or any later version;
Your patch - GNU General Public License, version 3 or any later version;

When we merged window-picker applet, Sebastian asked for permission to "downgrade" license:
https://github.com/lanoxx/window-picker-applet/commit/c9738bab0777a1da4feda8f25cd53c1cb0710c71

So you will need to ask author if he allows to use his code under GPLv2 or later, probably by adding comment here.

Then update patch to also include bug url. Same thing with your patch in 763889 bug.
Comment 2 Balló György 2016-03-19 12:36:18 UTC
Ah, I see. I opened a new issue in their bug tracker:
https://github.com/mate-desktop/mate-applets/issues/183
Comment 3 Balló György 2016-03-20 16:33:02 UTC
Created attachment 324368 [details] [review]
Add Command applet v2

The original author agreed to re-license Command and Timer applets under GPL-2+:
https://github.com/mate-desktop/mate-applets/issues/183#issuecomment-198745034
Comment 4 Alberts Muktupāvels 2016-03-21 12:54:34 UTC
Both applets use deprecated GtkTable, can you port to GtkGrid?
Comment 5 Balló György 2016-03-21 19:57:29 UTC
Created attachment 324502 [details] [review]
command, timerapplet: port from GtkTable to GtkGrid

Done, see the attachment. Tested and works for me.
Comment 6 Alberts Muktupāvels 2016-03-21 20:26:55 UTC
Review of attachment 324502 [details] [review]:

Thanks!