diff mbox

[2/3] gitignore: Explicitely ignore sub-directories in lib/packages

Message ID 1444385822-24254-3-git-send-email-damien.lespiau@intel.com
State Accepted
Headers show

Commit Message

Damien Lespiau Oct. 9, 2015, 10:17 a.m. UTC
This file ignores everything at the moment. This doesn't look like a
good idea, there are also jquery files there.

We ignore django/ because installing django there is what currently is
documented to do.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
 lib/packages/.gitignore | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Stephen Finucane Oct. 9, 2015, 11:21 p.m. UTC | #1
> This file ignores everything at the moment. This doesn't look like a

> good idea, there are also jquery files there.

> 

> We ignore django/ because installing django there is what currently is

> documented to do.

> 

> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>


Subject has two typo in it:

	s/Explicitely/Explicitly/
	s/sub-directories/subdirectories/

Otherwise:

Acked-by: Stephen Finucane <stephen.finucane@intel.com>
Stephen Finucane Oct. 27, 2015, 2:01 a.m. UTC | #2
> > This file ignores everything at the moment. This doesn't look like a

> > good idea, there are also jquery files there.

> >

> > We ignore django/ because installing django there is what currently is

> > documented to do.

> >

> > Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>

> 

> Subject has two typo in it:

> 

> 	s/Explicitely/Explicitly/

> 	s/sub-directories/subdirectories/

> 

> Otherwise:

> 

> Acked-by: Stephen Finucane <stephen.finucane@intel.com>


Merged (with aforementioned typo fixes + < 50 char summary). Thanks, Damien :)

Stephen
diff mbox

Patch

diff --git a/lib/packages/.gitignore b/lib/packages/.gitignore
index 72e8ffc..d3e4ba5 100644
--- a/lib/packages/.gitignore
+++ b/lib/packages/.gitignore
@@ -1 +1 @@ 
-*
+django