This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	martinvonz	D9390 errors: introduce SecurityError and use it in a few places
Closed	martinvonz	D9389 errors: raise InputError when too few arguments given to alias
Closed	martinvonz	D9388 errors: raise InputError on bad top-level flags
Closed	martinvonz	D9393 errors: raise StateError on uncommitted changes when merge starts
Closed	martinvonz	D9392 errors: raise StateError when there are unresolves merge conflicts
Closed	martinvonz	D9602 errors: raise InputError on recursive template definition
Closed	martinvonz	D9391 errors: raise StateError when push fails because it creates new heads
Closed	martinvonz	D9387 errors: raise InputError on early parse error in dispatch
Closed	martinvonz	D9386 errors: raise more specifc errors from narrowcommands
Closed	martinvonz	D9601 errors: use detailed exit code 50 for StorageError
Closed	martinvonz	D9600 errors: raise InputError if an ambiguous revision id prefix is used
Closed	martinvonz	D9599 localrepo: delete obsolete comment about `prefix in repo` raising exception

Diff 24271

	# unlike runsymbol(), here 'symbol' is always taken as template name			# unlike runsymbol(), here 'symbol' is always taken as template name
	# even if it exists in mapping. this allows us to override mapping			# even if it exists in mapping. this allows us to override mapping
	# by web templates, e.g. 'changelogtag' is redefined in map file.			# by web templates, e.g. 'changelogtag' is redefined in map file.
	return context._load(exp[1])			return context._load(exp[1])
	raise error.ParseError(_(b"expected template specifier"))			raise error.ParseError(_(b"expected template specifier"))


	def _runrecursivesymbol(context, mapping, key):			def _runrecursivesymbol(context, mapping, key):
	raise error.Abort(_(b"recursive reference '%s' in template") % key)			raise error.InputError(_(b"recursive reference '%s' in template") % key)


	def buildtemplate(exp, context):			def buildtemplate(exp, context):
	ctmpl = [compileexp(e, context, methods) for e in exp[1:]]			ctmpl = [compileexp(e, context, methods) for e in exp[1:]]
	return (templateutil.runtemplate, ctmpl)			return (templateutil.runtemplate, ctmpl)


	def buildfilter(exp, context):			def buildfilter(exp, context):


	buildmap() -> gettemplate(), where no thunk was made:			buildmap() -> gettemplate(), where no thunk was made:

	$ cat << EOF > issue4758			$ cat << EOF > issue4758
	> changeset = '{files % changeset}\n'			> changeset = '{files % changeset}\n'
	> EOF			> EOF
	$ hg log --style ./issue4758			$ hg log --style ./issue4758
	abort: recursive reference 'changeset' in template			abort: recursive reference 'changeset' in template
	[255]			[10]

	not a recursion if a keyword of the same name exists:			not a recursion if a keyword of the same name exists:

	$ cat << EOF > issue4758			$ cat << EOF > issue4758
	> changeset = '{tags % rev}'			> changeset = '{tags % rev}'
	> rev = '{rev} {tag}\n'			> rev = '{rev} {tag}\n'
	> EOF			> EOF
	$ hg log --style ./issue4758 -r tip			$ hg log --style ./issue4758 -r tip