Page MenuHomePhabricator

prefetch: fix the hg executable location for background prefetch
ClosedPublic

Authored by singhsrb on Dec 5 2017, 5:17 PM.

Details

Summary

The background prefetch uses the incorrect executable while running
the prefetch command due to which there are no logs for the background
prefetch. This commit fixes that by specifying the correct executable.

Test Plan
  • Ran all the tests.
  • Tested the output of util.hgexecutable() with system hg and chg, and verified that it returns the correct path. As expected, util.hgcmd() returns the corresponding path for hg and chg in this case.

Diff Detail

Repository
rFBHGX Facebook Mercurial Extensions
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

singhsrb created this revision.Dec 5 2017, 5:17 PM
Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 5 2017, 5:17 PM

Basically the same issue as D1221. I am not even sure if this is used anywhere but I guess no harm in fixing it.

durham accepted this revision.Dec 5 2017, 5:24 PM
This revision is now accepted and ready to land.Dec 5 2017, 5:24 PM
This revision was automatically updated to reflect the committed changes.