From: Eli Zaretskii <eliz@gnu.org>
To: Matthew Leach <matthew@mattleach.net>
Cc: 37564@debbugs.gnu.org
Subject: bug#37564: [PATCH] don't export LINES and COLUMNS env vars in term to fix ncurses applications
Date: Tue, 01 Oct 2019 21:58:25 +0300 [thread overview]
Message-ID: <83y2y48f8u.fsf@gnu.org> (raw)
In-Reply-To: <87sgocfh8c.fsf@mattleach.net> (message from Matthew Leach on Tue, 01 Oct 2019 19:33:39 +0100)
> From: Matthew Leach <matthew@mattleach.net>
> Cc: 37564@debbugs.gnu.org
> Date: Tue, 01 Oct 2019 19:33:39 +0100
>
> >> I'm curious as to which programs you are referring? AFAIK, if a program
> >> tries to read the LINES and COLUMNS environment variables, using
> >> `getenv()', they don't exist.
> >
> > What makes you say that?
>
> If I run the C program in my previous post in an xterm and resize I get:
>
> getenv: (null) (null)
> ioctl: 24 80
> getenv: (null) (null)
> ioctl: 42 169
>
> You can see that doing getenv() on LINES and COLUMNS returns a null
> pointer. I've tested this same behaviour on gnome-terminal and numerous
> other terminal emulators.
I simply tried "env | fgrep LINES" and I do see the variable, although
it wasn't there before I launched Emacs. 'env' is just another
program accessing the environment, right? I have no idea why your
test program doesn't see that, but then I'm far from being an expert
on these issues.
> > Emacs exports these variables into the environment that is passed to
> > child subprocesses, so those subprocesses will definitely see them
> > using getenv.
>
> I'm no expert on how these things work but it doesn't appear to be the
> way that it's done, on my machine anyway. Perhaps there are programs
> that uses these variables on MacOS or Windows?
I tried the above on a Trisquel GNU/Windows machine, FWIW.
> I'm testing this on the latest Emacs Git version. I can see that
> window-adjust-process-window-size-function is used in term.el however,
> this doesn't appear to update the environment variables exported to the
> terminal process.
AFAIR, it's supposed to send the corresponding ioctl command, in
addition to setting LINES and COLUMNS.
> If I run the same program with term on Emacs git and
> resize I get:
>
> getenv: 31 80
> ioctl: 30 89
> getenv: 31 80
> ioctl: 30 90
> getenv: 31 80
> ioctl: 30 91
> getenv: 31 80
> ioctl: 31 92
> getenv: 31 80
> ioctl: 31 93
> getenv: 31 80
> ioctl: 31 94
> ...
>
> Notice that the values returned by getenv don't change.
Again, the "env | fgrep LINES" method does show the change when I
resize the window on the machine I tried this, so I'm unsure why you
don't see it.
> Given that this functionality is broken, I'm suggesting that we don't
> export the variables.
I don't agree that it's broken; it isn't for me. I think someone who
understands more than I do about this stuff should chime in and
explain why we see such different results.
next prev parent reply other threads:[~2019-10-01 18:58 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-30 19:59 bug#37564: [PATCH] don't export LINES and COLUMNS env vars in term to fix ncurses applications Matthew Leach
2019-10-01 6:01 ` Eli Zaretskii
2019-10-01 15:47 ` Matthew Leach
2019-10-01 16:07 ` Eli Zaretskii
2019-10-01 18:33 ` Matthew Leach
2019-10-01 18:58 ` Eli Zaretskii [this message]
2019-10-01 19:14 ` Eli Zaretskii
2019-10-01 19:24 ` Matthew Leach
2019-10-02 2:28 ` Eli Zaretskii
2019-10-03 17:51 ` Glenn Morris
2019-10-04 18:48 ` Matthew Leach
2020-01-20 19:10 ` Stefan Kangas
2020-01-20 19:30 ` Eli Zaretskii
2020-01-21 0:58 ` Glenn Morris
2020-01-21 17:11 ` Eli Zaretskii
2020-01-21 18:10 ` Matthew Leach
2020-01-22 3:38 ` Glenn Morris
2020-01-22 15:46 ` Eli Zaretskii
2020-06-04 16:01 ` Glenn Morris
2020-08-15 12:37 ` Lars Ingebrigtsen
2020-01-24 18:08 ` Stefan Kangas
2020-01-21 3:04 ` Stefan Kangas
2019-10-04 18:47 ` Matthew Leach
2019-10-07 4:11 ` Lars Ingebrigtsen
2019-10-07 9:16 ` Andreas Schwab
2019-10-08 16:38 ` Lars Ingebrigtsen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=83y2y48f8u.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=37564@debbugs.gnu.org \
--cc=matthew@mattleach.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
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).