1190494 - Update push_printurls hook to operate on repository relative path

Status: RESOLVED FIXED

(Developer Services :: Mercurial: hg.mozilla.org, defect)

Description

[Gregory Szorc [:gps]]

Expect a patch shortly.

Comment 1

[Gregory Szorc [:gps]]

Attachment: MozReview Request: hghooks: deduce repository path from filesystem path (bug 1190494); r?RyanVM

hghooks: deduce repository path from filesystem path (bug 1190494); r?RyanVM

Before, we maintained a manual mapping of repository directory name to
expanded path. This is a lot of extra work for something we can just
grab from the filesystem.

Refactor the hook to deduce the repository relative path from the
repository's location on the filesystem, relative to a root directory.

The test for an unknown repository not displaying info has been deleted
because the hook is now globally installed.

Comment 2

[Gregory Szorc [:gps]]

Comment on attachment 8642557 [details]
MozReview Request: hghooks: deduce repository path from filesystem path (bug 1190494); r?RyanVM

hghooks: deduce repository path from filesystem path (bug 1190494); r?RyanVM

Before, we maintained a manual mapping of repository directory name to
expanded path. This is a lot of extra work for something we can just
grab from the filesystem.

Refactor the hook to deduce the repository relative path from the
repository's location on the filesystem, relative to a root directory.

The test for an unknown repository not displaying info has been deleted
because the hook is now globally installed.

Comment 3

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8642557 [details]
MozReview Request: hghooks: deduce repository path from filesystem path (bug 1190494); r?RyanVM

https://reviewboard.mozilla.org/r/14729/#review13319

Ship It!

Comment 4

[Gregory Szorc [:gps]]

url:        https://hg.mozilla.org/hgcustom/version-control-tools/rev/ffed96be1576ab3ee5dbde5ff5ce40fa628da4a9
changeset:  ffed96be1576ab3ee5dbde5ff5ce40fa628da4a9
user:       Gregory Szorc <[e-mail]>
date:       Tue Aug 04 11:07:13 2015 -0700
description:
hghooks: deduce repository path from filesystem path (bug 1190494); r=RyanVM

Before, we maintained a manual mapping of repository directory name to
expanded path. This is a lot of extra work for something we can just
grab from the filesystem.

Refactor the hook to deduce the repository relative path from the
repository's location on the filesystem, relative to a root directory.

The test for an unknown repository not displaying info has been deleted
because the hook is now globally installed.

Comment 5

[Gregory Szorc [:gps]]

Deploying this now.

Comment 6

[Gregory Szorc [:gps]]

I've performed some pushes to version-control-tools today and the hook appears to be working as expected.