[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Make CVS HEAD version build out of the box
From: |
Eli Zaretskii |
Subject: |
Re: Make CVS HEAD version build out of the box |
Date: |
Sun, 30 Sep 2007 18:05:43 +0200 |
> 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?