This is an archive of the discontinued Mercurial Phabricator instance.

procutil: move mainfrozen() to new

Authored by martinvonz on Nov 15 2019, 2:12 PM.



The i18n module practically depends on procutil for mainfrozen() but
since procutil depends on i18n, it would be a circular dependency if
i18n depended directly on procutil. The cycle is currently resolved by
having the higher-level util module calculate the "datapath" and
inject it into i18n. Extracting mainfrozen() to a new module lets us
clean up the dependencies.

Diff Detail

rHG Mercurial
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

martinvonz created this revision.Nov 15 2019, 2:12 PM
indygreg accepted this revision.Nov 16 2019, 2:04 PM
indygreg added a subscriber: indygreg.

I'm not super thrilled about yet another module in the startup path. But the reasoning makes sense. We can always refactor later if the post resources world turns out to remove the circular dependency.

This revision is now accepted and ready to land.Nov 16 2019, 2:04 PM
This revision was automatically updated to reflect the committed changes.