Page MenuHomePhabricator
Differential D15945

Correctly detect new version of xhpast build
ClosedPublic
Actions

Authored by richardvanvelzen on May 19 2016, 10:52 AM.
Tags
None
Referenced Files
F15403529: D15945.diff
Tue, Mar 18, 3:43 AM
F15400118: D15945.diff
Mon, Mar 17, 9:49 AM
F15384362: D15945.id38395.diff
Fri, Mar 14, 7:38 PM
F15382809: D15945.id38398.diff
Fri, Mar 14, 2:33 PM
F15381167: D15945.diff
Fri, Mar 14, 6:13 AM
Unknown Object (File)
Thu, Feb 27, 5:34 AM
Unknown Object (File)
Wed, Feb 26, 8:36 PM
Unknown Object (File)
Feb 16 2025, 6:51 AM
View All Files
Subscribers
Firehed
Korvin

Details

Reviewers
epriestley
chad
Group Reviewers
Blessed Reviewers
Commits
rPHU7ff4bd8dfb0b: (stable) Promote 2016 Week 21
rPHUbd56873ae4c0: Correctly detect new version of xhpast build
Summary

When running lint after xhpast is updated, it tests whether the version is up-to-date. Because the version isn't reset before the test, it would think that the build failed, requiring a second run of lint

Test Plan

Downgraded xhpast, ran lint. Saw broken xhpast, but correct xhpast on second run. Applied patch, and got correct lint results regardless of version.

Diff Detail

Repository
rPHU libphutil
Branch
rebuild-xhpast
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12267
Build 15501: arc lint + arc unit

Event Timeline

richardvanvelzen updated this revision to Diff 38395.May 19 2016, 10:52 AM
richardvanvelzen retitled this revision from to Correctly detect new version of xhpast build.
richardvanvelzen updated this object.
richardvanvelzen edited the test plan for this revision. (Show Details)
richardvanvelzen added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 19 2016, 10:52 AM
Herald added subscribers: Firehed, Korvin. · View Herald Transcript
richardvanvelzen added a comment.May 19 2016, 10:53 AM

Not really high-prio, but a minor annoyance after getting a fresh xhpast version.

epriestley accepted this revision.May 19 2016, 1:53 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 19 2016, 1:53 PM
chad accepted this revision.May 19 2016, 2:04 PM
chad added a reviewer: chad.
Closed by commit rPHUbd56873ae4c0: Correctly detect new version of xhpast build (authored by richardvanvelzen, committed by richardvanvelzen). · Explain WhyMay 19 2016, 2:08 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
parser/
xhpast/
bin/
4 lines

Diff 38395

View Options

src/parser/xhpast/bin/PhutilXHPASTBinary.php

Loading...
Phabricator · Built by Phacility