This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	durin42	D8375 manifest: leave a TODO where we may have more work for sha1 portability
Abandoned	durin42	D8424 manifest: fix missing `el` on an `elif` spotted by yuya
Closed	durin42	D8374 fuzz: tell manifest fuzzer about longer node hashes
Closed	durin42	D8373 manifest: add test for long hashes
Closed	durin42	D8372 manifest: remove a final 40-byte assumption from pure-python parser
Closed	durin42	D8371 manifest: fix yet another 20-byte-hash assumption
Closed	durin42	D8370 manifest: fix another pure-parsing 20-byte assumption
Closed	durin42	D8369 manifest: teach treemanifest about long hashes
Closed	durin42	D8368 manifest: start removing 40-byte hash restrictions from C code
Closed	durin42	D8367 manifest: remove a 20-byte-hash assumption from pure manifest parsing

Diff 21096

	if prev is not None and prev > l:			if prev is not None and prev > l:
	raise ValueError(b'Manifest lines not in sorted order.')			raise ValueError(b'Manifest lines not in sorted order.')
	prev = l			prev = l
	f, n = l.split(b'\0')			f, n = l.split(b'\0')
	nl = len(n)			nl = len(n)
	if 64 < nl:			if 64 < nl:
	# modern hash, full width			# modern hash, full width
	yield f, bin(n[:64]), n[64:]			yield f, bin(n[:64]), n[64:]
	if 40 < nl < 45:			elif 40 < nl < 45:
	# legacy hash, always sha1			# legacy hash, always sha1
	yield f, bin(n[:40]), n[40:]			yield f, bin(n[:40]), n[40:]
	else:			else:
	yield f, bin(n), b''			yield f, bin(n), b''


	def _text(it):			def _text(it):
	files = []			files = []

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	21096			Apr 15 2020, 10:56 AM	★	★