Re: [RFC/PATCH 11/18] merge: Add a new --index-only option, not yet implemented

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Elijah Newren <newren@xxxxxxxxx> writes:

> +--index-only::
> +	Perform merge on index only, leaving working tree alone.  Most
> +	users do NOT want to use this flag, as it will leave the
> +	working tree and the index completely out of sync, which is
> +	very likely to confuse users and prevent a subsequent 'git
> +	merge --abort' from working.

This whole paragraph is a strong sign that this feature as-is is not
a good addition.

On the other hand ...

> +	It is intended for script
> +	writers to have a way to easily check whether a merge would
> +	succeed and which files would conflict, typically from bare
> +	clones.

... this may be a good goal to aim for, but I think you are
understating.  You seem to be wanting a lot more than "easily check
whether..."; isn't this more like "It is for scripts to create a
merge and advance the branch when no need for manual conflict
resolution in a bare repository, and to learn which paths would
require manual conflicts when it fails", no?

I'd think either (1) limit this to bare repository only, or (2) do
the "git merge --into master en/topic" I outlined in the other
message, (or both) would be a sensible alternative for a feature
whose description has to begin with "Most users do not want this".
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]