configure: Initial support for --target-os=symbian

Message ID 1301519067-14660-1-git-send-email-martin@martin.st
State Committed
Headers show

Commit Message

Martin Storsjö March 30, 2011, 9:04 p.m.
---
 configure |    9 ++++++++-
 1 files changed, 8 insertions(+), 1 deletions(-)

Comments

Mans Rullgard March 30, 2011, 9:22 p.m. | #1
Martin Storsjö <martin@martin.st> writes:

> ---
>  configure |    9 ++++++++-
>  1 files changed, 8 insertions(+), 1 deletions(-)
>
> diff --git a/configure b/configure
> index fab4f2b..5e76a28 100755
> --- a/configure
> +++ b/configure
> @@ -1822,7 +1822,7 @@ set_default host_cc
>  
>  exesuf() {
>      case $1 in
> -        mingw32*|cygwin*|*-dos|freedos|opendos|os/2*) echo .exe ;;
> +        mingw32*|cygwin*|*-dos|freedos|opendos|os/2*|symbian) echo .exe ;;
>      esac
>  }
>  
> @@ -2480,6 +2480,13 @@ case $target_os in
>          add_cppflags -D_QNX_SOURCE
>          network_extralibs="-lsocket"
>          ;;
> +    symbian)
> +        SLIBSUF=".dll"
> +        enable dos_paths
> +        # This include flag is added here instead of in extra_cflags,
> +        # since it mustn't end up in asflags

I don't think that comment is necessary.

> +        add_cflags --include=$sysinclude/gcce/gcce.h
> +        ;;
>      none)
>          ;;
>      *)
> -- 

OK

Obvious question: can we run fate on that?
Martin Storsjö March 31, 2011, 12:49 p.m. | #2
On Wed, 30 Mar 2011, Måns Rullgård wrote:

> Martin Storsjö <martin@martin.st> writes:
> 
> > ---
> >  configure |    9 ++++++++-
> >  1 files changed, 8 insertions(+), 1 deletions(-)
> >
> > diff --git a/configure b/configure
> > index fab4f2b..5e76a28 100755
> > --- a/configure
> > +++ b/configure
> > @@ -1822,7 +1822,7 @@ set_default host_cc
> >  
> >  exesuf() {
> >      case $1 in
> > -        mingw32*|cygwin*|*-dos|freedos|opendos|os/2*) echo .exe ;;
> > +        mingw32*|cygwin*|*-dos|freedos|opendos|os/2*|symbian) echo .exe ;;
> >      esac
> >  }
> >  
> > @@ -2480,6 +2480,13 @@ case $target_os in
> >          add_cppflags -D_QNX_SOURCE
> >          network_extralibs="-lsocket"
> >          ;;
> > +    symbian)
> > +        SLIBSUF=".dll"
> > +        enable dos_paths
> > +        # This include flag is added here instead of in extra_cflags,
> > +        # since it mustn't end up in asflags
> 
> I don't think that comment is necessary.
> 
> > +        add_cflags --include=$sysinclude/gcce/gcce.h
> > +        ;;
> >      none)
> >          ;;
> >      *)
> > -- 
> 
> OK

Pushed with the comment removed.

> Obvious question: can we run fate on that?

Not without serious amounts of ugly hacking. I think FATE on Windows CE 
would be simpler to get up and running, and that's not particularly easy 
either. I'm not aware of either of them mounting a remote filesystem, for 
starters.

I can set up a compile-only FATE for this configuration, though, I already 
run one for Win CE, catching build failures at least.

// Martin
Mans Rullgard March 31, 2011, 12:52 p.m. | #3
Martin Storsjö <martin@martin.st> writes:

> On Wed, 30 Mar 2011, Måns Rullgård wrote:
>
>> Martin Storsjö <martin@martin.st> writes:
>> 
>> > +    symbian)
>
>> Obvious question: can we run fate on that?
>
> Not without serious amounts of ugly hacking. I think FATE on Windows CE 
> would be simpler to get up and running, and that's not particularly easy 
> either. I'm not aware of either of them mounting a remote filesystem, for 
> starters.

Never mind then.

Patch

diff --git a/configure b/configure
index fab4f2b..5e76a28 100755
--- a/configure
+++ b/configure
@@ -1822,7 +1822,7 @@  set_default host_cc
 
 exesuf() {
     case $1 in
-        mingw32*|cygwin*|*-dos|freedos|opendos|os/2*) echo .exe ;;
+        mingw32*|cygwin*|*-dos|freedos|opendos|os/2*|symbian) echo .exe ;;
     esac
 }
 
@@ -2480,6 +2480,13 @@  case $target_os in
         add_cppflags -D_QNX_SOURCE
         network_extralibs="-lsocket"
         ;;
+    symbian)
+        SLIBSUF=".dll"
+        enable dos_paths
+        # This include flag is added here instead of in extra_cflags,
+        # since it mustn't end up in asflags
+        add_cflags --include=$sysinclude/gcce/gcce.h
+        ;;
     none)
         ;;
     *)