Bug 71246

Summary:	kdeutils/kdepasswd/passwd.cpp lacks #include -> doesn't compile on Solaris
Product:	[Unmaintained] configure	Reporter:	Torsten Kasch <tk>
Component:	general	Assignee:	Stephan Kulow <coolo>
Status:	RESOLVED FIXED
Severity:	normal	CC:	drutitsky
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Solaris
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	patch for kdeutils/kdepasswd/passwd.cpp

Description Torsten Kasch 2003-12-26 09:36:15 UTC

Version:            (using KDE Devel)
Installed from:    Compiled sources
Compiler:          gcc version 2.95.3 20010315 (release) 
OS:          Solaris

Trying to compile kdeutils on my Solaris 8 x86 box fails with

source='passwd.cpp' object='passwd.lo' libtool=yes \
depfile='.deps/passwd.Plo' tmpdepfile='.deps/passwd.TPlo' \
depmode=gcc /bin/ksh ../admin/depcomp \
/bin/ksh ../libtool --silent --mode=compile --tag=CXX g++ -pipe -DHAVE_CONFIG_H -I. -I. -I.. -I/vol/kde-cvs/include -I/vol/qt-3.2.3/include   -I/vol/local/include -I/vol/graphics/include  -DQT_THREAD_SUPPORT  -D_REENTRANT -D_POSIX_PTHREAD_SEMANTICS -DUSE_SOLARIS -DSVR4  -Wnon-virtual-dtor -Wno-long-long -Wundef -Wall -W -Wpointer-arith -Wwrite-strings -DNDEBUG -DNO_DEBUG -O2 -fno-exceptions -fno-check-new -fno-common -DQT_CLEAN_NAMESPACE -DQT_NO_ASCII_CAST -DQT_NO_STL -DQT_NO_COMPAT -DQT_NO_TRANSLATION  -c -o passwd.lo `test -f 'passwd.cpp' || echo './'`passwd.cpp
passwd.cpp: In method `int PasswdProcess::exec(const char *, const char *, int = 0)':
passwd.cpp:81: implicit declaration of function `int setenv(...)'
make[3]: *** [passwd.lo] Error 1
make[3]: Leaving directory `/vol/src/kde/cvs/kdeutils/kdepasswd'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/vol/src/kde/cvs/kdeutils/kdepasswd'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/vol/src/kde/cvs/kdeutils'
make: *** [all] Error 2

Comment 1 Torsten Kasch 2003-12-26 09:40:06 UTC

Created attachment 3841 [details]
patch for kdeutils/kdepasswd/passwd.cpp

Applying the trivial patch which adds the missing #include fixed this problem.

Comment 2 Thiago Macieira 2003-12-26 14:14:40 UTC

Looks ok to me.

Comment 3 Dirk Mueller 2004-01-10 12:47:25 UTC

Subject: kdeutils/kdepasswd

CVS commit by mueller: 

fix compile
CCMAIL: 71246-done@bugs.kde.org


  M +3 -1      passwd.cpp   1.13


--- kdeutils/kdepasswd/passwd.cpp  #1.12:1.13
@@ -9,4 +9,6 @@
  */
 
+#include <config.h> // setenv
+
 #include <stdio.h>
 #include <stdlib.h>

Comment 4 Dirk Mueller 2004-01-10 12:55:38 UTC

*** Bug 72157 has been marked as a duplicate of this bug. ***