bugGNU Screen - Bugs: bug #9139, configure fails to detect solaris...

 
 

bug #9139: configure fails to detect solaris correctly

Submitted by:  Juergen Weigert <jnweiger>
Submitted on:  Fri 28 May 2004 01:01:01 PM UTC  
 
Category: Build/InstallSeverity: 3 - Normal
Priority: 5 - NormalStatus: Duplicate
Privacy: PublicAssigned to: Michael Schröder <mls>
Open/Closed: ClosedRelease: None
Fixed Release: NonePlanned Release: None
Work Required: None

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Mon 23 Jun 2008 08:56:17 PM UTC, comment #4:

Bug itself is claimed by jw (in comments) to have been fixed; the 2-arg/3-arg setenv patch may be found in bug 15921, so marking this as dupe.

Micah Cowan <micahcowan>
 Spam posted by an anonymous
 Spam posted by an anonymous
 Spam posted by an anonymous
Thu 13 Dec 2007 03:11:07 PM UTC, comment #3:

Can you all make a new release? It seems a bit extreme to have screen on Solaris broken for three and a half years.

Anonymous
Wed 01 Nov 2006 05:56:26 AM UTC, comment #2:

Attaching a proper (I hope) autoconf patch to stop all this ifdef hacking.

Anonymous
Tue 20 Dec 2005 06:51:36 PM UTC, comment #1:

patch applied in cvs. current cvs HEAD fixes most of your issues differently.

Juergen Weigert <jnweiger>
Project Administrator
Fri 28 May 2004 01:01:01 PM UTC, original submission:

From: Michael Schloh von Bennewitz <michael.schloh@cw.com>
In screen-4.0.2, configure attempts to detect if it is being run on a
Solaris platform. This test fails, because it checks for a 'SVR4' identifier
at buildtime. Compilers on Solaris consistently define '__sun' and '__SVR4'
identifiers, so it's probably best to try to compile a small program using
those. The configure.in distributed with screen comes close, but not
close enough (see attached patch).

A more important problem involves a call to 'setenv(3)' on platforms which
the configure script has determined to have a 'setenv(3)' system call
available. Newer versions of Solaris (like Solaris 10) do have setenv(3) and
configure thus detects it. Unfortunately setenv(3) will be called with two
parameters, although the setenv(3) distributed with Solaris 10 specifies
three parameters. My patch fixes this by checking for the presence of
Solaris before calling setenv(3), just as already done when checking for the
presence of Linux. Here's my environment, and how I'm building screen on
around 20 different platforms:

Compiler: GCC 3.4.0
CPPFLAGS: None
CFLAGS: -O2 -pipe
Make: GNU Make 3.80
LDFLAGS: None

$ ./configure --prefix=/opkg --with-sys-screenrc=/opkg/etc/screen/screenrc --enable-colors256 --disable
-socket-dir --disable-locale
[...]
$make -j4
[...]

By the way, the patch I've attached only corrects configure.in and misc.c
(the source containing the problem reference to setenv(3)). Should you
choose to integrate these changes to a future distribution you'll need to
run autoconf of course, to regenerate the configure script which will only
then have the corresponding (and correct) tests for __SVR4 and __sun.

Our comprehensive patch for screen (with 'configure' as well) is available

http://cvs.openpkg.org/rlog?f=openpkg-src/screen/screen.patch

Juergen Weigert <jnweiger>
Project Administrator

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #11127:  setenv_autoconf.patch added by None (2KiB - application/octet-stream - Patch to use autoconf to detect 3 arg vs. 2 arg setenv)
file #1357:  screen-402-openpkg.diff added by jnweiger (6KiB - text/x-diff - patch from the above url)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by micahcowan (Posted a comment)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 11 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 23 Jun 2008 08:56:17 PM UTCmicahcowanCategoryNone=>Build/Install
      Severity1 - Wish=>3 - Normal
      StatusIn Progress=>Duplicate
      Open/ClosedOpen=>Closed
    Wed 01 Nov 2006 05:56:26 AM UTCNoneAttached File-=>Added setenv_autoconf.patch, #11127
    Tue 20 Dec 2005 06:51:36 PM UTCjnweigerStatusNone=>In Progress
      Assigned toscreen=>mls
    Fri 28 May 2004 01:25:52 PM UTCjnweigerAssigned toNone=>screen
      Carbon-Copy-=>Added michael --DOT-- schloh --AT-- cw --DOT-- com
    Fri 28 May 2004 01:01:01 PM UTCjnweigerAttached File-=>Added screen-402-openpkg.diff, #1349
      Carbon-Copy-=>Added screen --AT-- uni-erlangen --DOT-- de

    Back to the top


    Powered by Savane 3.1-cleanup1