Hi Caijian,

Please see comments in line.

caijian guo - Sun Microsystems - Beijing China said the following on 
Thursday 26 February 2009 07:29 AM:
> Hi Paul,
>
> Thanks very much for your review
> It works well when I use gcc, but it failed when I used sun compiler 
> below :
>
> bash-3.2# ./configure CC=/ws/onnv-tools/teamware/bin/cc
> checking for a BSD-compatible install... /usr/bin/ginstall -c
> checking whether build environment is sane... yes
> checking for gawk... gawk
> checking whether make sets $(MAKE)... yes
> checking for style of include used by make... GNU
> checking for gcc... /ws/onnv-tools/teamware/bin/cc
> checking for C compiler default output file name...
> configure: error: C compiler cannot create executables
> See `config.log' for more details.
> bash-3.2# cat config.log
> ...
> ...
> configure:2372: checking for gcc
> configure:2399: result: /ws/onnv-tools/teamware/bin/cc
> configure:2637: checking for C compiler version
> configure:2644: /ws/onnv-tools/teamware/bin/cc --version >&5
> cc: Warning: option -- passed to ld
> usage: cc [ options] files. Use 'cc -flags' for details
> configure:2647: $? = 1
> configure:2654: /ws/onnv-tools/teamware/bin/cc -v >&5
> usage: cc [ options] files. Use 'cc -flags' for details
> configure:2657: $? = 1
> configure:2664: /ws/onnv-tools/teamware/bin/cc -V >&5
> cc: Sun C 5.5 Patch 112761-06 2004/01/13
> usage: cc [ options] files. Use 'cc -flags' for details
> configure:2667: $? = 1
> configure:2690: checking for C compiler default output file name
> configure:2717: /ws/onnv-tools/teamware/bin/cc -O2 -Wall conftest.c >&5
> cc: Warning: option -2 passed to ld
> cc: illegal option -Wall

cc compiler which is part of Sun Studio doesn't recognize '-02',' -Wall' 
flags. Hence the configure has failed.

So you would need to set appropriate CFLAGS before running configure.

Note that you could make use of the CFLAGS for Sun Studio 12 compilers 
set in usr/src/Makefile.master

Thanks,
Srirama

> configure:2720: $? = 1
> configure:2758: result:
> configure: failed program was:
> | /* confdefs.h. */
> | #define PACKAGE_NAME "bwm-ng"
> | #define PACKAGE_TARNAME "bwm-ng"
> | #define PACKAGE_VERSION "0.6"
> | #define PACKAGE_STRING "bwm-ng 0.6"
> | #define PACKAGE_BUGREPORT "bwmng at gropp.org"
> | #define PACKAGE "bwm-ng"
> | #define VERSION "0.6"
> | /* end confdefs.h. */
> |
> | int
> | main ()
> | {
> |
> | ;
> | return 0;
> | }
> configure:2765: error: C compiler cannot create executables
> See `config.log' for more details.
>
>
>
>
> bash-3.2# ./configure CC=/ws/onnv-tools/SUNWspro/SS11/bin/cc
> checking for a BSD-compatible install... /usr/bin/ginstall -c
> checking whether build environment is sane... yes
> checking for gawk... gawk
> checking whether make sets $(MAKE)... yes
> checking for style of include used by make... GNU
> checking for gcc... /ws/onnv-tools/SUNWspro/SS11/bin/cc
> checking for C compiler default output file name...
> configure: error: C compiler cannot create executables
> See `config.log' for more details.
> bash-3.2#
>
>
>
>
> Caijian
>
>
>
> ? 2009?02?25? 18:46, Paul Cunningham ??:
>> Caijian,
>>
>> This mainly looks good to me, see below for a couple of comments ...
>>
>> Paul
>>
>> caijian guo - Sun Microsystems - Beijing China wrote:
>>>
>>> I am porting "bwm-ng". It is a live network and disk io throughput 
>>> monitor.
>>> Obtains statistics data from "kstat"; Supports both ipv4 and v6; Works
>>> well on both x86 and sparc.
>>>
>>> Webrev at:
>>> http://cr.opensolaris.org/~caijian/bwm-ng/
>>
>> === Start of Comments ===
>>
>> 1. usr/src/cmd/bwm-ng/METADATA
>> The SRC: line should probably be ...
>> SRC: http://www.gropp.org/bwm-ng/bwm-ng-0.6.tar.gz
>
>
> Accept
>
>
>
>>
>> 2. usr/src/cmd/bwm-ng/Makefile.sfw
>> I see you are using gcc to build this: Will it build with
>> the Sun compilers, if so can you use that instead (as
>> preferred compiler)?
>>
>> === End of Comments =====
>
> _______________________________________________
> sfwnv-discuss mailing list
> sfwnv-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/sfwnv-discuss

Reply via email to