Re: [PATCH spice-streaming-agent v2 2/3] src/unitests: add temporary files to .gitignore

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2018-02-13 at 09:08 -0500, Frediano Ziglio wrote:
> > 
> > Also, leading / is unnecessary.
> > 
> > Signed-off-by: Lukáš Hrázký <lhrazky@xxxxxxxxxx>
> > ---
> >  src/unittests/.gitignore | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> > 
> > diff --git a/src/unittests/.gitignore b/src/unittests/.gitignore
> > index 36548a1..e871e91 100644
> > --- a/src/unittests/.gitignore
> > +++ b/src/unittests/.gitignore
> > @@ -1 +1,4 @@
> > -/test-hexdump
> > +hexdump.sh.log
> > +hexdump.sh.trs
> > +test-hexdump
> > +test-suite.log
> 
> Would not be better, if we decide to name all tests test-XXXX, to
> exclude test-*.log and test-*.trs?

Sounds good.

> The leading / is to avoid to catch files in subdirectories, in
> this case I don't think we are going to exclude more files than
> expected ones but doesn't hurt to reduce the scope.

I see. Will add it again.

I just realized this is outdated by your renaming patch. I'll update
it.

Lukas
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]