bugGNU Screen - Bugs: bug #15921, [PATCH] 2-arg/3-arg setenv checking

 
 

bug #15921: [PATCH] 2-arg/3-arg setenv checking

Submitter:  Robert Millan <robertmh>
Submitted:  Mon 27 Feb 2006 07:10:52 PM UTC
   
 
Category:  Build/Install Severity:  3 - Normal
Priority:  * 7 - High Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed Release:  Cur Dev Sources
Fixed Release:  4.1.0 Planned Release:  4.1.0
Work Required:  0 - Hours
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Tue 04 May 2010 04:30:47 PM UTC, comment #6: 

That was over a year and a half ago.  When if ever will 4.1.0 be released?

Anthony D'Atri <anthony11>
Fri 08 Aug 2008 05:14:54 AM UTC, comment #5: 

Addressed for 4.1.0 in 2cc00bf.

Micah Cowan <micahcowan>
  Spam posted by anonymous
  Spam posted by anonymous
Wed 01 Nov 2006 05:58:51 AM UTC, comment #2: 

Here's a patch to Do The Right Thing and use autoconf to detect the number of setenv() arguments.

Anonymous
Tue 02 May 2006 12:25:18 PM UTC, comment #1: 

Applied, thanks!

Michael Schröder <mls>
Group administrator
Mon 27 Feb 2006 07:10:52 PM UTC, original submission:  


Fails to build on GNU/kFreeBSD.  The fix is simple:

Index: src/misc.c
===================================================================
RCS file: /sources/screen/screen/src/misc.c,v
retrieving revision 1.24
diff -u -r1.24 misc.c
--- src/misc.c  19 Dec 2005 20:20:47 -0000      1.24
+++ src/misc.c  27 Feb 2006 19:10:15 -0000
@@ -613,7 +613,7 @@
    */
 # endif /* NEEDSETENV */
 #else /* USESETENV */
-# if defined(linux) || defined(_convex_) || (BSD >= 199103) || defined(_AIX)
+# if defined(linux) || defined(_convex_) || (BSD >= 199103) || defined(AIX) || defined(__GLIBC_)
   setenv(var, value, 1);
 # else
   setenv(var, value);

Robert Millan <robertmh>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #11129:  setenv_autoconf.patch added by None (2KiB - application/octet-stream - Use autoconf to detech the number of arguments to setenv())

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    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 12 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2008-08-08 micahcowan StatusConfirmed Fixed
        Open/ClosedOpen Closed
        Fixed ReleaseNone 4.1.0
    2008-06-23 micahcowan CategoryNone Build/Install
        Priority5 - Normal 7 - High
        ReleaseNone Cur Dev Sources
        Planned ReleaseNone 4.1.0
        Work RequiredNone 0 - Hours
        Summary[PATCH] Fails to build on GNU/kFreeBSD [PATCH] 2-arg/3-arg setenv checking
    2008-06-15 micahcowan StatusFixed Confirmed
    2006-11-01 None Attached File- Added setenv_autoconf.patch, #11129
    2006-05-02 mls StatusNone Fixed

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code