Re: [PATCH] Add -n argument to show-environment to specify a variable name

2012-05-06 Thread Thomas Adam
On 5 May 2012 18:41, Nicholas Marriott wrote: > On Sun, Apr 29, 2012 at 11:47:15PM +0100, Thomas Adam wrote: >> Hi, >> >> On 22 March 2012 11:14, Nicholas Marriott >> wrote: >> > Hi >> > >> > I tweaked this a bit so the output format is the same for one or many >> > variables and applied it, tha

Re: [PATCH] Add -n argument to show-environment to specify a variable name

2012-05-05 Thread Nicholas Marriott
On Sun, Apr 29, 2012 at 11:47:15PM +0100, Thomas Adam wrote: > Hi, > > On 22 March 2012 11:14, Nicholas Marriott wrote: > > Hi > > > > I tweaked this a bit so the output format is the same for one or many > > variables and applied it, thanks! > > Sorry to bring this up now after this patch has g

Re: [PATCH] Add -n argument to show-environment to specify a variable name

2012-04-29 Thread Thomas Adam
Hi, On 22 March 2012 11:14, Nicholas Marriott wrote: > Hi > > I tweaked this a bit so the output format is the same for one or many > variables and applied it, thanks! Sorry to bring this up now after this patch has gone out, but I'm curious to know if the following patch isn't more preferred wh

Re: [PATCH] Add -n argument to show-environment to specify a variable name

2012-03-22 Thread Nicholas Marriott
Hi I tweaked this a bit so the output format is the same for one or many variables and applied it, thanks! On Wed, Mar 21, 2012 at 05:45:34PM +0900, Kazuhiko Sakaguchi wrote: > Hi > > Thanks! The attached patch fixed the bug. > > 2012/3/17 Nicholas Marriott : > > Hi > > > > I think that when

Re: [PATCH] Add -n argument to show-environment to specify a variable name

2012-03-21 Thread Kazuhiko Sakaguchi
Hi Thanks! The attached patch fixed the bug. 2012/3/17 Nicholas Marriott : > Hi > > I think that when you print a single option you need to check > envent->value is not NULL (meaning the variable is removed from > environment), same as inside the loop. > > Also this patch doesn't apply, I guess

Re: [PATCH] Add -n argument to show-environment to specify a variable name

2012-03-13 Thread Nicholas Marriott
Hi Looks good but I think there is no need for -n, just use the first argument if argc != 0, like the recent changes to show-option. Thanks On Tue, Mar 13, 2012 at 06:56:34AM +0900, Kazuhiko Sakaguchi wrote: > Hi, > > This patch add a -n argument to show-environment. It specifies a variable

[PATCH] Add -n argument to show-environment to specify a variable name

2012-03-12 Thread Kazuhiko Sakaguchi
Hi, This patch add a -n argument to show-environment. It specifies a variable name. example: $ tmux show-environment -g -n TERM rxvt-unicode-256color --- a/cmd-show-environment.c +++ b/cmd-show-environment.c @@ -31,8 +31,8 @@ int cmd_show_environment_exec(struct cmd *, struct cmd_ctx *); c