Page MenuHomePhabricator

In the "Version Information" panel, try to include branchpoints
ClosedPublic

Authored by epriestley on Dec 23 2016, 4:34 PM.
Tags
None
Referenced Files
F18648447: D17103.id41140.diff
Sat, Sep 20, 10:07 AM
F18563713: D17103.diff
Tue, Sep 9, 7:53 AM
F18442921: D17103.diff
Sun, Aug 31, 6:32 PM
F18124241: D17103.id.diff
Aug 13 2025, 12:33 PM
F17950269: D17103.id.diff
Aug 1 2025, 12:14 AM
F17729570: D17103.id.diff
Jul 19 2025, 7:29 PM
F17636337: D17103.id.diff
Jul 11 2025, 10:48 AM
F17632678: D17103.id.diff
Jul 10 2025, 3:43 PM
Subscribers
None

Details

Summary

Fixes T12040. In T12039, a user running local patches followed the report instructions as far as grabbing version information, but didn't update or revert their local changes or try against a clean install before reporting.

This obviously isn't ideal for us, but it's understandable (grabbing version information is much easier than upgrading/reverting), and we can do better about making this information useful: when compiling version information, try to figure out the branchpoint from a known upstream master branch by listing remotes, then running git merge-base against them.

Additionally, explicitly document that we want upstream hashes. We have to have a fallback case in this document anyway (for when you can't get to Config) so hopefully this makes it more likely that we get useful information in initial reports.

Test Plan

Screen Shot 2016-12-23 at 8.24.44 AM.png (765×1 px, 218 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable