Closed Bug 547242 Opened 10 years ago Closed 9 years ago
.pl needs to use Release::* from build/tools/lib/perl
We created a couple new Perl modules as part of bug 492913, patcher-config-bump.pl should be updated to use GetProductDetails(), GetReleaseBlock(), BumpFilePath() instead of duplicating their functionality.
9 years ago
9 years ago
No longer blocks: 478420
I had to update BumpFilePath and GetPrettyVersion, because the got out of date with the patcher-config-bum.pl versions (which is exactly why this bug is important!). Specifically, GetPrettyVersion didn't handle the new style Beta pretty names, and BumpFilePath didn't handle Mac platform exceptions correctly. The rest of the patch is just swapping them in where appropriate. The unit tests are broken, sadly, so to test I re-ran the patcher config bump/creation for: - 6.0b2 - 5.0.1 - 3.6.19 (minor & major) All post-bump configs matched the currently checked in copy.
Attachment #545978 - Flags: review?(rail)
Comment on attachment 545978 [details] [diff] [review] update & use Perl library functions Looks good.
Attachment #545978 - Flags: review?(rail) → review+
Attachment #545978 - Flags: checked-in+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.