[m-rev.] for review: Provide a helpful error if no C sharp compiler could be found

Julien Fischer jfischer at opturion.com
Thu May 23 22:12:19 AEST 2013


On Thu, May 23, 2013 at 8:48 PM, Paul Bone <paul at bone.id.au> wrote:

> When I was addressing the review comments of my earlier submitted patches,
> I
> saw the opportunity to add the following additional change.
>
> --
>
> Provide a helpful error if no C sharp compiler could be found
>
> m4/mercury.m4:
>     If the user specified one or more possible C sharp compilers and none
>     of them were suitable, then report an error and abort.
> ---
>  m4/mercury.m4 | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/m4/mercury.m4 b/m4/mercury.m4
> index 88aa79f..dd34cc8 100644
> --- a/m4/mercury.m4
> +++ b/m4/mercury.m4
> @@ -369,6 +369,13 @@ EOF
>      esac
>  done
>
> +# If the user specified one or more compilers and we couldn't find any
> +# then abort configuration.
> +if test "$mercury_cv_with_csharp_compiler" != "" -a "$CSC" = ""; then
> +    AC_MSG_ERROR([No C sharp compilers could be found.]


How about:

    No working C# compiler could be found.

Looks fine otherwise.

Cheers,
Julien.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20130523/7608aeb4/attachment.html>


More information about the reviews mailing list