Differences between Phabricator and Phorge
This page can help to sum up the differences between Phabricator and Phorge to prevent conflicts or duplicate work.
Normal users probably are not interested in this page. This page is internally useful since Phorge often introduces changes not developed by Phabricator. But, sometime Phabricator also introduces unique changes, that could be pulled in Phorge. Some times, we do the same thing but with different approaches.
In short
Phorge has many more bug fixes and features than Phabricator 😎
This page just shows:
- commits that are only present in Phabricator (very few)
- commits that are somehow present in both
So that we can eventually import them in Phorge if we like that.
Differences from upstream Phabricator
https://secure.phabricator.com/source/phabricator/history/master/
List of recent changes (most recent up):
| Phab Topic | in Phabricator | in Phorge | Notes |
|---|---|---|---|
| PHP8 refactor ProfileMenu | 58995268dd97 | we fixed in a different way | Could be interesting to merge in Phorge. |
| PHP8 fixes | f6214f060e78 | (we have these) | Basically in Phab they reinvented the wheel after 3 months of day-by-day fixes in Phorge. They are probably missing things we already fixed. |
| Fix Figlet regression | bc6f4786a2e3 | rP71e4eee275f5 | Phab has just a fix for this ↓. Phorge does not need this. |
| Update Figlet in PHP8 | d5c63c86e7e4 | rP71e4eee275f5 | Phab introduced a regression in line 150. Phorge does not need that fix. |
| Fix Pholio | a83cb99e856a | rPa8a38835e15e | Phorge can take inspiration. |
| (no older differences) | (no older differences) | ||
Differences from upstream Arcanist
https://secure.phabricator.com/diffusion/ARC/history/master/
List of recent changes (most recent up):
| Phab Topic | in Phabricator | in Phorge | Notes |
|---|---|---|---|
| Fix for Mercurial 6.4 | 0fc22183e796 | rARC5cdafa4002d8 | C. Speck betrayed Phorge 🤨 than turned back ✨ |
| Fix utf8_decode() | d87d5f0e02e2 | rARC08dfffd5caf7 | The approach in Phorge is more efficient. |
| Fix Diff PHP8.1 | c39ab20eb371 | rARC0e32dbc1ac8f | |
| (no older differences) | (no older differences) | ||
Unique features of Phorge
Apart from the lines in common mentioned above,
To discover how many bug fixes and features are Phorge-only, explore the Phorge's Change Log.
- Last Author
- valerio.bozzolan
- Last Edited
- Jul 23 2023, 15:53
Event Timeline
what's the point of this page? it's very easy to get a detailed difference between the code bases using git; Why create a manually-maintained list as well?
@avivey The problem is, I cannot annotate my git with comments and share them to folks in Wikimedia Foundation. Also, it's not simple to have a 1:1 match between our commits and theirs in secure.
Probably a way to automate that, is to add an additional fields in Diffusion commits like "upstream commit hash". But at the moment a manual wiki page solves some problems