[Qemu-devel] [PATCH 1/5] tests/tcg: target/mips: Remove an unnecessary file

Aleksandar Markovic posted 5 patches 6 years, 8 months ago
Maintainers: Aleksandar Markovic <amarkovic@wavecomp.com>, Aurelien Jarno <aurelien@aurel32.net>, Aleksandar Rikalo <arikalo@wavecomp.com>
There is a newer version of this series
[Qemu-devel] [PATCH 1/5] tests/tcg: target/mips: Remove an unnecessary file
Posted by Aleksandar Markovic 6 years, 8 months ago
From: Aleksandar Markovic <amarkovic@wavecomp.com>

Remove a file that was added long time ago by mistake.

Signed-off-by: Aleksandar Markovic <amarkovic@wavecomp.com>
---
 tests/tcg/mips/mips64-dspr2/.directory | 2 --
 1 file changed, 2 deletions(-)
 delete mode 100644 tests/tcg/mips/mips64-dspr2/.directory

diff --git a/tests/tcg/mips/mips64-dspr2/.directory b/tests/tcg/mips/mips64-dspr2/.directory
deleted file mode 100644
index c75a914..0000000
--- a/tests/tcg/mips/mips64-dspr2/.directory
+++ /dev/null
@@ -1,2 +0,0 @@
-[Dolphin]
-Timestamp=2012,8,3,16,41,52
-- 
2.7.4


Re: [Qemu-devel] [PATCH 1/5] tests/tcg: target/mips: Remove an unnecessary file
Posted by Eric Blake 6 years, 8 months ago
On 2/12/19 1:53 PM, Aleksandar Markovic wrote:
> From: Aleksandar Markovic <amarkovic@wavecomp.com>
> 
> Remove a file that was added long time ago by mistake.

Might be worth mentioning commit d70080c4 (in 2012).

Is this the sort of file that is likely enough to be created during
typical workflows that it should be listed in the project's .gitignore
for everyone?  Or is it particular enough to your environment that you
could 'echo .directory >> .git/info/exclude' to ensure it doesn't happen
again for your setup, without bothering other developers with your
exclusion?

Either way, the cleanup itself makes sense.

Reviewed-by: Eric Blake <eblake@redhat.com>

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

Re: [Qemu-devel] [PATCH 1/5] tests/tcg: target/mips: Remove an unnecessary file
Posted by Aleksandar Markovic 6 years, 8 months ago
> From: Eric Blake <eblake@redhat.com>
> Sent: Tuesday, February 12, 2019 9:04 PM
> To: Aleksandar Markovic; qemu-devel@nongnu.org
> Cc: Aleksandar Rikalo; alex.bennee@linaro.org; Aleksandar Markovic; aurelien@aurel32.net
> Subject: Re: [Qemu-devel] [PATCH 1/5] tests/tcg: target/mips: Remove an unnecessary file
> 
> On 2/12/19 1:53 PM, Aleksandar Markovic wrote:
> > From: Aleksandar Markovic <amarkovic@wavecomp.com>
> >
> > Remove a file that was added long time ago by mistake.
> 
> Might be worth mentioning commit d70080c4 (in 2012).

I am going to mention it in the next version of this series.

> 
> Is this the sort of file that is likely enough to be created during
> typical workflows that it should be listed in the project's .gitignore
> for everyone?  Or is it particular enough to your environment that you
> could 'echo .directory >> .git/info/exclude' to ensure it doesn't happen
> again for your setup, without bothering other developers with your
> exclusion?
> 

It is particular enough, no need to update .gitingnore for everyone, IMHO.

> Either way, the cleanup itself makes sense.
> 
> Reviewed-by: Eric Blake <eblake@redhat.com>
> 
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc.           +1-919-301-3226
> Virtualization:  qemu.org | libvirt.org

Thanks,
Aleksandar


Re: [Qemu-devel] [PATCH 1/5] tests/tcg: target/mips: Remove an unnecessary file
Posted by Alex Bennée 6 years, 8 months ago
Aleksandar Markovic <aleksandar.markovic@rt-rk.com> writes:

> From: Aleksandar Markovic <amarkovic@wavecomp.com>
>
> Remove a file that was added long time ago by mistake.
>
> Signed-off-by: Aleksandar Markovic <amarkovic@wavecomp.com>

Acked-by: Alex Bennée <alex.bennee@linaro.org>

> ---
>  tests/tcg/mips/mips64-dspr2/.directory | 2 --
>  1 file changed, 2 deletions(-)
>  delete mode 100644 tests/tcg/mips/mips64-dspr2/.directory
>
> diff --git a/tests/tcg/mips/mips64-dspr2/.directory b/tests/tcg/mips/mips64-dspr2/.directory
> deleted file mode 100644
> index c75a914..0000000
> --- a/tests/tcg/mips/mips64-dspr2/.directory
> +++ /dev/null
> @@ -1,2 +0,0 @@
> -[Dolphin]
> -Timestamp=2012,8,3,16,41,52


--
Alex Bennée