Re: intend-to-edit flag

2013-07-04 Thread Thomas Koch
On Thursday, July 04, 2013 08:10:07 PM Ævar Arnfjörð Bjarmason wrote:
> Why don't you just start using Git and see if this becomes a practical
> problem rather than devising some elaborate solution to work around
> something that probably won't be an issue anyway?

I've been giving talks about Git already in 2008. I know that we don't need 
locking. But my boss says we need it. - So he'll get it.

Thomas Koch, http://www.koch.ro
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: intend-to-edit flag

2013-07-04 Thread John Keeping
On Thu, Jul 04, 2013 at 08:10:07PM +0200, Ævar Arnfjörð Bjarmason wrote:
> On Thu, Jul 4, 2013 at 7:56 PM, Thomas Koch  wrote:
> > we're evaluating Git to be used in our companies Tool. But a hard 
> > requirement
> > is the possibility to set an "intend-to-edit" flag on a file (better path).
> > Notice that I did not use the word "lock"! :-)
> >
> > One easy implementation might be a special branch "XYZ-locks" that contains 
> > an
> > empty blob for every flagged file. So our tool just needs to check, whether 
> > a
> > blob exists for the path that's intended to edit, tries to push a commit 
> > that
> > touches the file and only allows editing if the push succeeds.
> 
> In my experience everyone who thinks this is a hard requirement is
> wrong.

I completely agree with this.

Having said that, if you're looking at using Gitolite (which you should
if you're hosing your own repositories and not using some other hosting
solution), there is a "lock" command [1].  Note that this cannot stop
you committing changes to "locked" files locally but it does stop you
pushing changes to the central repository that touch locked files.

[1] http://gitolite.com/gitolite/locking.html
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: intend-to-edit flag

2013-07-04 Thread Ævar Arnfjörð Bjarmason
On Thu, Jul 4, 2013 at 7:56 PM, Thomas Koch  wrote:
> we're evaluating Git to be used in our companies Tool. But a hard requirement
> is the possibility to set an "intend-to-edit" flag on a file (better path).
> Notice that I did not use the word "lock"! :-)
>
> One easy implementation might be a special branch "XYZ-locks" that contains an
> empty blob for every flagged file. So our tool just needs to check, whether a
> blob exists for the path that's intended to edit, tries to push a commit that
> touches the file and only allows editing if the push succeeds.

In my experience everyone who thinks this is a hard requirement is
wrong.

Sure you can implement something to do this, but more likely than not
you think you need it because your current centralized SCM does it and
you think you can't live without it.

I work with a couple of hundred devs all grinding on the same
repository and it's really rare to have:

 * People who edit the same code within each other's pull/push window AND
 * Have the edits to those files not be smoothly resolved by automatic
   merging (i.e. because it was to completely different parts of the
   file).

When it does happen every once in a while it's trivial to solve it,
you just resolve conflicts, talk to the other guy etc.

Why don't you just start using Git and see if this becomes a practical
problem rather than devising some elaborate solution to work around
something that probably won't be an issue anyway?
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html