701169 - Talos should have a basic .gitignore file

Status: RESOLVED FIXED

(Testing :: Talos, defect)

Description

[William Lachance (:wlach)]

I'm currently importing talos into eideticker via a git submodule (imported from the git mirror of talos here: https://github.com/mozilla/talos). Similar to what we already do for the git mirror of m-c, it would be convenient to have a simple .gitignore in the root of the repository so that the output of "git status" is a bit clearer.

Comment 1

[William Lachance (:wlach)]

Attachment: Basic .gitignore for talos

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 573300 [details] [diff] [review]
Basic .gitignore for talos

Review of attachment 573300 [details] [diff] [review]:
-----------------------------------------------------------------

browser_output.txt seems like overkill.  In that same regard what about remoteapp.ini?  Otherwise a great addition to the talos repository.

Comment 3

[William Lachance (:wlach)]

(In reply to Joel Maher (:jmaher) from comment #2)
> Comment on attachment 573300 [details] [diff] [review] [diff] [details] [review]
> Basic .gitignore for talos
> 
> Review of attachment 573300 [details] [diff] [review] [diff] [details] [review]:
> -----------------------------------------------------------------
> 
> browser_output.txt seems like overkill.  In that same regard what about
> remoteapp.ini?  Otherwise a great addition to the talos repository.

wrt browser_output.txt. Don't blame me, I'm just copying the existing behaviour in .hgignore. :) Seriously, I think it's ok to ignore it, since it falls under the definition of an automatically generated file. I can't see how we'd ever want to commit it or otherwise pay attention to it version-control wise.

wrt remoteapp.ini, yes, we should be ignoring this (along with a bunch of other mobile-specific files) in both .hgignore and .gitignore though I think this would be better addressed in a separate bug.

If this sounds ok, could you push when you get the chance? Thanks!

Comment 4

[William Lachance (:wlach)]

Pushed: http://hg.mozilla.org/build/talos/rev/c338c13b79ac