From: Kirill Smelkov <kirr@landau.phys.spbu.ru>
To: Andreas Ericsson <ae@op5.se>
Cc: Kirill Smelkov <kirr@landau.phys.spbu.ru>,
Bert Wesarg <bert.wesarg@googlemail.com>,
Daniel Barkalow <barkalow@iabervon.org>,
Petr Baudis <pasky@suse.cz>,
Git Mailing List <git@vger.kernel.org>
Subject: Re: [TopGit PATCH] .gitignore += vim swap files
Date: Fri, 19 Sep 2008 11:52:13 +0400 [thread overview]
Message-ID: <20080919075213.GE4423@roro3> (raw)
In-Reply-To: <48D359FD.90203@op5.se>
On Fri, Sep 19, 2008 at 09:51:25AM +0200, Andreas Ericsson wrote:
> Kirill Smelkov wrote:
>> On Fri, Sep 19, 2008 at 09:10:24AM +0200, Bert Wesarg wrote:
>>> On Fri, Sep 19, 2008 at 07:06, Kirill Smelkov <kirr@landau.phys.spbu.ru> wrote:
>>>> On Thu, Sep 18, 2008 at 03:30:35PM -0400, Daniel Barkalow wrote:
>>>>> On Thu, 18 Sep 2008, Bert Wesarg wrote:
>>>>>
>>>>>> On Thu, Sep 18, 2008 at 18:29, Kirill Smelkov <kirr@landau.phys.spbu.ru> wrote:
>>>>>>> Signed-off-by: Kirill Smelkov <kirr@landau.phys.spbu.ru>
>>>>>>> ---
>>>>>>> .gitignore | 2 ++
>>>>>>> 1 files changed, 2 insertions(+), 0 deletions(-)
>>>>>>>
>>>>>>> diff --git a/.gitignore b/.gitignore
>>>>>>> index 8868f2d..aa39db4 100644
>>>>>>> --- a/.gitignore
>>>>>>> +++ b/.gitignore
>>>>>>> @@ -18,3 +18,5 @@ tg-import.txt
>>>>>>> tg-remote
>>>>>>> tg-remote.txt
>>>>>>> tg
>>>>>>> +
>>>>>>> +*.swp
>>>>>> can't you do this in your .git/info/excludes?
>>>>> It's generally better to put a core.excludesfile entry in your
>>>>> ~/.gitconfig pointing to a ignore file with editor temporaries for the
>>>>> editor(s) you personally use. This will then apply to all git projects you
>>>>> work on.
>>>> Yes, this makes sense, thanks.
>>>>
>>>> I've had to add the following to my ~/.gitconfig
>>>>
>>>> [core]
>>>> excludesfile= /home/kirr/.gitignore-kirr
>>>>
>>>>
>>>> because it does not work when '~' is used for $HOME
>>>>
>>>> [core]
>>>> excludesfile= ~/.gitignore-kirr # does not work
>>> Have you tried:
>>>
>>> [core]
>>> excludesfile = .gitignore-kirr
>>> There was a discussion about relative files in config. but I didn't tried this.
>>
>> It does not work for me -- it picks ./.gitignore-kirr instead of
>> ~/.gitignore-kirr
>>
>>> Anyway, I think this is the best solution. Don't globber the repos with this.
>>
>> Don't you care more about machines than humans? :)
>>
>> I think when we work on a tree, all the autogenerated files should be
>> ignored. This includes *.o and friends for sure, but since we also use
>> editors, and most people use same editors it makes sense to tune repo to
>> be ready to hack on with common setups.
>>
>
> Most people do not use same editors. In our office there's 8 programmers
> and 6 editors in use (geany, jed, vim, nedit, zend studio, kate).
>
>> And I think this
>>
>> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=7088655477b51a5a248fa54190388e1283ba7ebf
>>
>> just justifies my words.
>>
>
> If topgit had 4374 contributors since 10 minor versions back, I'd be
> more inclined to agree with you. Even so, you'll have to accept that
> this patch gets dropped[1] and stop trying to argue for it after it's
> already been discussed (at length).
>
> [1]
> I'm not too sure who the topgit maintainer actually is, but the majority
> seem to be against this patch.
I'm ok with this patch being dropped.
--
Всего хорошего, Кирилл.
next prev parent reply other threads:[~2008-09-19 8:01 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-09-18 16:29 A couple of TopGit tweaks Kirill Smelkov
2008-09-18 16:29 ` [TopGit PATCH] .gitignore += vim swap files Kirill Smelkov
2008-09-18 17:24 ` martin f krafft
2008-09-18 17:30 ` Kirill Smelkov
2008-09-18 17:38 ` Bert Wesarg
2008-09-18 17:43 ` Kirill Smelkov
2008-09-18 17:56 ` Bert Wesarg
2008-09-18 17:50 ` Kirill Smelkov
2008-09-18 17:54 ` martin f krafft
2008-09-18 19:30 ` Daniel Barkalow
2008-09-19 5:06 ` Kirill Smelkov
2008-09-19 7:10 ` Bert Wesarg
2008-09-19 7:28 ` Kirill Smelkov
2008-09-19 7:51 ` Andreas Ericsson
2008-09-19 7:52 ` Kirill Smelkov [this message]
2008-09-19 10:50 ` Bert Wesarg
2008-09-19 14:22 ` Default exclude rules for Git Petr Baudis
2008-09-19 15:46 ` Junio C Hamano
2008-09-19 16:33 ` Petr Baudis
2008-09-19 16:42 ` Avery Pennarun
2008-09-18 16:29 ` [TopGit PATCH] tg help: <something>: improve readability Kirill Smelkov
2008-09-18 16:29 ` [TopGit PATCH] tg import: fix + make more robust Kirill Smelkov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080919075213.GE4423@roro3 \
--to=kirr@landau.phys.spbu.ru \
--cc=ae@op5.se \
--cc=barkalow@iabervon.org \
--cc=bert.wesarg@googlemail.com \
--cc=git@vger.kernel.org \
--cc=pasky@suse.cz \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox