[PATCH] convert: Perforce source for conversion to Mercurial

[timeless]

Frank Kingswood wrote:
> diff -r cf427b04d5c0 -r d602f5013d17 hgext/convert/__init__.py
> +    The Perforce (P4) importer does a straight import, ignoring labels,
> +    branches and integrations. It will set a tag on the final changeset
> +    referencing the Perforce changelist number.
> +
> +    Source history can be retrieved starting at a specific revision,
> +    instead of being integrally converted. Only single branch

please excuse my ignorance, but I don't understand "integrally" in
this sentence. Could you please explain or rephrase?

> +    conversions are supported.
> +
> +    --config convert.p4.startrev=0           (perforce changelist number)
> +        specify start Perforce revision.

specify the starting Perforce revision

or initial or ....

> diff -r cf427b04d5c0 -r d602f5013d17 hgext/convert/p4.py
> +# Mercurial import from Perforce.

I should check to see if this is the standard way to write this.

This appears to be a sentence but fails my parser.

Import from Perforce into Mercurial.

> +# This software may be used and distributed according to the terms
> +# of the GNU General Public License, incorporated herein by reference.

i should check, but it seems odd to me that this is an unversioned referenced.

> +    def _parse(self, ui, path):
> +        "Prepare list of P4 filenames and revisions for import"

for => to ?

> +        ui.status(_('reading p4 views\n'))

> +        # prepare a list to check depot filenames to make local view

?