1121962 - Make small fixes to upstream sync script

Status: RESOLVED FIXED

(Testing :: web-platform-tests, defect)

Description

[James Graham [:jgraham]]

Turns out that hardcoding jgraham/web-platform-tests isn't a great idea.

Comment 1

[James Graham [:jgraham]]

Attachment: MozReview Request: bz://1121962/jgraham

Comment 2

[James Graham [:jgraham]]

/r/2539 - Bug 1121962 - Small fixes to web-platform-tests upstreaming script

Pull down this commit:

hg pull review -r 668a183c432343bdccfecf553160238cb32e84f6

Comment 3

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://reviewboard.mozilla.org/r/2537/#review1683

::: testing/web-platform/update/upstream.py
(Diff revision 1)
> +        name = name.rstrip(".git")

I recently discovered that rstrip("foo") doesn't do what you'd think it does:

>>> "zgit.tgi".rstrip(".git")
'z'

Comment 4

[James Graham [:jgraham]]

/r/2539 - fixup! Bug 1121962 - Small fixes to web-platform-tests upstreaming script

Pull down this commit:

hg pull review -r cc446ed786111d8698e63bebf63b27ccb14c4bf3

Comment 5

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://reviewboard.mozilla.org/r/2537/#review1685

Ship It!

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/36748b01f705

Comment 8

[James Graham [:jgraham]]

Attachment: MozReview Request: fixup! Bug 1121962 - Small fixes to web-platform-tests upstreaming script