codesite | 1 Jan 01:20 2009
Picon

Issue 254 in xmonad: XMonad.Contrib.Util.Run.runInTerm does not work with terminal = "gnome-terminal"


Comment #4 on issue 254 by gwern0: XMonad.Contrib.Util.Run.runInTerm does  
not work with terminal = "gnome-terminal"
http://code.google.com/p/xmonad/issues/detail?id=254

OK. If you really want to not go through the shell, is there something  
wrong with
'safeRunInTerm'? I recall that this was the use-case the safe/unsafe  
distinction was
meant to cover. If there is nothing wrong with safeRunInTerm, perhaps you  
should
lobby the maintainers of Prompt.Man and Prompt.Ssh to use that instead.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
codesite | 1 Jan 11:25 2009
Picon

Issue 254 in xmonad: XMonad.Contrib.Util.Run.runInTerm does not work with terminal = "gnome-terminal"


Comment #5 on issue 254 by twpayne: XMonad.Contrib.Util.Run.runInTerm does  
not work with terminal = "gnome-terminal"
http://code.google.com/p/xmonad/issues/detail?id=254

I've only got basic notions of Haskell, but my understanding is that  
safeRunInTerm
avoids the shell but the command to be run is still converted to a string.   
Basically:

runInTerm (based on unsafeSpawn) does this (pseudo code):
   exec 'sh -c "terminal -e ssh localhost"'  -- convert to string, pass to sh
safeRunInTerm (based on safeSpawn) does this:
   exec 'terminal -e ssh localhost'  -- convert to string. don't pass to sh
but what's really needed is this:
   exec ['terminal', '-e', 'ssh localhost'] -- don't convert to string,  
invoke
                                            -- exec with list of arguments
Note how in the last example 'ssh localhost' is passed as a single  
argument, where in
both runInTerm and safeRunInTerm it is split into two arguments (because a  
string
passed to exec is implicitly split on whitespace to produce the list of  
arguments).

So, unfortunately I think the changes run a bit deeper than just using  
safeSpawn :-(

--
You received this message because you are listed in the owner
(Continue reading)

Brent Yorgey | 2 Jan 02:15 2009

Re: darcs patch: Prompt: Change Filemode to 600 for history-file (fixe...

On Thu, Dec 18, 2008 at 01:17:03AM +0100, Dominik Bruhn wrote:
> Thu Dec 18 01:16:01 CET 2008  Dominik Bruhn <dominik@...>
>   *  Prompt: Change Filemode to 600 for history-file (fixes bug 244)

Applied, thanks.

-Brent
codesite | 2 Jan 02:15 2009
Picon

Issue 244 in xmonad: .xmonad/history permissions may allow information leakage

Updates:
	Status: Fixed

Comment #3 on issue 244 by byorgey: .xmonad/history permissions may allow  
information leakage
http://code.google.com/p/xmonad/issues/detail?id=244

(No comment was entered for this change.)

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
Brent Yorgey | 2 Jan 03:33 2009

Re: another patch: modifies selection

On Sun, Dec 21, 2008 at 09:25:23PM -0500, Yaakov Nemoy wrote:
> Hey List,
> 
> Here's another patch.  It modifies the XSelection module so you can
> pass a string modifying function to the promptSelect functions.  This
> lets you run your own escape routines for various reasons.

These functions seem useful.  However, would you mind giving them
better names, adding some Haddock documentation, and resubmitting the
patch?  Also, why is the unsafe variant necessary?

-Brent
Brent Yorgey | 2 Jan 03:40 2009

Re: patch for ewmhDesktopLogHookCustom

On Wed, Dec 24, 2008 at 07:04:19PM -0500, rupa wrote:
> Hi,
> 
> I hope this patch is formatted correctly. I have tested it to the best of my
> ability and it fixes my issues nicely.

Applied, thanks.  I've marked issue #246 as fixed.

-Brent
codesite | 2 Jan 03:41 2009
Picon

Issue 246 in xmonad: ewmhDesktopsLogHookCustom throws(?) Maybe.fromJust: Nothing

Updates:
	Status: Fixed

Comment #1 on issue 246 by byorgey: ewmhDesktopsLogHookCustom throws(?)  
Maybe.fromJust: Nothing
http://code.google.com/p/xmonad/issues/detail?id=246

(No comment was entered for this change.)

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
Brent Yorgey | 2 Jan 03:55 2009

Re: New Layout: ThreeColumnsMiddle

On Tue, Dec 30, 2008 at 02:49:43PM +0100, Carsten Otto wrote:
> Hi,
> 
> I modified the layout "ThreeColumns" such that the master window is
> centered. In my opinion this layout is especially useful when using a
> huge screen (30" with 2560x1600 in my case).
> 
> Like in the original layout, there can be more than one master window.
> Furthermore, the size of the middle section can be changed.
> 
> Code and screenshot: http://server.c-otto.de/xmonad/
> 
> Sorry for not providing a patch, but I have no experience with darcs.

Hi Carsten, looks nice!  Please do consider submitting this as a darcs
patch.  There is actually a tutorial to walk you step-by-step through
exactly that process:

  http://haskell.org/haskellwiki/Xmonad/xmonad_development_tutorial

Hopefully the tutorial should make it easy -- if there's anything
that's confusing or unclear, consider it a bug in the tutorial and
please let us know!

-Brent
Carsten Otto | 2 Jan 10:08 2009
Picon

Re: New Layout: ThreeColumnsMiddle

On Thu, Jan 01, 2009 at 09:55:05PM -0500, Brent Yorgey wrote:
>   http://haskell.org/haskellwiki/Xmonad/xmonad_development_tutorial
> 
> Hopefully the tutorial should make it easy -- if there's anything
> that's confusing or unclear, consider it a bug in the tutorial and
> please let us know!

There is.

It seems XSelection.hs makes use of C preprocessing, which cannot be
handled when generating haddock documentation.

Running Haddock for xmonad-contrib-0.8...
dist/build/tmp/./XMonad/Util/XSelection.hs:"dist/build/tmp/./XMonad/Util/XSelection.hs":
33:1: Parse error

I will try to submit my patch without running this test.

Bye,
--

-- 
Carsten Otto
carsten@...
www.c-otto.de
On Thu, Jan 01, 2009 at 09:55:05PM -0500, Brent Yorgey wrote:
>   http://haskell.org/haskellwiki/Xmonad/xmonad_development_tutorial
> 
> Hopefully the tutorial should make it easy -- if there's anything
> that's confusing or unclear, consider it a bug in the tutorial and
(Continue reading)

Carsten Otto | 2 Jan 10:12 2009
Picon

Re: New Layout: ThreeColumnsMiddle

On Thu, Jan 01, 2009 at 09:55:05PM -0500, Brent Yorgey wrote:
> Hi Carsten, looks nice!  Please do consider submitting this as a darcs
> patch.  There is actually a tutorial to walk you step-by-step through
> exactly that process:

Attached.
-- 
Carsten Otto
carsten@...
www.c-otto.de
Fri Jan  2 10:10:19 CET 2009  xmonad@...
  * ThreeColumnsMiddle

New patches:

[ThreeColumnsMiddle
xmonad@...**20090102091019] {
addfile ./XMonad/Layout/ThreeColumnsMiddle.hs
hunk ./XMonad/Layout/ThreeColumnsMiddle.hs 1
+{-# LANGUAGE FlexibleInstances, MultiParamTypeClasses #-}
+
+-----------------------------------------------------------------------------
+-- |
+-- Module      :  XMonad.Layout.ThreeColumnsMiddle
+-- Copyright   :  (c) Carsten Otto <xmonad@...>,
+--                based on ThreeColumns (c) Kai Grossjohann <kai <at> emptydomain.de>
+-- License     :  BSD3-style (see LICENSE)
+-- 
(Continue reading)


Gmane