> On Fri, 2009-06-19 at 09:03 -0700, Eric House wrote:
> > The patch that follows was generated against
> > https://cegcc.svn.sourceforge.net/svnroot/cegcc/tags/cegcc-0.55.
> > Unfortunately, WinMo .exes produced from this tree don't work for me,
> 
> Why ?

I'm not sure.  It won't load on the simulator with a message something
like "some required component could not be found."  It reminded me of
some reports I've seen about problems building for 6.1.  I was
focussed on getting the windres problem fixed so I went back to
something that I knew worked.  Eventually I'll figure out if it's a
new problem and, if so, report it.

> > I'm not sure how to proceed with getting this fixed upstream.  The
> > whole mingw world seems behind on Linux.
> 
> Getting a bug fixed in binutils is not hard : just send the fix to the
> right mailing list (bug-binut...@gnu.org). Doing the diff against their
> CVS (instead of cegcc's SVN) would make their work easier.

Do you expect to pull from binutils trunk anytime soon?

Last I checked the tip of mingw didn't build on Linux, and discussions
on their boards suggest it's not a priority for them until the gcc
world settles down.  If I can't build, can I submit a patch that I
haven't been able to test even to see if it compiles?  (I tried
setting up cygwin to build on XP and got nowhere: I'm a Windows newbie
and don't have time to learn that OS.)

--Eric

> > Index: src/binutils/binutils/winduni.c
> > ===================================================================
> > --- src/binutils/binutils/winduni.c (revision 1340)
> > +++ src/binutils/binutils/winduni.c (working copy)
> > @@ -42,7 +42,7 @@
> >  #include "winduni.h"
> >  #include "safe-ctype.h"
> >  
> > -#if HAVE_ICONV_H
> > +#if HAVE_ICONV
> >  #include <iconv.h>
> >  #endif
> >  
> > @@ -616,7 +616,7 @@
> >      *length = len;
> >  }
> >  
> > -#ifdef HAVE_ICONV_H
> > +#ifdef HAVE_ICONV
> >  static int
> >  iconv_onechar (iconv_t cd, const char *s, char *d, int d_len, const char 
> > **n_s, char **n_d)
> >  {
> > @@ -625,7 +625,7 @@
> >    for (i = 1; i <= 32; i++)
> >      {
> >        char *tmp_d = d;
> > -      const char *tmp_s = s;
> > +      char *tmp_s = (char*)s;
> >        size_t ret;
> >        size_t s_left = (size_t) i;
> >        size_t d_left = (size_t) d_len;
> > @@ -652,7 +652,7 @@
> >      return NULL;
> >    return lim->iconv_name;
> >  }
> > -#endif /* HAVE_ICONV_H */
> > +#endif /* HAVE_ICONV */
> >  
> >  static rc_uint_type
> >  wind_MultiByteToWideChar (rc_uint_type cp, const char *mb,
> > @@ -666,7 +666,7 @@
> >    /* Convert to bytes. */
> >    ret *= sizeof (unichar);
> >  
> > -#elif defined (HAVE_ICONV_H)
> > +#elif defined (HAVE_ICONV)
> >    int first = 1;
> >    char tmp[32];
> >    char *p_tmp;
> > @@ -739,7 +739,7 @@
> >  
> >    ret = (rc_uint_type) WideCharToMultiByte (cp, 0, u, -1, mb, mb_len,
> >                                         NULL, & used_def);
> > -#elif defined (HAVE_ICONV_H)
> > +#elif defined (HAVE_ICONV)
> >    int first = 1;
> >    char tmp[32];
> >    char *p_tmp;
> 
> -- 
> Danny Backx ; danny.backx - at - scarlet.be ; http://danny.backx.info
> 

-- 
******************************************************************************
* From the desktop of: Eric House, xwo...@eehouse.org                        *
*       Crosswords 4.2 for MS Smartphone and PocketPC is OUT: xwords.sf.net  *
******************************************************************************

------------------------------------------------------------------------------
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
_______________________________________________
Cegcc-devel mailing list
Cegcc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cegcc-devel

Reply via email to