You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bigtop.apache.org by bm...@apache.org on 2013/11/16 22:42:53 UTC

git commit: BIGTOP-809. Remove harmless scary message from solr build.

Updated Branches:
  refs/heads/branch-0.5 adda5e4cd -> 56a7e0477


BIGTOP-809. Remove harmless scary message from solr build.

Using -p will make mkdir not complain if the given directory already exists


Project: http://git-wip-us.apache.org/repos/asf/bigtop/repo
Commit: http://git-wip-us.apache.org/repos/asf/bigtop/commit/56a7e047
Tree: http://git-wip-us.apache.org/repos/asf/bigtop/tree/56a7e047
Diff: http://git-wip-us.apache.org/repos/asf/bigtop/diff/56a7e047

Branch: refs/heads/branch-0.5
Commit: 56a7e04770be187d05f3609dc68913bf6c4b9425
Parents: adda5e4
Author: Bruno Mahé <br...@bmahe.net>
Authored: Sun Dec 16 13:45:11 2012 -0800
Committer: Bruno Mahé <br...@bmahe.net>
Committed: Sun Dec 16 13:45:11 2012 -0800

----------------------------------------------------------------------
 bigtop-packages/src/common/solr/do-component-build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/bigtop/blob/56a7e047/bigtop-packages/src/common/solr/do-component-build
----------------------------------------------------------------------
diff --git a/bigtop-packages/src/common/solr/do-component-build b/bigtop-packages/src/common/solr/do-component-build
index e0b958f..5a69547 100755
--- a/bigtop-packages/src/common/solr/do-component-build
+++ b/bigtop-packages/src/common/solr/do-component-build
@@ -23,7 +23,7 @@ BUILD_OPTS="-Dversion=${FULL_VERSION}                                  \
 
 # FIXME: this needs to be fixed at the product level
 ant $BUILD_OPTS ivy-bootstrap
-mkdir solr/test-framework/lib || :
+mkdir -p solr/test-framework/lib
 
 ant $BUILD_OPTS clean
 


Re: git commit: BIGTOP-809. Remove harmless scary message from solr build.

Posted by Bruno Mahé <bm...@apache.org>.
On 11/16/2013 01:42 PM, bmahe@apache.org wrote:
> Updated Branches:
>    refs/heads/branch-0.5 adda5e4cd -> 56a7e0477
>
>
> BIGTOP-809. Remove harmless scary message from solr build.
>
> Using -p will make mkdir not complain if the given directory already exists
>
>
> Project: http://git-wip-us.apache.org/repos/asf/bigtop/repo
> Commit: http://git-wip-us.apache.org/repos/asf/bigtop/commit/56a7e047
> Tree: http://git-wip-us.apache.org/repos/asf/bigtop/tree/56a7e047
> Diff: http://git-wip-us.apache.org/repos/asf/bigtop/diff/56a7e047
>
> Branch: refs/heads/branch-0.5
> Commit: 56a7e04770be187d05f3609dc68913bf6c4b9425
> Parents: adda5e4
> Author: Bruno Mahé <br...@bmahe.net>
> Authored: Sun Dec 16 13:45:11 2012 -0800
> Committer: Bruno Mahé <br...@bmahe.net>
> Committed: Sun Dec 16 13:45:11 2012 -0800
>
> ----------------------------------------------------------------------
>   bigtop-packages/src/common/solr/do-component-build | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> ----------------------------------------------------------------------
>
>
> http://git-wip-us.apache.org/repos/asf/bigtop/blob/56a7e047/bigtop-packages/src/common/solr/do-component-build
> ----------------------------------------------------------------------
> diff --git a/bigtop-packages/src/common/solr/do-component-build b/bigtop-packages/src/common/solr/do-component-build
> index e0b958f..5a69547 100755
> --- a/bigtop-packages/src/common/solr/do-component-build
> +++ b/bigtop-packages/src/common/solr/do-component-build
> @@ -23,7 +23,7 @@ BUILD_OPTS="-Dversion=${FULL_VERSION}                                  \
>
>   # FIXME: this needs to be fixed at the product level
>   ant $BUILD_OPTS ivy-bootstrap
> -mkdir solr/test-framework/lib || :
> +mkdir -p solr/test-framework/lib
>
>   ant $BUILD_OPTS clean
>
>


I didn't mean to push that change.
But since it was reviewed, commited on trunk and not going to impact 
0.5.0, I don't see a reason to revert it either.

Let me know if anyone feels otherwise.

Thanks,
Bruno