Page MenuHomePhabricator

Revert "Reorder remarkup block rules for consistency with PhabricatorMarkupEngine"
ClosedPublic

Authored by richardvanvelzen on May 6 2016, 3:11 PM.
Tags
None
Referenced Files
F15404976: D15862.id.diff
Tue, Mar 18, 9:53 AM
F15402841: D15862.diff
Tue, Mar 18, 12:36 AM
F15391346: D15862.id38208.diff
Sat, Mar 15, 9:26 AM
F15388345: D15862.id38207.diff
Sat, Mar 15, 3:41 AM
Unknown Object (File)
Wed, Feb 26, 10:05 PM
Unknown Object (File)
Feb 16 2025, 6:47 AM
Unknown Object (File)
Feb 9 2025, 1:31 AM
Unknown Object (File)
Jan 29 2025, 4:59 PM
Subscribers
Tokens
"Doubloon" token, awarded by epriestley.

Details

Summary

This reverts commit c0370f2f3c140d5048034ffdab617af7631d14ad.

The sorting behavior has changed between PHP 5 and 7. The msort at libphutil/src/markup/engine/PhutilRemarkupEngine.php:44 reorders slightly differently.

Test Plan

php5 ../arcanist/scripts/arcanist.php unit now doesn't show an error.

Diff Detail

Repository
rPHU libphutil
Branch
revert-remarkup-fix
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12082
Build 15222: arc lint + arc unit

Event Timeline

richardvanvelzen retitled this revision from to Revert "Reorder remarkup block rules for consistency with PhabricatorMarkupEngine".
richardvanvelzen updated this object.
richardvanvelzen edited the test plan for this revision. (Show Details)
richardvanvelzen added a reviewer: epriestley.

Thanks. I filed T10929 to fix this properly.

Awesome.

I'll create a dedicated PHP 5 vm to use for Phabricator development from now on. I'm not wanting to risk this kind of breakage.

(I'll be away from my machine for the weekend, when accepted it'd be best if you land this since I probably won't be able to)

This revision is now accepted and ready to land.May 6 2016, 3:15 PM

I'll land this in, like, 5 minutes if it seems like you immediately stood up from your desk and returned to your mountain fortress after mashing submit.

This revision was automatically updated to reflect the committed changes.