This is an archive of the discontinued Mercurial Phabricator instance.

sparse: make path absoluteness checks cross-platform
ClosedPublic

Authored by ikostia on Jul 28 2017, 7:05 AM.

Download Raw Diff

Details

Reviewers

mjpieters
simonfar

Group Reviewers

Restricted Project

Commits

rFBHGXc6e1f7828465: sparse: make path absoluteness checks cross-platform

Summary

Let's be friendlier to our friends on Windows.

Test Plan

Diff Detail

Repository

rFBHGX Facebook Mercurial Extensions

Branch

default

Lint

Lint OK

Unit

Unit Tests OK

Event Timeline

ikostia created this revision.Jul 28 2017, 7:05 AM

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 28 2017, 7:05 AM

simonfar requested changes to this revision.Jul 28 2017, 8:19 AM

simonfar added a subscriber: simonfar.

simonfar added inline comments.

hgext3rd/sparse.py
762	This is no longer true - can you rephrase it?

This revision now requires changes to proceed.Jul 28 2017, 8:19 AM

arddress @simonfar's comment

simonfar accepted this revision.Jul 28 2017, 9:18 AM

This revision is now accepted and ready to land.Jul 28 2017, 9:18 AM

Closed by commit rFBHGXc6e1f7828465: sparse: make path absoluteness checks cross-platform (authored by ikostia). · Explain WhyJul 28 2017, 9:29 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			hgext3rd/sparse.py (2 lines)

Diff	ID	Description	Created	Lint	Unit
Base		Base
Diff 1	434		Jul 28 2017, 7:05 AM	★	★
Diff 2	436	arddress @simonfar's comment	Jul 28 2017, 9:15 AM	★	★
Diff 3	444	rFBHGXc6e1f782846528475476546a1fb919f4057813f1	Jul 28 2017, 9:23 AM	★	★

Commit	Local	Parents	Author	Summary	Date
acad2c29f989	3608	016faf2c99b0	Kostia Balytskyi	sparse: make path absoluteness checks cross-platform	Jul 28 2017, 7:02 AM

Diff 434

hgext3rd/sparse.py

	newinclude = set()			newinclude = set()
	newexclude = set()			newexclude = set()
	newprofiles = set()			newprofiles = set()
	else:			else:
	newinclude = set(oldinclude)			newinclude = set(oldinclude)
	newexclude = set(oldexclude)			newexclude = set(oldexclude)
	newprofiles = set(oldprofiles)			newprofiles = set(oldprofiles)

	if any(pat.startswith('/') for pat in pats):			if any(os.path.isabs(pat) for pat in pats):
	err = _('paths cannot start with /')			err = _('paths cannot start with /')
				simonfarUnsubmitted Not Done This is no longer true - can you rephrase it? simonfar: This is no longer true - can you rephrase it?
	hint = _('do not use absolute paths')			hint = _('do not use absolute paths')
	raise error.Abort(err, hint=hint)			raise error.Abort(err, hint=hint)

	if (not ui.configbool('sparse', 'includereporootpaths', False)			if (not ui.configbool('sparse', 'includereporootpaths', False)
	and (include or exclude)):			and (include or exclude)):
	# supplied file patterns should be treated as relative			# supplied file patterns should be treated as relative
	# to current working dir, so we need to convert them first			# to current working dir, so we need to convert them first
	pats = scmutil.match(repo['.'], pats).files()			pats = scmutil.match(repo['.'], pats).files()