This is an archive of the discontinued Mercurial Phabricator instance.

Diff	ID	Description	Created	Lint	Unit
Base		Base
Diff 1	29399		Jul 19 2021, 6:40 AM	★	★
Diff 2	29552		Jul 20 2021, 3:21 AM	★	★
Diff 3	29634	rHG33beeb32f73af94632346062c88186eea94e5cbf	Jul 16 2021, 4:30 PM	★	★

Commit	Parents	Author	Summary	Date
736a31c3a790	3c4965988a36	Pierre-Yves David		Jul 16 2021, 4:30 PM

Status	Author	Revision
Closed	marmoute	D11200 relnotes: document the change in the dirstate API
Closed	marmoute	D11199 dirstate: deprecate the `add` method
Closed	marmoute	D11198 mq: replace `add` call with newer API
Closed	marmoute	D11197 largefile: use `updatefile` instead of `add` in `synclfdirstate`
Closed	marmoute	D11196 largefile: use `update_file` instead of `add` in `mergerecordupdates`
Closed	marmoute	D11195 dirstate: deprecate the `drop` method
Closed	marmoute	D11194 narrow: stop using `drop` in the `updateworkingcopy`
Closed	marmoute	D11193 mq: replace `drop` call with newer API
Closed	marmoute	D11192 largefile: use `update_file` instead of `drop` during `rollback`
Closed	marmoute	D11191 largefile: use `update_file` instead of `drop` in `synclfdirstate`
Closed	marmoute	D11190 sparse: use `update_file` instead of `drop`
Closed	marmoute	D11189 sparse: use `update_file` instead of `drop` in `refreshwdir`
Closed	marmoute	D11188 amend: use `update_file` instead of `drop`
Closed	marmoute	D11187 context: use `update_file` instead of `drop` in `markcommitted`
Closed	marmoute	D11186 dirstate: deprecate the `merge` method in all cases
Closed	marmoute	D11185 mq: drop the use of `dirstate.merged
Closed	marmoute	D11184 dirstate: deprecate the `otherparent` method in all cases
Closed	marmoute	D11183 dirstate: deprecate the `normallookup` method in all cases
Closed	marmoute	D11182 narrow: stop using `normallookup` during a test
Closed	marmoute	D11181 narrow: use `update_file` instead of `normallookup` after update
Closed	marmoute	D11180 test: use the internal `_normallookup` in `test-rebuildstate.t`
Closed	marmoute	D11179 sparse: use `update_file` instead of `normallookup` in refreshwdir
Closed	marmoute	D11178 keywork: use `update_file_p1` instead of `normallookup`
Closed	marmoute	D11177 fix: use `set_possibly_dirty` instead of `normallookup`
Closed	marmoute	D11176 eol: use `set_possibly_dirty` instead of `normallookup`
Closed	marmoute	D11175 mq: replace `normallookup` call with newer API
Closed	marmoute	D11174 revert: use `set_possibly_dirty` instead of `normal_lookup`
Closed	marmoute	D11173 dirstate: enforce `possibly_dirty` in `set_tracked`
Closed	marmoute	D11172 record: use `update_file` instead of `normallookup`
Closed	marmoute	D11171 copy: use `set_tracked` instead of `normallookup` in `dirstatecopy`
Closed	marmoute	D11170 amend: use `update_file` instead of `normallookup`
Closed	marmoute	D11169 mergestate: use `update_file` to handle `for ACTION_EXEC`
Closed	marmoute	D11168 mergestate: use `update_file` to handle `for ACTION_ADD_MODIFIED`
Closed	marmoute	D11167 largefile: use `update_file` for `synclfdirstate` "m" case
Closed	marmoute	D11166 largefile: use `update_file` for `synclfdirstate` "n" case
Closed	marmoute	D11165 largefile: replace the `normallookup` call when creating a new lfdirstate
Closed	marmoute	D11164 largefile: use `update_file` instead of `normallookup` in `updatelfiles`
Closed	marmoute	D11163 largefile: use `set_possibly_dirty` instead of `normallookup`
Closed	marmoute	D11162 dirstate: add a `set_possibly_dirty` method
Closed	marmoute	D11161 dirstate: deprecate the `normal` method in all cases
Closed	marmoute	D11160 test: use `set_traced` in `test-context`
Closed	marmoute	D11159 mq: replace usage of `normal` with newer API
Closed	marmoute	D11158 largefile: use `update_file` instead of `normal` in `synclfdirstate`
Closed	marmoute	D11157 largefile: use `set_clean` instead of `normal` in `repo.status`
Closed	marmoute	D11156 largefile: use `set_clean` instead of `normal` in `lfdirstatestatus`
Closed	marmoute	D11155 largefile: use `update_file` instead of `normal` in `updatelfiles`
Closed	marmoute	D11154 largefile: respect the `normal` signature
Closed	marmoute	D11153 keywork: use `set_clean` instead of `normal`
Closed	marmoute	D11152 sparse: use `update_file` instead of `normal` in `refreshwdir`
Closed	marmoute	D11151 sparse: use `update_file` instead of `normal` during `applyupdates`
Closed	marmoute	D11150 sparse: apply update with in a `parentchange` context
Closed	marmoute	D11149 revert: use `set_clean` instead of `normal`
Closed	marmoute	D11148 context: use `update_file` instead of `normal` in `markcommitted`
Closed	marmoute	D11147 context: use new API instead of `normal` during fixup post status
Closed	marmoute	D11146 dirstate: introduce a `set_clean` method
Closed	marmoute	D11145 largefile: use `update_file` instead of `normal`
Closed	marmoute	D11144 largefile: adjust the dirstate post update within a `parentchange` context
Closed	marmoute	D11143 dirstate: deprecate `dirstate.remove` in all cases
Closed	marmoute	D11142 largefile: use `update_file` instead of `remove` in `synclfdirstate`
Closed	marmoute	D11141 largefile: rearrange conditionnal in `synclfdirstate`
Closed	marmoute	D11140 largefile: use `update_file` instead of `remove` in `mergerecordupdates`
Closed	marmoute	D11139 mq: use `update_file_p1` instead of `remove` when adjusting dirstate
Closed	marmoute	D11138 dirstate: properly update `_lastnormaltime` in `update_file_p1`
Closed	marmoute	D11137 dirstate: properly update `_lastnormaltime` in `update_file`
Closed	marmoute	D11136 dirstate: use `reset_state` in `update_file_p1`
Closed	marmoute	D11135 dirstate: rename `update_file_reference` to `update_file_p1`
Closed	marmoute	D11134 dirstate-map: move most of `dirstate.update_file` logic in the dsmap
Closed	marmoute	D11133 dirstate: replace `update_parent_file_data` with simpler `update_parent` call
Closed	marmoute	D11132 dirstate: factor out the part retrieve "filedata" out of `normal`
Closed	marmoute	D11131 dirstate-map: factor out the change to _dirs and _alldirs on removing
Closed	marmoute	D11130 dirstate-map: factor out the change to _dirs and _alldirs on dropping
Closed	marmoute	D11129 dirstate-map: factor out the change to _dirs and _alldirs on adding
Closed	marmoute	D11128 dirstate: replace a dead conditional branch with an assert in `update_file`
Closed	marmoute	D11127 dirstate-map: do not use `size` to gate copy dropping during remove_file
Closed	marmoute	D11126 dirstate: drop a duplicated assert
Closed	marmoute	D11125 dirstate: drop duplicated check
Closed	marmoute	D11124 dirstate-item: introduce a `dm_otherparent` property
Closed	marmoute	D11123 dirstate-item: introduce a `dm_nonnormal` property
Closed	marmoute	D11122 dirstate: add a `set_possibly_dirty` in `fakedirstatewritetime`
Closed	marmoute	D11121 dirstatemap: use `set_possibly_dirty` in `clearambiguoustimes`
Closed	marmoute	D11120 dirstate-item: use `set_possibly_dirty` in `pure.pack_dirstate`
Closed	marmoute	D11119 dirstate-item: add a `set_possibly_dirty` method
Closed	marmoute	D11118 dirstate-item: add a `from_v1_data` constructor
Closed	marmoute	D11117 dirstate-item: use an explicit __init__ function instead of the attrs one
Closed	marmoute	D11116 dirstate: deprecated `drop` outside of `update/merge`
Closed	marmoute	D11115 dirstate: deprecated `remove` outside of `update/merge`
Closed	marmoute	D11114 dirstate: deprecate the `add` method outside of update/merge context
Closed	marmoute	D11113 largefiles: replace use of synclfdirstate with drop
Closed	marmoute	D11112 largefile: use `parentchange` markcommitted
Closed	marmoute	D11111 largefile: use `set_untracked` in the override of copy/rename
Closed	marmoute	D11110 largefile: use `set_untracked` in the `forget` override
Closed	marmoute	D11109 largefile: directly use set_untracked() for removing files
Closed	marmoute	D11108 largefile: use `parentchange` during rollback
Closed	marmoute	D11107 largefile: consider `updatelfiles` as a `parentchange`
Closed	marmoute	D11106 largefile: use `set_tracker` in the `hg copy` overwrite
Closed	marmoute	D11105 largefile: use parentchange during mergerecordupdates
Closed	marmoute	D11104 largefile: use `set_tracker` in the `hg add` overwrite
Closed	marmoute	D11103 largefile: synchronise the dirstate within a `parentchange` context

Diff 29552

mercurial/dirstate.py

	filename,			filename,
	wc_tracked,			wc_tracked,
	p1_tracked,			p1_tracked,
	p2_tracked=False,			p2_tracked=False,
	merged=False,			merged=False,
	clean_p1=False,			clean_p1=False,
	clean_p2=False,			clean_p2=False,
	possibly_dirty=False,			possibly_dirty=False,
				parentfiledata=None,
	):			):
	"""update the information about a file in the dirstate			"""update the information about a file in the dirstate

	This is to be called when the direstates parent changes to keep track			This is to be called when the direstates parent changes to keep track
	of what is the file situation in regards to the working copy and its parent.			of what is the file situation in regards to the working copy and its parent.

	This function must be called within a `dirstate.parentchange` context.			This function must be called within a `dirstate.parentchange` context.

	assert p2_tracked			assert p2_tracked
	self.otherparent(filename)			self.otherparent(filename)
	elif not p1_tracked and p2_tracked and wc_tracked:			elif not p1_tracked and p2_tracked and wc_tracked:
	self._addpath(filename, from_p2=True, possibly_dirty=possibly_dirty)			self._addpath(filename, from_p2=True, possibly_dirty=possibly_dirty)
	self._map.copymap.pop(filename, None)			self._map.copymap.pop(filename, None)
	elif possibly_dirty:			elif possibly_dirty:
	self._addpath(filename, possibly_dirty=possibly_dirty)			self._addpath(filename, possibly_dirty=possibly_dirty)
	elif wc_tracked:			elif wc_tracked:
	self.normal(filename)			self.normal(filename, parentfiledata=parentfiledata)
	# XXX We need something for file that are dirty after an update			# XXX We need something for file that are dirty after an update
	else:			else:
	assert False, 'unreachable'			assert False, 'unreachable'

	@requires_parents_change
	def update_parent_file_data(self, f, filedata):
	"""update the information about the content of a file

	This function should be called within a `dirstate.parentchange` context.
	"""
	self.normal(f, parentfiledata=filedata)

	def _addpath(			def _addpath(
	self,			self,
	f,			f,
	mode=0,			mode=0,
	size=None,			size=None,
	mtime=None,			mtime=None,
	added=False,			added=False,
	merged=False,			merged=False,

mercurial/mergestate.py

	f,			f,
	p1_tracked=False,			p1_tracked=False,
	p2_tracked=True,			p2_tracked=True,
	wc_tracked=True,			wc_tracked=True,
	clean_p2=True,			clean_p2=True,
	)			)
	else:			else:
	parentfiledata = getfiledata[f] if getfiledata else None			parentfiledata = getfiledata[f] if getfiledata else None
	repo.dirstate.update_parent_file_data(f, parentfiledata)			repo.dirstate.update_file(
				f,
				p1_tracked=True,
				wc_tracked=True,
				parentfiledata=parentfiledata,
				)

	# merge			# merge
	for f, args, msg in actions.get(ACTION_MERGE, []):			for f, args, msg in actions.get(ACTION_MERGE, []):
	f1, f2, fa, move, anc = args			f1, f2, fa, move, anc = args
	if branchmerge:			if branchmerge:
	# We've done a branch merge, mark this file as merged			# We've done a branch merge, mark this file as merged
	# so that we properly record the merger later			# so that we properly record the merger later
	repo.dirstate.update_file(			repo.dirstate.update_file(

This is an archive of the discontinued Mercurial Phabricator instance.

dirstate: replace `update_parent_file_data` with simpler `update_parent` callClosedPublic

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 29552

mercurial/dirstate.py

mercurial/mergestate.py

dirstate: replace `update_parent_file_data` with simpler `update_parent` call
ClosedPublic

Revision Contents
Changeset List