* [PATCH mintty] Add an XWin XDG menu entry @ 2015-07-06 23:12 Yaakov Selkowitz 2015-07-07 9:47 ` Thomas Wolff 0 siblings, 1 reply; 7+ messages in thread From: Yaakov Selkowitz @ 2015-07-06 23:12 UTC (permalink / raw) To: cygwin-apps --- Thomas, could you please ship a new release with this patch ASAP? cygwin/mintty.cygport | 2 ++ 1 file changed, 2 insertions(+) diff --git a/cygwin/mintty.cygport b/cygwin/mintty.cygport index 35ee031..26f5423 100644 --- a/cygwin/mintty.cygport +++ b/cygwin/mintty.cygport @@ -32,4 +32,6 @@ src_install() { doman docs/mintty.1 dodoc COPYING LICENSE.Oxygen LICENSE.PuTTY dodoc cygwin/README cygwin/setup.hint + + make_desktop_entry mintty "Cygwin Terminal" utilities-terminal "System;TerminalEmulator" "" "OnlyShowIn=X-Cygwin;" } -- 2.4.4 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH mintty] Add an XWin XDG menu entry 2015-07-06 23:12 [PATCH mintty] Add an XWin XDG menu entry Yaakov Selkowitz @ 2015-07-07 9:47 ` Thomas Wolff 2015-07-07 11:47 ` Yaakov Selkowitz 0 siblings, 1 reply; 7+ messages in thread From: Thomas Wolff @ 2015-07-07 9:47 UTC (permalink / raw) To: cygwin-apps On 07.07.2015 01:12, Yaakov Selkowitz wrote: > --- > Thomas, could you please ship a new release with this patch ASAP? Are you sure this is desired? Mintty isn't related to X Windows; it could be listed in its menu anyway, but... one more opinion? Thomas > cygwin/mintty.cygport | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/cygwin/mintty.cygport b/cygwin/mintty.cygport > index 35ee031..26f5423 100644 > --- a/cygwin/mintty.cygport > +++ b/cygwin/mintty.cygport > @@ -32,4 +32,6 @@ src_install() { > doman docs/mintty.1 > dodoc COPYING LICENSE.Oxygen LICENSE.PuTTY > dodoc cygwin/README cygwin/setup.hint > + > + make_desktop_entry mintty "Cygwin Terminal" utilities-terminal "System;TerminalEmulator" "" "OnlyShowIn=X-Cygwin;" > } ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH mintty] Add an XWin XDG menu entry 2015-07-07 9:47 ` Thomas Wolff @ 2015-07-07 11:47 ` Yaakov Selkowitz 2015-07-07 20:00 ` Thomas Wolff 0 siblings, 1 reply; 7+ messages in thread From: Yaakov Selkowitz @ 2015-07-07 11:47 UTC (permalink / raw) To: cygwin-apps On Tue, 2015-07-07 at 11:47 +0200, Thomas Wolff wrote: > On 07.07.2015 01:12, Yaakov Selkowitz wrote: > > --- > > Thomas, could you please ship a new release with this patch ASAP? > Are you sure this is desired? Mintty isn't related to X Windows; Well, not for a full X desktop, and that's the reason for the OnlyShowIn. But within the context of a multiwindow session it makes sense to provide this option. -- Yaakov ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH mintty] Add an XWin XDG menu entry 2015-07-07 11:47 ` Yaakov Selkowitz @ 2015-07-07 20:00 ` Thomas Wolff 2015-07-07 23:40 ` Michael DePaulo 0 siblings, 1 reply; 7+ messages in thread From: Thomas Wolff @ 2015-07-07 20:00 UTC (permalink / raw) To: cygwin-apps Am 07.07.2015 um 13:47 schrieb Yaakov Selkowitz: > On Tue, 2015-07-07 at 11:47 +0200, Thomas Wolff wrote: >> On 07.07.2015 01:12, Yaakov Selkowitz wrote: >>> --- >>> Thomas, could you please ship a new release with this patch ASAP? >> Are you sure this is desired? Mintty isn't related to X Windows; > Well, not for a full X desktop, and that's the reason for the > OnlyShowIn. But within the context of a multiwindow session it makes > sense to provide this option. OK, anyone else to approve this? Sorry I'm asking but some people might think it's alien in an X menu. (Would that addition actually pass without a mintty dependency on xdg? Otherwise, since mintty is in cygwin-base, xdg itself could also include the entry.) Thomas ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH mintty] Add an XWin XDG menu entry 2015-07-07 20:00 ` Thomas Wolff @ 2015-07-07 23:40 ` Michael DePaulo 2015-07-08 12:44 ` Thomas Wolff 0 siblings, 1 reply; 7+ messages in thread From: Michael DePaulo @ 2015-07-07 23:40 UTC (permalink / raw) To: cygwin-apps On Tue, Jul 7, 2015 at 4:00 PM, Thomas Wolff <towo@towo.net> wrote: > Am 07.07.2015 um 13:47 schrieb Yaakov Selkowitz: >> >> On Tue, 2015-07-07 at 11:47 +0200, Thomas Wolff wrote: >>> >>> On 07.07.2015 01:12, Yaakov Selkowitz wrote: >>>> >>>> --- >>>> Thomas, could you please ship a new release with this patch ASAP? >>> >>> Are you sure this is desired? Mintty isn't related to X Windows; >> >> Well, not for a full X desktop, and that's the reason for the >> OnlyShowIn. But within the context of a multiwindow session it makes >> sense to provide this option. > > OK, anyone else to approve this? Sorry I'm asking but some people might > think it's alien in an X menu. > (Would that addition actually pass without a mintty dependency on xdg? > Otherwise, since mintty is in cygwin-base, xdg itself could also include the > entry.) > Thomas I think this makes sense because: 1. Many people prefer mintty over X11 terminals. 2. Many non-technical users do not understand what X11 is. They probably just know that they they can launch Cygwin "Apps" from this taskbar menu, and the Cygwin terminal is seemingly another Cygwin "App". Believe me, I have seen misconceptions about X11 at work, even from users who use terminals. [1] 3. From an end-user's perspective, mintty isn't significantly different from an X11 app in MultiWindow mode. For example, mintty has native window decorations like most X11 apps do, you can share clipboard between mintty and X11 apps, etc. [1]. A user was using a proprietary Windows X server with XDMCP, and she thought that the remote machine's login screen (display manager's greeter), desktop environment, terminal emulator, etc was part of the X server. She was amazed that they did not change when I migrated her to Cygwin XWin with XDMCP. (Don't worry, I later migrated all my users to X2Go or X11 forwarded over SSH.) ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH mintty] Add an XWin XDG menu entry 2015-07-07 23:40 ` Michael DePaulo @ 2015-07-08 12:44 ` Thomas Wolff 2015-07-08 16:53 ` Yaakov Selkowitz 0 siblings, 1 reply; 7+ messages in thread From: Thomas Wolff @ 2015-07-08 12:44 UTC (permalink / raw) To: cygwin-apps On 08.07.2015 01:40, Michael DePaulo wrote: > On Tue, Jul 7, 2015 at 4:00 PM, Thomas Wolff <towo@towo.net> wrote: >> Am 07.07.2015 um 13:47 schrieb Yaakov Selkowitz: >>> On Tue, 2015-07-07 at 11:47 +0200, Thomas Wolff wrote: >>>> On 07.07.2015 01:12, Yaakov Selkowitz wrote: >>>>> --- >>>>> Thomas, could you please ship a new release with this patch ASAP? >>>> Are you sure this is desired? Mintty isn't related to X Windows; >>> Well, not for a full X desktop, and that's the reason for the >>> OnlyShowIn. But within the context of a multiwindow session it makes >>> sense to provide this option. >> OK, anyone else to approve this? Sorry I'm asking but some people might >> think it's alien in an X menu. >> (Would that addition actually pass without a mintty dependency on xdg? >> Otherwise, since mintty is in cygwin-base, xdg itself could also include the >> entry.) >> Thomas > I think this makes sense because: > ... Convinced. According to file:///C:/cygwin/usr/share/doc/cygport/manual.html#robo164 , Categories and Additional_Fields parameters should be colon-separated, so I would put: make_desktop_entry mintty "Cygwin Terminal" utilities-terminal "System:Terminal Emulator" "" "OnlyShowIn=X-Cygwin" -- about the icon, shouldn't mintty's own icon be used here? How would it be passed to cygport, does it need to be installed first? Thomas ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH mintty] Add an XWin XDG menu entry 2015-07-08 12:44 ` Thomas Wolff @ 2015-07-08 16:53 ` Yaakov Selkowitz 0 siblings, 0 replies; 7+ messages in thread From: Yaakov Selkowitz @ 2015-07-08 16:53 UTC (permalink / raw) To: cygwin-apps On Wed, 2015-07-08 at 14:44 +0200, Thomas Wolff wrote: > Convinced. According to > file:///C:/cygwin/usr/share/doc/cygport/manual.html#robo164 , > Categories and Additional_Fields parameters should be colon-separated, > so I would put: > make_desktop_entry mintty "Cygwin Terminal" utilities-terminal > "System:Terminal Emulator" "" "OnlyShowIn=X-Cygwin" This in incorrect. Please include the patch *exactly* as sent. > -- about the icon, shouldn't mintty's own icon be used here? How would > it be passed to cygport, does it need to be installed first? If anything, I'd take the /Cygwin-Terminal.ico and use *that* as the mintty icon in this context. Would you like a patch? -- Yaakov ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-07-08 16:53 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2015-07-06 23:12 [PATCH mintty] Add an XWin XDG menu entry Yaakov Selkowitz 2015-07-07 9:47 ` Thomas Wolff 2015-07-07 11:47 ` Yaakov Selkowitz 2015-07-07 20:00 ` Thomas Wolff 2015-07-07 23:40 ` Michael DePaulo 2015-07-08 12:44 ` Thomas Wolff 2015-07-08 16:53 ` Yaakov Selkowitz
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).