HomePhabricator

Remove `break` from hunk migration

Description

Remove break from hunk migration

Summary: This break statement causes ./bin/hunks migrate to only migrate one-hunk-at-a-time, which is unnecessary and slightly misleading. Instead, allow the script to migrate all legacy hunks to modern storage. In particular, this means that we can recommend that installs run this command sometime before D13222 is landed.

Test Plan: It's a pain to setup the data necessary to test this, but this is identical to the change that I made on our production install when I migrated our hunk storage.

Reviewers: epriestley, Blessed Reviewers

Reviewed By: epriestley, Blessed Reviewers

Subscribers: epriestley, Korvin

Differential Revision: https://secure.phabricator.com/D13288

Details

Provenance
joshuaspenceAuthored on
joshuaspencePushed on Jun 14 2015, 10:32 PM
Reviewer
Blessed Reviewers
Differential Revision
D13288: Remove `break` from hunk migration
Parents
rP32f4c962fc6e: Remove "@stable" annotations
Branches
Unknown
Tags
Unknown