[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