[Skyeye-developer] Makefile problems and proposed modification

[Anthony Lee]

Hi Wang,

My sugesstion is adding the following lines to the front of "Makefile".
IMHO, it's useful to try cross-compile manually and test whether
the changes provide SkyEye to work on other platforms by setting
$(OSTYPE) instead of finding from the output of "uname".


####################################################
ifndef OSTYPE

ifeq ($(findstring CYGWIN,$(shell uname -m)),CYGWIN)
OSTYPE=cygwin
TERM=cygwin
endif

ifeq ($(findstring MINGW32,$(shell uname -m)),MINGW32)
OSTYPE=msys
endif

endif
###################################################


Best Regards.
Lee


2007/4/2, Yonghao Wang <wyonghao at gmail.com>:
> Hi All
>
> I checked out latest trunk of skyeye's makefile. I use cygwin/winxp, however
> there are problems with getting environments variables. For example: Anthony
> Lee's change of makefile to check system environment variables and add bfd
> library support:
>
> BFD_LIBS =
>
> ifdef NO_BFD
>     CFLAGS += -DNO_BFD
> else
>     # Anthony Lee 2006-09-18 : for getopt/bfd on Cygwin
>     # BFD_LIBS -------------------------------- START
>     ifneq ($(OSTYPE),msys)
>      ifeq ($(TERM),cygwin)
>     BFD_LIBS += -lc
>     endif
>     endif
>
>     BFD_LIBS += -lbfd -liberty
>
>     ifneq ($(OSTYPE),msys)
>     ifeq ($(TERM),cygwin)
>     BFD_LIBS += -lintl
>     endif
>     endif
>     # BFD_LIBS -------------------------------- END
> endif
>
> But the problem is that current bash doesn't automatically export env
> variables such as OSTYPE, TERM etc, so my makefile return from $(TERM) and
> $(OSTYPE) are always empty. Therefore the cygwin mailing list suggest you
> should use 'uname' instead of OSTYPE. My changes to my working copy are
> adding:
>
> OSTYPE = $(shell uname -msr)
>
> and replace ALL
>         ifeq ($(TERM),cygwin)
> with
> ifeq ($(findstring CYGWIN,$(OSTYPE)),CYGWIN)
>
> Now it checks everything on my cygwin/winxp OK and make without any problem.
> I suppose we also need to replace
>     ifeq ($(OSTYPE),msys)
> with
> ifeq ($(findstring MINGW32,$(OSTYPE)),MINGW32)
>
> But I don't know about uname value of beos so
>     ifeq ($(OSTYPE),beos)
> should be replaced with something as well.
>
> So if you have those system and test for us, we can have a working patch.
> You suggestions are welcome.
>
> Y.Wang
>
> _______________________________________________
> Skyeye-developer mailing list
> Skyeye-developer at lists.gro.clinux.org
> http://lists.gro.clinux.org/cgi-bin/mailman/listinfo/skyeye-developer
>
>