On 12/7/08, Daniel Barkalow <barkalow@xxxxxxxxxxxx> wrote: > > There is not much work for CE_NO_CHECKOUT on plumbling level except > > some fixes. The last half of the series, for porcelain level, you will > > see more. > > > For the porcelain level, do we need the difference to be in the index? If > the porcelain knows the sparse checkout area and can instruct the plumbing > appropriately, the information shouldn't need to be stored in the index This was discussed since the beginning of this feature. I recall that the index reflects worktree, and because we mark CE_NO_CHECKOUT on file basis, it's best to save the information there, not separately. We do save high level information to form the checkout area (sparse patterns) in the last half, but basically you should be able to live without that. > unless it's ever important to remember whether an entry is CE_VALID due to > having been outside the checkout when the index was written, even though > the checkout area now includes it. I don't have a good intuition as to > what ought to happen if the user manually changes what's specified for > checkout without actually updating the index and working tree. So if a user changes worktree without updating index, they will have the same results as they do now: files are shown as modified if they don't have CE_NO_CHECKOUT set. If those files do, they are considered 'orphaned' or staled and are recommended to be removed/updated to avoid unexpected consequences (not availble this this first half series because that belongs to "git status"). -- Duy -- 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
