[PATCH] build.sh: verbose error output for missing components

Gaetan Nadon memsize at videotron.ca
Wed Nov 3 11:09:46 PDT 2010


On Wed, 2010-11-03 at 14:47 +0100, Simon Braunschmidt wrote:

> Instead of Trevor Woerners proposal to automagically clone missing
> components,
> 

I originally made the suggestion, but you convinced me.

> http://lists.x.org/archives/xorg-devel/2010-November/014883.html
> 
> I would rather want to inform the user more verbosely
> about missing components and how to obtain them.
> 
> This could avoid situations where the user calls the build.sh script
> from the wrong location, and build.sh subsequently littering the users
> machine with unwanted git repositories in the wrong place.

Good point.

> 
> One such situation would be calling "somedir/build.sh L" instead of
> "somedir/build.sh -L", starting a build in the current directory
> for PREFIX "L" instead of listing the missing components (with the
> users intention having been to just list the components in build order).
> 
> Signed-off-by: Simon Braunschmidt <sbraun at emlix.com>
> ---
>  build.sh |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/build.sh b/build.sh
> index 207ff2b..0afbe50 100755
> --- a/build.sh
> +++ b/build.sh
> @@ -1155,9 +1155,14 @@ date
>  if [ X"$nonexistent_components" != X ]; then
>      echo ""
>      echo "***** Skipped components (not available) *****"
> +	echo "Could neighter find a git repository (at the <module/component> paths)"
> +	echo "or a tarball (at the <module/> paths or ./) for:"
> +	echo "    <module/component>"
>      for mod in $nonexistent_components; do
>  	echo "    $mod"
>      done
> +    echo "You may want to provide the --clone option to build.sh"
> +    echo "to automatically git-clone the missing components"
>      echo ""
>  fi
>  


Acked-by: Gaetan Nadon <memsize at videotron.ca>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-devel/attachments/20101103/686d2194/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20101103/686d2194/attachment.pgp>


More information about the xorg-devel mailing list