[ldapvi] Portability issue: setenv missing on Solaris 8

[David Lichteblau]

Quoting Dagobert Michelsen (dam at blastwave.org):
>> do you have a patch for me or should I try to write a patch and send it
>> to you for testing?  I don't currently have access to Solaris, so I
>> wouldn't be able to test this myself.
>
> Sorry, no patch from me as my autoconf skills are limited.

Okay, I have pushed the following patch to the git archive.  Does that
work for you?

diff --git a/ldapvi/arguments.c b/ldapvi/arguments.c
index a6cbb8e..63e6213 100644
--- a/ldapvi/arguments.c
+++ b/ldapvi/arguments.c
@@ -724,7 +724,7 @@ parse_arguments(int argc, const char **argv, cmdline *result, GPtrArray *ctrls)
 	}
 
 	if (result->profileonlyp)
-		if (setenv("LDAPNOINIT", "thanks", 1)) syserr();
+		if (putenv("LDAPNOINIT=thanks")) syserr();
 
 	/* don't free! */
 /* 	poptFreeContext(ctx); */


> As it would be nice to have DESTDIR-support have you thought
> about including this patch?
>
>   <http://lists.askja.de/pipermail/ldapvi/2007-July/000031.html>

IIRC that's already in ldapvi's git archive at
http://www.lichteblau.com/git/ldapvi.git


d.