Summary: | app-misc/screen imposes strange limit on the length of $TERM | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Stelian Ionescu <bugs> |
Component: | [OLD] Unspecified | Assignee: | Sven Wegener <swegener> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | ainsaar, pchrist, shell-tools |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://savannah.gnu.org/bugs/?30880 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
Raise the length of d_termnane to NAME_MAX
Patch better suited for upstream, should compile on BSDs too Fix typo in last patch Fix hardcoded size of TERM in another place as well |
Description
Stelian Ionescu
2010-11-17 11:03:25 UTC
Created attachment 254617 [details, diff]
Raise the length of d_termnane to NAME_MAX
Comment on attachment 254617 [details, diff]
Raise the length of d_termnane to NAME_MAX
If strlen(getenv("TERM)) > sizeof(d_termname), screen refuses to start. I've rebuilt screen with that patch and it seems to work
Stelian, thanks for your report. This should go upstream too. Created attachment 254625 [details, diff]
Patch better suited for upstream, should compile on BSDs too
Created attachment 254627 [details, diff]
Fix typo in last patch
Might be similar to: http://savannah.gnu.org/bugs/?30880 +*screen-4.0.3-r2 (17 Nov 2010) + + 17 Nov 2010; Justin Lecher <jlec@gentoo.org> + +files/4.0.3-extend-d_termname.patch, screen-4.0.3.ebuild, + +screen-4.0.3-r2.ebuild: + Fix for long names in TERM, thanks Stelian Ionescu #345863 + Created attachment 254669 [details, diff]
Fix hardcoded size of TERM in another place as well
As noticed in the Savannah bug, there was another place in the source code where the terminal name was being copied, so I updated that too
Sorry for missing that one Is this fixed now? Yes, works here Thanks closing then. |