Re: [Mesa-dev] [PATCH 2/4] docs: remove redundant parameter for scons build basic testing

2017-07-15 Thread Andres Gomez 
On Mon, 2017-07-10 at 12:22 +0100, Emil Velikov wrote:
> On 8 July 2017 at 20:59, Andres Gomez  wrote:
> > toolchain=crossmingw is the default option for platform=windows when
> > invoking scons.
> > 
> 
> It may be a bit redundant, but I'd rather keep it. I doubt it's so
> interesting to track when it became the default and if/when that will
> change ;-)

OK, I'll drop this patch.

-- 
Br,

Andres
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 2/4] docs: remove redundant parameter for scons build basic testing

2017-07-10 Thread Emil Velikov 
On 8 July 2017 at 20:59, Andres Gomez  wrote:
> toolchain=crossmingw is the default option for platform=windows when
> invoking scons.
>
It may be a bit redundant, but I'd rather keep it. I doubt it's so
interesting to track when it became the default and if/when that will
change ;-)

-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 2/4] docs: remove redundant parameter for scons build basic testing

2017-07-10 Thread Eric Engestrom 
On Saturday, 2017-07-08 22:59:39 +0300, Andres Gomez wrote:
> toolchain=crossmingw is the default option for platform=windows when
> invoking scons.

Has that always been true though? Or at least since the minimal scons
version supported?

> 
> Signed-off-by: Andres Gomez 
> ---
>  docs/releasing.html | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/docs/releasing.html b/docs/releasing.html
> index ceb6e365e4..8e6e4d1a6d 100644
> --- a/docs/releasing.html
> +++ b/docs/releasing.html
> @@ -450,7 +450,7 @@ Here is one solution that I've been using.
>   # You may need to unset LLVM if you set it before:
>   # unset LLVM_CONFIG
>   tar -xaf mesa-$__version.tar.xz  cd mesa-$__version
> - scons platform=windows toolchain=crossmingw
> + scons platform=windows
>   cd ..  rm -rf mesa-$__version
>  
>   # Test the automake binaries
> -- 
> 2.11.0
> 
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH 2/4] docs: remove redundant parameter for scons build basic testing

2017-07-08 Thread Andres Gomez 
toolchain=crossmingw is the default option for platform=windows when
invoking scons.

Signed-off-by: Andres Gomez 
---
 docs/releasing.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/docs/releasing.html b/docs/releasing.html
index ceb6e365e4..8e6e4d1a6d 100644
--- a/docs/releasing.html
+++ b/docs/releasing.html
@@ -450,7 +450,7 @@ Here is one solution that I've been using.
# You may need to unset LLVM if you set it before:
# unset LLVM_CONFIG
tar -xaf mesa-$__version.tar.xz  cd mesa-$__version
-   scons platform=windows toolchain=crossmingw
+   scons platform=windows
cd ..  rm -rf mesa-$__version
 
# Test the automake binaries
-- 
2.11.0

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev