Page MenuHomePhabricator

packaging: add configparser to inno requirements file
ClosedPublic

Authored by indygreg on Wed, Jan 22, 11:03 PM.

Details

Summary

This dependency is missing and pip complains about it in
strict hashing mode. How this was missed, I have no clue.

Diff Detail

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

Event Timeline

indygreg created this revision.Wed, Jan 22, 11:03 PM
pulkit accepted this revision.Thu, Jan 23, 7:12 AM
This revision is now accepted and ready to land.Thu, Jan 23, 7:12 AM
This revision was automatically updated to reflect the committed changes.

This is needed on stable to be able to build the Inno installer there.

Yeah, it was flagged as stable in the branch metadata. But whoever queued it didn't see that. (Perhaps we should improve hg phabread to warn on branch name mismatch?)

I'll manually queue this for stable.

Yeah, it was flagged as stable in the branch metadata. But whoever queued it didn't see that. (Perhaps we should improve hg phabread to warn on branch name mismatch?)
I'll manually queue this for stable.

I'd be interested in something like this. But we've got more than default and stable at work, so it might need to be on the import side? The --exact parameter won't generally work because e.g. the amend entry in extras doesn't get carried in the patch, so that would fail to apply entirely. So it would need to be something like --exact-parent, which probably isn't as useful for Mercurial reviewers. Coupled with --bypass, you don't really need to be on the correct branch, I don't think. But it all seems like a related problem.