X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C7E5BC.259139C0@onstor-exch02.onstor.net>; Thu, 23 Aug 2007 11:31:05 -0800
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----_=_NextPart_001_01C7E5BC.259139C0"
References: <BB375AF679D4A34E9CA8DFA650E2B04E02F3D698@onstor-exch02.onstor.net> <BB375AF679D4A34E9CA8DFA650E2B04E02F3D69B@onstor-exch02.onstor.net> <BB375AF679D4A34E9CA8DFA650E2B04E051AED2B@onstor-exch02.onstor.net>
Content-class: urn:content-classes:message
Subject: RE: Help, please
Date: Thu, 23 Aug 2007 11:30:08 -0800
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E02F3D6AC@onstor-exch02.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: Help, please
Thread-Index: AcflCRD7wk9o8wOFQ6iviaZj9xSgYQABZ2CCAAGawqAAKbqVSQ==
From: "Ken Renshaw" <ken.renshaw@onstor.com>
To: "Brian DeForest" <brian.deforest@onstor.com>,
	"Ed Kwan" <ed.kwan@onstor.com>,
	"Ron Bhanukitsiri" <ronb@onstor.com>
Cc: "Jeff Miller" <jeff.miller@onstor.com>,
	"Andy Sharp" <andy.sharp@onstor.com>

This is a multi-part message in MIME format.

------_=_NextPart_001_01C7E5BC.259139C0
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable

Thanks Brian, that seems to have done the trick in getting the cg =
compiles working again.

I'll check in the fix to authen-lib.c as soon as I finish test-compiling =
the other variants.

-Ken


-----Original Message-----
From: Brian DeForest
Sent: Wed 8/22/2007 4:37 PM
To: Ken Renshaw; Ed Kwan; Ron Bhanukitsiri
Cc: Jeff Miller
Subject: RE: Help, please
=20
Looks like something subtle.  Perhaps these lines are needed?  =20

1,2d0
<
< #define _GNU_SOURCE
40,41d37
< #include "../ssc-bsdcompat/bsdcompat.h"
<

-----Original Message-----
From: Ken Renshaw=20
Sent: Wednesday, August 22, 2007 3:49 PM
To: Ken Renshaw; Ed Kwan; Brian DeForest; Ron Bhanukitsiri
Subject: RE: Help, please

If you just check out the top of tree and do this is shows right up:

[build@k2 ssc-authen-lib]$ make PROD=3Dcg ssc
compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/privs-cfg.d

compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/idmap-cfg.d

compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/dom-cfg.d

compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/cifs-map-cfg.d

compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/authen-localaccounts-api.d

compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/admin-session.d

compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/authen-lib.d

make[1]: Entering directory =
`/perforce/trees/dev/nfx-tree/code/ssc-authen-lib'
/usr/bin/mipsel-linux-gnu-gcc-4.1 -fPIC -fms-extensions =
-Wno-pointer-sign -W -Wno-sign-compare -Werror -DEXTENDED_SECURITY =
-DNFX_KERBEROS -DNETEEE_FRAGMENT  -G 0  -DONSTOR_CHANGE =
-DLDAP_DEPRECATED -DOS_INCL=3D\"linux.h\"  -I./linux =
-I../../Includes/cg/SSC -I../../Includes/cg -I../../Includes/linux  =
-I../../Includes -I. -I../../Build/cg/opt -DLANGUAGE_C -DLINUX_TEST    =
-g -DSSC -DSSC_mips -DNFX_MOD_SSC -c authen-lib.c -o =
../../Build/cg/opt/Objects/SSC/ssc-authen-lib/authen-lib.o
cc1: warnings being treated as errors
authen-lib.c: In function =E2authen_getAdsStatus=E2:
authen-lib.c:2690: warning: assignment makes pointer from integer =
without a cast
make[1]: *** =
[../../Build/cg/opt/Objects/SSC/ssc-authen-lib/authen-lib.o] Error 1
make[1]: Leaving directory =
`/perforce/trees/dev/nfx-tree/code/ssc-authen-lib'
make: *** [ssc] Error 2
[build@k2 ssc-authen-lib]$



-----Original Message-----
From: Ken Renshaw
Sent: Wed 8/22/2007 3:46 PM
To: Ed Kwan; Brian DeForest; Ron Bhanukitsiri
Subject: Help, please
=20
Hi guys, the merge of 3.0.1 into dev has a problem in authen-lib.c.  All =
the zonda targets build fine, but I neglected to build all the linux and =
cougar targets in my testing and of course this change 24968 to =
authen-lib.c breaks the cougar SSC build with this error:

> authen-lib.c: In function 'authen_getAdsStatus':
> authen-lib.c:2690: warning: assignment makes pointer from integer
> without a cast

The cougar build is currently broken due to this, so if one of you C =
programmers would be so kind as to help fix this I'd be much obliged. =
The change that has Ron's original fix in it  is P4 24968 and if you =
check out the top of //depot/dev/nfx-tree/... you'll have the code. The =
diff for the merge is just this, so offhand I don't see the error:

[build@k2 nfx-tree]$ p4 diff2 -dw =
//depot/dev/nfx-tree/code/ssc-authen-lib/authen-lib.c#2 //depot
=3D=3D=3D=3D //depot/dev/nfx-tree/code/ssc-authen-lib/authen-lib.c#2 =
(text) - //depot/dev/nfx-tree/code/s
1,2d0
<
< #define _GNU_SOURCE
40,41d37
< #include "../ssc-bsdcompat/bsdcompat.h"
<
571c567
<     for(i=3D0; i < sid->nbSubAuthy; i++) {
---
>     for(i=3D0; i < MIN(sid->nbSubAuthy, SID_MAX_SUB_AUTHORITIES); i++) =
{
2879,2880d2874
<     char    shDomainAdmin[(SEC_DOMAIN_MAX + SEC_NAME_MAX) * 2];
<     char    shDomainPwd[SEC_PWD_MAX * 2];
2883,2885d2876
<     authen_conformShellString(domainAdmin, shDomainAdmin, =
sizeof(shDomainAdmin));
<     authen_conformShellString(domainPasswd, shDomainPwd, =
sizeof(shDomainPwd));
<
2913,2914c2904,2905
<     AUTH_SET_CMD_ARG2(cmdargs, i, "-U", shDomainAdmin);
<     AUTH_SET_CMD_ARG2(cmdargs, i, "-a", shDomainPwd);
---
>     AUTH_SET_CMD_ARG2(cmdargs, i, "-U", domainAdmin);
>     AUTH_SET_CMD_ARG2(cmdargs, i, "-a", domainPasswd);


Could one of you please help fix this?

Thanks,

-Ken




------_=_NextPart_001_01C7E5BC.259139C0
Content-Type: text/html;
	charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV=3D"Content-Type" CONTENT=3D"text/html; =
charset=3Diso-8859-1">
<META NAME=3D"Generator" CONTENT=3D"MS Exchange Server version =
6.5.7652.24">
<TITLE>RE: Help, please</TITLE>
</HEAD>
<BODY>
<!-- Converted from text/plain format -->

<P><FONT SIZE=3D2>Thanks Brian, that seems to have done the trick in =
getting the cg compiles working again.<BR>
<BR>
I'll check in the fix to authen-lib.c as soon as I finish test-compiling =
the other variants.<BR>
<BR>
-Ken<BR>
<BR>
<BR>
-----Original Message-----<BR>
From: Brian DeForest<BR>
Sent: Wed 8/22/2007 4:37 PM<BR>
To: Ken Renshaw; Ed Kwan; Ron Bhanukitsiri<BR>
Cc: Jeff Miller<BR>
Subject: RE: Help, please<BR>
<BR>
Looks like something subtle.&nbsp; Perhaps these lines are =
needed?&nbsp;&nbsp;<BR>
<BR>
1,2d0<BR>
&lt;<BR>
&lt; #define _GNU_SOURCE<BR>
40,41d37<BR>
&lt; #include &quot;../ssc-bsdcompat/bsdcompat.h&quot;<BR>
&lt;<BR>
<BR>
-----Original Message-----<BR>
From: Ken Renshaw<BR>
Sent: Wednesday, August 22, 2007 3:49 PM<BR>
To: Ken Renshaw; Ed Kwan; Brian DeForest; Ron Bhanukitsiri<BR>
Subject: RE: Help, please<BR>
<BR>
If you just check out the top of tree and do this is shows right up:<BR>
<BR>
[build@k2 ssc-authen-lib]$ make PROD=3Dcg ssc<BR>
compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/privs-cfg.d<BR>
<BR>
compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/idmap-cfg.d<BR>
<BR>
compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/dom-cfg.d<BR>
<BR>
compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/cifs-map-cfg.d<BR>
<BR>
compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/authen-localaccounts-api.d<=
BR>
<BR>
compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/admin-session.d<BR>
<BR>
compiling depend file =
../../Build/cg/opt/Depends/SSC/ssc-authen-lib/authen-lib.d<BR>
<BR>
make[1]: Entering directory =
`/perforce/trees/dev/nfx-tree/code/ssc-authen-lib'<BR>
/usr/bin/mipsel-linux-gnu-gcc-4.1 -fPIC -fms-extensions =
-Wno-pointer-sign -W -Wno-sign-compare -Werror -DEXTENDED_SECURITY =
-DNFX_KERBEROS -DNETEEE_FRAGMENT&nbsp; -G 0&nbsp; -DONSTOR_CHANGE =
-DLDAP_DEPRECATED -DOS_INCL=3D\&quot;linux.h\&quot;&nbsp; -I./linux =
-I../../Includes/cg/SSC -I../../Includes/cg -I../../Includes/linux&nbsp; =
-I../../Includes -I. -I../../Build/cg/opt -DLANGUAGE_C =
-DLINUX_TEST&nbsp;&nbsp;&nbsp; -g -DSSC -DSSC_mips -DNFX_MOD_SSC -c =
authen-lib.c -o =
../../Build/cg/opt/Objects/SSC/ssc-authen-lib/authen-lib.o<BR>
cc1: warnings being treated as errors<BR>
authen-lib.c: In function =E2authen_getAdsStatus=E2:<BR>
authen-lib.c:2690: warning: assignment makes pointer from integer =
without a cast<BR>
make[1]: *** =
[../../Build/cg/opt/Objects/SSC/ssc-authen-lib/authen-lib.o] Error 1<BR>
make[1]: Leaving directory =
`/perforce/trees/dev/nfx-tree/code/ssc-authen-lib'<BR>
make: *** [ssc] Error 2<BR>
[build@k2 ssc-authen-lib]$<BR>
<BR>
<BR>
<BR>
-----Original Message-----<BR>
From: Ken Renshaw<BR>
Sent: Wed 8/22/2007 3:46 PM<BR>
To: Ed Kwan; Brian DeForest; Ron Bhanukitsiri<BR>
Subject: Help, please<BR>
<BR>
Hi guys, the merge of 3.0.1 into dev has a problem in =
authen-lib.c.&nbsp; All the zonda targets build fine, but I neglected to =
build all the linux and cougar targets in my testing and of course this =
change 24968 to authen-lib.c breaks the cougar SSC build with this =
error:<BR>
<BR>
&gt; authen-lib.c: In function 'authen_getAdsStatus':<BR>
&gt; authen-lib.c:2690: warning: assignment makes pointer from =
integer<BR>
&gt; without a cast<BR>
<BR>
The cougar build is currently broken due to this, so if one of you C =
programmers would be so kind as to help fix this I'd be much obliged. =
The change that has Ron's original fix in it&nbsp; is P4 24968 and if =
you check out the top of //depot/dev/nfx-tree/... you'll have the code. =
The diff for the merge is just this, so offhand I don't see the =
error:<BR>
<BR>
[build@k2 nfx-tree]$ p4 diff2 -dw =
//depot/dev/nfx-tree/code/ssc-authen-lib/authen-lib.c#2 //depot<BR>
=3D=3D=3D=3D //depot/dev/nfx-tree/code/ssc-authen-lib/authen-lib.c#2 =
(text) - //depot/dev/nfx-tree/code/s<BR>
1,2d0<BR>
&lt;<BR>
&lt; #define _GNU_SOURCE<BR>
40,41d37<BR>
&lt; #include &quot;../ssc-bsdcompat/bsdcompat.h&quot;<BR>
&lt;<BR>
571c567<BR>
&lt;&nbsp;&nbsp;&nbsp;&nbsp; for(i=3D0; i &lt; sid-&gt;nbSubAuthy; i++) =
{<BR>
---<BR>
&gt;&nbsp;&nbsp;&nbsp;&nbsp; for(i=3D0; i &lt; MIN(sid-&gt;nbSubAuthy, =
SID_MAX_SUB_AUTHORITIES); i++) {<BR>
2879,2880d2874<BR>
&lt;&nbsp;&nbsp;&nbsp;&nbsp; char&nbsp;&nbsp;&nbsp; =
shDomainAdmin[(SEC_DOMAIN_MAX + SEC_NAME_MAX) * 2];<BR>
&lt;&nbsp;&nbsp;&nbsp;&nbsp; char&nbsp;&nbsp;&nbsp; =
shDomainPwd[SEC_PWD_MAX * 2];<BR>
2883,2885d2876<BR>
&lt;&nbsp;&nbsp;&nbsp;&nbsp; authen_conformShellString(domainAdmin, =
shDomainAdmin, sizeof(shDomainAdmin));<BR>
&lt;&nbsp;&nbsp;&nbsp;&nbsp; authen_conformShellString(domainPasswd, =
shDomainPwd, sizeof(shDomainPwd));<BR>
&lt;<BR>
2913,2914c2904,2905<BR>
&lt;&nbsp;&nbsp;&nbsp;&nbsp; AUTH_SET_CMD_ARG2(cmdargs, i, =
&quot;-U&quot;, shDomainAdmin);<BR>
&lt;&nbsp;&nbsp;&nbsp;&nbsp; AUTH_SET_CMD_ARG2(cmdargs, i, =
&quot;-a&quot;, shDomainPwd);<BR>
---<BR>
&gt;&nbsp;&nbsp;&nbsp;&nbsp; AUTH_SET_CMD_ARG2(cmdargs, i, =
&quot;-U&quot;, domainAdmin);<BR>
&gt;&nbsp;&nbsp;&nbsp;&nbsp; AUTH_SET_CMD_ARG2(cmdargs, i, =
&quot;-a&quot;, domainPasswd);<BR>
<BR>
<BR>
Could one of you please help fix this?<BR>
<BR>
Thanks,<BR>
<BR>
-Ken<BR>
<BR>
<BR>
<BR>
</FONT>
</P>

</BODY>
</HTML>
------_=_NextPart_001_01C7E5BC.259139C0--
