Re: [sdcc-devel] Cygming configure fails

[Borut R. <bor...@si...>]

Raphael Neider wrote:
> Hi Borut,
>
>   
>> Cygming build (cygwin + -mno-nocygwin) fails with the following error:
>>     
>
> [snip]
>
>   
>> If I remove --build=`"$SRCDIR/config.guess"` --host=pic16 configure
>> options from device/lib/pic16/configure.gnu, everything is OK.
>>
>> On the other hand, Raphael wrote in device/lib/pic16/configure.ac:
>>
>> # The (generated) configure script should be called with
>> #     --build=<your-build-system> --target=pic16
>> # to force cross-compilation mode; rather use the wrapper
>> # ./configure.gnu instead of ./configure directly.
>>
>> Raphael, do you have an idea how to solve the problem? Are
>> --build=`"$SRCDIR/config.guess"` and --host=pic16 configure options
>> really needed?
>>     
>
> I feel this is a remainder of previous attempts to make autoconf succeed  
> using AC_PROG_CC(sdcc) instead the manual approach taken now. I removed  
> configure.gnu and the obsolete comment in configure.ac in r5433.
>
>
>
> I have another build problem concerning small-stack-auto/:
>
> make[4]: Entering directory  
> `/home/neider/NO_BACKUP/private/git/build/device/lib/small-stack-auto'
> make -C ../mcs51 all
> make[5]: Entering directory  
> `/home/neider/NO_BACKUP/private/git/build/device/lib/mcs51'
> ar -Scq build/small-stack-auto/mcs51.lib crtstart.rel crtxinit.rel  
> crtxclear.rel crtclear.rel crtxstack.rel crtpagesfr.rel crtbank.rel  
> crtcall.rel
> ar: build/small-stack-auto/mcs51.lib: No such file or directory
> make[5]: *** [build/small-stack-auto/mcs51.lib] Error 1
> make[5]: Leaving directory  
> `/home/neider/NO_BACKUP/private/git/build/device/lib/mcs51'
> make[4]: *** [all] Error 2
> make[4]: Leaving directory  
> `/home/neider/NO_BACKUP/private/git/build/device/lib/small-stack-auto'
> make[3]: [port-specific-objects] Error 2 (ignored)
> make[3]: Leaving directory  
> `/home/neider/NO_BACKUP/private/git/build/device/lib'
>
> It seems ar tries to write to
>
> /home/neider/NO_BACKUP/private/git/build/device/lib/mcs51/build-small-stack-auto/mcs51.lib
>
> while the directory that is created for its use is
>
> /home/neider/NO_BACKUP/private/git/build/device/lib/mcs51/../build-small-stack-auto/mcs51.lib
>
> It seems the PORTDIR variable is inherited from the calling make rather  
> than be overwritten by the small-stack-auto/Makefile.
>
>
>   
... now I know why I introduced abspath... Now I have to find an other 
solution :-(

> ... and there are a lot of (relatively new?) warnings such as
>
> ../../../device/lib/_isspace.c:29: warning 185: comparison of 'signed  
> char' with 'unsigned char' requires promotion to int
>
>   

I noticed that too, but I don't know which change triggered it.

> which should probably be addressed; either by using 'char isXXX(char c)'  
> or by casting all character literals to (unsigned char)'x'. The first  
> option probably introduces overhead for less/greater-than comparisons (c  
>   
>> = '0' && c <= '9') due to sign handling, the second option is just ugly  
>>     
> ;-) and more work.
> I will 'fix' the PIC libraries later.
>
> Best regards,
> Raphael
>
>   
Borut