[GRASS5] Re: [GRASSLIST:1363] Re: compiling grass57

[Paul Kelly]

On Fri, 26 Sep 2003, Paul Kelly wrote:

> On Sat, 27 Sep 2003, Hamish wrote:
>
> > Is it a good idea to have the 5.7 'make mix' command check
> > src/CMD/VERSION for 5.3* ? That would stop some of the questions.
>
> Probably better for the configure script to check, and it could check for
> 'cvs' in the version number which should be more robust for the long term.
> The following patch to configure.in seems to work:
>
> --- configure.in.orig   Fri Sep 26 16:11:46 2003
> +++ configure.in        Fri Sep 26 16:26:02 2003
> @@ -35,8 +35,11 @@
>
>  if test -d "$with_grass50"; then
>      GRASS50="$with_grass50"
> +    if ! grep cvs "${GRASS50}/src/CMD/VERSION" >/dev/null 2>&1 ; then
> +        AC_MSG_ERROR([*** GRASS 5.3 source directory \"$with_grass50\" does not contain CVS HEAD version.])
> +    fi
>  else
> -    AC_MSG_ERROR([*** GRASS 5.0 source directory \"$with_grass50\" does not exist.])
> +    AC_MSG_ERROR([*** GRASS 5.3 source directory \"$with_grass50\" does not exist.])
>  fi
>
>  AC_SUBST(GRASS50)
>
> There seems to be a problem with CVS access from inside my University at
> the minute (looks like another new firewall rule) so I can't commit the
> change right now.
>

My CVS access is working again now so I have committed a similar fix to
the 5.7 configure script.

Paul