[Qemu-devel] [PATCH v2] docker: Prefix src copy dir with "."

[Qemu-devel] [PATCH v2] docker: Prefix src copy dir with "."

[Fam Zheng]

So they don't show up in "ls" command, and don't clutter editor's quick
open plugins easily (such as CtrlP of VIM).

Signed-off-by: Fam Zheng <famz@redhat.com>

---

Update .gitignore entry too.
---
 .gitignore                    | 2 +-
 tests/docker/Makefile.include | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/.gitignore b/.gitignore
index 88ec249..5963994 100644
--- a/.gitignore
+++ b/.gitignore
@@ -113,5 +113,5 @@
 cscope.*
 tags
 TAGS
-docker-src.*
+.docker-src.*
 *~
diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
index 4f4707d..de57320 100644
--- a/tests/docker/Makefile.include
+++ b/tests/docker/Makefile.include
@@ -28,7 +28,7 @@ make-archive-maybe = $(if $(wildcard $1/*), \
 		"  ARCHIVE $(notdir $2)"))
 
 CUR_TIME := $(shell date +%Y-%m-%d-%H.%M.%S.$$$$)
-DOCKER_SRC_COPY := docker-src.$(CUR_TIME)
+DOCKER_SRC_COPY := .docker-src.$(CUR_TIME)
 
 $(DOCKER_SRC_COPY):
 	@mkdir $@
-- 
2.7.4

Re: [Qemu-devel] [PATCH v2] docker: Prefix src copy dir with "."

[Paolo Bonzini]

On 08/08/2016 08:48, Fam Zheng wrote:
> So they don't show up in "ls" command, and don't clutter editor's quick
> open plugins easily (such as CtrlP of VIM).

It is also harder to remove it this way though.  Perhaps move it into
tests/docker?

Paolo

> Signed-off-by: Fam Zheng <famz@redhat.com>
> 
> ---
> 
> Update .gitignore entry too.
> ---
>  .gitignore                    | 2 +-
>  tests/docker/Makefile.include | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/.gitignore b/.gitignore
> index 88ec249..5963994 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -113,5 +113,5 @@
>  cscope.*
>  tags
>  TAGS
> -docker-src.*
> +.docker-src.*
>  *~
> diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
> index 4f4707d..de57320 100644
> --- a/tests/docker/Makefile.include
> +++ b/tests/docker/Makefile.include
> @@ -28,7 +28,7 @@ make-archive-maybe = $(if $(wildcard $1/*), \
>  		"  ARCHIVE $(notdir $2)"))
>  
>  CUR_TIME := $(shell date +%Y-%m-%d-%H.%M.%S.$$$$)
> -DOCKER_SRC_COPY := docker-src.$(CUR_TIME)
> +DOCKER_SRC_COPY := .docker-src.$(CUR_TIME)
>  
>  $(DOCKER_SRC_COPY):
>  	@mkdir $@
> 

Re: [Qemu-devel] [PATCH v2] docker: Prefix src copy dir with "."

[Fam Zheng]

On Mon, 08/08 11:03, Paolo Bonzini wrote:
> 
> 
> On 08/08/2016 08:48, Fam Zheng wrote:
> > So they don't show up in "ls" command, and don't clutter editor's quick
> > open plugins easily (such as CtrlP of VIM).
> 
> It is also harder to remove it this way though.  Perhaps move it into
> tests/docker?

That isn't ideal. CtrlP does non-contingous sequence matching on the path name
I type in, and the docker directory has many characters that distracts the
matching a lot.

Any other options you can think of?

Fam