Re: [dev] [st][PATCH] Add support for utmp in st

2014-10-14 Thread koneu
random...@fastmail.us wrote: > What is Home? What is life?

Re: [dev] [st][PATCH] Add support for utmp in st (DISREGARD LAST)

2014-10-13 Thread k0ga
> Because that is the character sent by the key in this position with this > expected function (i.e. the also, for better or worse, what modern linux systems have standardized > on (I haven't done a survey of bsd, solaris, etc). Linux has not standardized anything. POSIX is the standard. You c

Re: [dev] [st][PATCH] Add support for utmp in st (DISREGARD LAST)

2014-10-13 Thread random832
Sorry I accidentally hit shift-enter and apparently that makes my email client send. On Mon, Oct 13, 2014, at 14:38, k...@shike2.com wrote: > But, why do you think is better DELETE than BACKSPACE? Because that is the character sent by the key in this position with this expected function (i.e. the

Re: [dev] [st][PATCH] Add support for utmp in st

2014-10-13 Thread random832
On Mon, Oct 13, 2014, at 14:38, k...@shike2.com wrote: > > On Sun, Oct 12, 2014, at 14:32, k...@shike2.com wrote: > > That doesn't mean that the question of what the default should be is not > > worth discussing. > > Default configuration was discussed here some time ago, and suckless > develope

Re: [dev] [st][PATCH] Add support for utmp in st

2014-10-13 Thread k0ga
> On Sun, Oct 12, 2014, at 14:32, k...@shike2.com wrote: > That doesn't mean that the question of what the default should be is not > worth discussing. Default configuration was discussed here some time ago, and suckless developers agreed with current configuration. Both options, Backspace genera

Re: [dev] [st][PATCH] Add support for utmp in st

2014-10-13 Thread random832
On Sun, Oct 12, 2014, at 14:32, k...@shike2.com wrote: > If the user doesn't like the key assignation on st he is free of changing > it > in his config.h (maybe we could add it to the FAQ). That doesn't mean that the question of what the default should be is not worth discussing. > > You didn't c

Re: [dev] [st][PATCH] Add support for utmp in st

2014-10-12 Thread k0ga
> Why shouldn't a non-login shell have a utmp session? And if this option > is to use a login shell, rather than merely using utmp, then I don't > think it should be a compile-time option - just because someone > sometimes wants a login shell (which could be done before, if desired, > by running e

Re: [dev] [st][PATCH] Add support for utmp in st

2014-10-12 Thread random832
On Sun, Oct 12, 2014, at 03:48, k...@shike2.com wrote: > And the profile runs in the same tty that st opens. St by default > executes a non login shell, so profile is not loaded, but utmp executes > a login shell (because it creates the utmp session, so it is more > logical for it to execute a log

Re: [dev] [st][PATCH] Add support for utmp in st

2014-10-12 Thread k0ga
> On Sat, Oct 11, 2014, at 04:07, k...@shike2.com wrote: >> Value of erase key for example, or in general the configuration >> of line kernel driver > > These can't come from the profile either; since st opens a new tty that > is not the same device the user logged in on. And the profile runs in

Re: [dev] [st][PATCH] Add support for utmp in st

2014-10-11 Thread random832
On Sat, Oct 11, 2014, at 04:07, k...@shike2.com wrote: > Value of erase key for example, or in general the configuration > of line kernel driver These can't come from the profile either; since st opens a new tty that is not the same device the user logged in on. (stty(1)). Backspace key in st ge

Re: [dev] [st][PATCH] Add support for utmp in st

2014-10-11 Thread k0ga
> > What exactly does "terminal configuration" mean here? TERM, LINES, and > COLUMNS? Shouldn't st itself be responsible for setting these? They > certainly don't belong in the profile. Value of erase key for example, or in general the configuration of line kernel driver (stty(1)). Backspace key

Re: [dev] [st][PATCH] Add support for utmp in st

2014-10-10 Thread random832
On Tue, Sep 23, 2014, at 01:18, Roberto E. Vargas Caballero wrote: > St runs an interactive shell and not a login shell, and it means > that profile is not loaded. The default terminal configuration > in some system is not the correct for st, but since profile is > not loaded there is no way of get

Re: [dev] [st][PATCH] Add support for utmp in st

2014-10-10 Thread k0ga
>> -setenv("SHELL", sh, 1); >> +setenv("SHELL", args[0], 1); > > I'm not sure whether this is a good idea? The user asked to execute > args[0] instead of the shell, but this doesn't imply that args[0] is > his shell. Yes, you are right, it should be setenv("SHELL", sh, 1); Can you send

Re: [dev] [st][PATCH] Add support for utmp in st

2014-10-10 Thread Marc André Tanner
Hi, On Tue, Sep 23, 2014 at 07:18:32AM +0200, Roberto E. Vargas Caballero wrote: > --- a/st.c > +++ b/st.c > @@ -1153,16 +1153,22 @@ execsh(void) { ... > - setenv("SHELL", sh, 1); > + setenv("SHELL", args[0], 1); I'm not sure whether this is a good idea? The user asked to execute args[0

[dev] [st][PATCH] Add support for utmp in st

2014-09-24 Thread Roberto E. Vargas Caballero
St runs an interactive shell and not a login shell, and it means that profile is not loaded. The default terminal configuration in some system is not the correct for st, but since profile is not loaded there is no way of getting a script configures the correct values. St doesn't update the utmp fi