git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Pavel Roskin <proski@gnu.org>
To: Petr Baudis <pasky@suse.cz>
Cc: git <git@vger.kernel.org>
Subject: Re: [PATCH] cg-pull to stop treating "master" specially, fix fetch_local for .git/HEAD
Date: Thu, 10 Nov 2005 15:11:47 -0500	[thread overview]
Message-ID: <1131653507.11283.31.camel@dv> (raw)
In-Reply-To: <20051110192430.GS30496@pasky.or.cz>

Hi, Petr!

On Thu, 2005-11-10 at 20:24 +0100, Petr Baudis wrote:
>   can you still remember why did you introduce this? In GNU cp
> documentation, I can see just
> 
>        -b, --backup
>               Make backups of files that are about to be overwritten or removed.
> 
> which doesn't make sense to me - -L dereferences symlinks.

You are right, it must be my error.  Anyway, it was so long ago that I
would need to review and retest it.

While at that, let's stop using symlinks.  git doesn't use symlinks on
Cygwin.  I think git should use that code on all OSes, since the
benefits of using symlinks are minimal (I think the only benefits are
their atomicity and resolving the reference in the kernel rather than in
userspace).  Having more uniform code for all platforms would simplify
development and testing.  It could also reduce requirements for the
transport protocols.  Finally, symlinks could be still used by the users
(if they know what they are doing) - git and cogito would simply become
symlink agnostic.

When files are copied around, symlinks are pain to deal with.  They
require special handling to be preserved both for remote operation and
dereferenced for local operation (that's what my patch was intended to
do).  I'm not even considering what would happen when cloning from Linux
to Windows or vice versa.

Sure, it can wait until git 1.0, but it would be great to keep this goal
in mind.

Disclaimer - I'm not reading the git mailing list, so if it was
discussed, I'm sorry, I don't intend to restart that discussion - just
give me the pointer and I'll read it.

-- 
Regards,
Pavel Roskin

  reply	other threads:[~2005-11-10 20:12 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-08-23 21:33 [PATCH] cg-pull to stop treating "master" specially, fix fetch_local for .git/HEAD Pavel Roskin
2005-11-10 19:24 ` Petr Baudis
2005-11-10 20:11   ` Pavel Roskin [this message]
2005-11-10 23:26 ` Josef Weidendorfer
2005-11-10 23:40   ` Petr Baudis
2005-11-10 23:56     ` Josef Weidendorfer
2005-11-11  0:09       ` Petr Baudis
2005-11-11  0:14     ` Pavel Roskin
2005-11-11  1:13       ` Josef Weidendorfer
2005-11-11  4:53         ` Pavel Roskin
2005-11-11 14:22           ` Josef Weidendorfer
2005-11-11 15:26             ` Pavel Roskin
2005-11-11 16:10               ` Josef Weidendorfer

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=1131653507.11283.31.camel@dv \
    --to=proski@gnu.org \
    --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;
as well as URLs for NNTP newsgroup(s).