bugGNU Screen - Bugs: bug #50089, Screen 4.5.0 doesn't build on SunOS

 
 

bug #50089: Screen 4.5.0 doesn't build on SunOS

Submitter:  None
Submitted:  Thu 19 Jan 2017 10:06:30 AM UTC
   
 
Category:  Build/Install Severity:  3 - Normal
Priority:  * 5 - Normal Status:  Fixed
Privacy:  Public Assigned to:  anaumov
Open/Closed:  Closed Release:  4.5.0
Fixed Release:  4.5.1 Planned Release:  4.5.1
Work Required:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 13 Feb 2017 08:37:14 PM UTC, comment #2: 

OpenIndiana x86:
root@sun:~/screen/src# uname -a
SunOS sun 5.11 illumos-2816291 i86pc i386 i86pc

root@sun:~/screen/src# cat ~/test.c
#include<stdio.h>
#include<limits.h>
int main()
{
printf("PATH_MAX = %d\n", PATH_MAX);
return 0;
}

root@sun:~/screen/src# ~/test
PATH_MAX = 1024

I added limits.h to master and screen-v4 branch:
http://git.savannah.gnu.org/cgit/screen.git/commit/?h=screen-v4&id=6994c70aa180a5617fe8fdd563b7653d0d16fa2a
http://git.savannah.gnu.org/cgit/screen.git/commit/?id=59b88a8411366d512a2db2c10c39cc9b83fcb13b

Please test it.

Alexander Naumov <anaumov>
Group administrator
Thu 19 Jan 2017 01:04:30 PM UTC, comment #1: 

The same issue shows up in GNU Hurd, which also doesn't know PATH_MAX because it doesn't seem to need it, i.e. doesn't have a limit for a path length.

In Debian we apply this clumsy patch for GNU Hurd which sets an arbitrary limit if none is given:

https://anonscm.debian.org/cgit/collab-maint/screen.git/tree/debian/patches/61-default-PATH_MAX-if-undefined-for-hurd.patch

Of course it would be nice if this would be solved in a cleaner way for an official fix.

Axel Beckert <abe>
Thu 19 Jan 2017 10:06:30 AM UTC, original submission:  

Building on illumos (SunOS 5.11) fails with:

gcc -c -I. -I.  -D_XPG4_2 -D__EXTENSIONS__ -DETCSCREENRC='"/etc/screenrc"' -DSCREENENCODINGS='"/usr/share/screen/utf8encodings"' -DHAVE_CONFIG_H -DGIT_REV=\"\" \
     -O  screen.c
screen.c: In function 'main':
screen.c:674:43: error: 'PATH_MAX' undeclared (first use in this function)
               if (strlen(screenlogfile) > PATH_MAX)

The problem is that screen.c needs to #include <limits.h> (just like tty.c does, which is the other place PATH_MAX i s used)

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #39652:  screen_4.5.0_coredump.txt added by None (2KiB - text/plain - screen dumps core instantly on Solaris)
file #39650:  screen_4.5.0_hack.txt added by None (4KiB - text/plain - Make a few minor changes and then manually link other wise link stage fails)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by anaumov (Updated the item)
  • -email is unavailable- added by abe (Posted a comment)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 9 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-02-22 anaumov Open/ClosedOpen Closed
    2017-02-22 anaumov StatusReady For Test Fixed
    2017-02-13 anaumov StatusNone Ready For Test
        Release5.0.0 4.5.0
        Fixed ReleaseNone 4.5.1
        Planned ReleaseNone 4.5.1
    2017-02-03 None Attached File- Added screen_4.5.0_coredump.txt, #39652
    2017-02-02 None Attached File- Added screen_4.5.0_hack.txt, #39650
    2017-01-20 anaumov Assigned toNone anaumov

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code