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

 
 

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

Submitted by:  Robert Millan <robertmh>
Submitted on:  Mon 27 Feb 2006 07:10:52 PM UTC  
 
Category: Build/InstallSeverity: 3 - Normal
Priority: 7 - HighStatus: Fixed
Privacy: PublicAssigned to: None
Open/Closed: ClosedRelease: Cur Dev Sources
Fixed Release: 4.1.0Planned Release: 4.1.0
Work Required: 0 - Hours

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

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

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 #3:

Addressed for 4.1.0 in 2cc00bf.

Micah Cowan <micahcowan>
 Spam posted by an anonymous
 Spam posted by an 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>
Project 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 File(s):
   
   
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
  • -unavailable- added by anthony11 (Posted a comment)
  • -unavailable- added by micahcowan (Updated the item)
  •  

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

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 08 Aug 2008 05:14:54 AM UTCmicahcowanStatusConfirmed=>Fixed
      Open/ClosedOpen=>Closed
      Fixed ReleaseNone=>4.1.0
    Mon 23 Jun 2008 09:02:57 PM UTCmicahcowanCategoryNone=>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
    Sun 15 Jun 2008 06:15:53 AM UTCmicahcowanStatusFixed=>Confirmed
    Wed 01 Nov 2006 05:58:51 AM UTCNoneAttached File-=>Added setenv_autoconf.patch, #11129
    Tue 02 May 2006 12:25:18 PM UTCmlsStatusNone=>Fixed

    Back to the top


    Powered by Savane 3.1-cleanup1