Re: Make CVS HEAD version build out of the box

[Eli Zaretskii]

> Date: Sun, 30 Sep 2007 17:28:02 +0800
> From: "Yongwei Wu" <address@hidden>
> Cc: address@hidden
> 
> I was really guessing my modification might not be right.  However, I
> do not know how to build Make from CVS, and have not found
> documentation.
> 
> Any simple guidelines?  I have access to Cygwin, native MinGW, and
> MSVC (7.1).

With Cygwin, the same guidelines as for Posix (Unix and GNU/Linux)
systems should work (see README.cvs).  However, the result will be a
Cygwin build of Make, not a native Windows port.  The Cygwin build is
recommended for use only with Cygwin tools, and MinGW/MSVC build is
recommended for use only with native Windows development tools.

With MinGW and MSVC, running build_w32.bat should work if you add the
line that produces config.h from config.h.W32.template.  But make sure
you edit template strings such as %VERSION% into something suitable
before you run the compiler.

> > > @@ -471,7 +471,7 @@ find_directory (const char *name)
> > >        /* Put back the trailing '\'.  If we don't, we're permanently
> > >           truncating the value!  */
> > >        if (p[-1] == '\0')
> > > -        p[-1] = '\\';
> > > +        ((char *)p)[-1] = '\\';
> > >  #endif
> >
> > I don't understand why you needed these changes.  Which compiler did
> > you use, and what error message(s) it emitted, that require such
> > changes?
> 
> In the CVS version, "name" and "p" are now declared const char *.  So
> any decent compiler should complain.

Sorry, I missed that.  However, as Paul points out, your change is not
correct.  Can you please suggest a change that doesn't try to modify
const variables?