From: Pete Wyckoff <pw@padd.com>
Date: Thu, 18 Aug 2011 15:18:47 GMT
Subject: Re: [PATCH 1/4] git-p4: Allow setting rename/copy detection threshold.
Message-ID: <20110818151847.GA2043@padd.com>
In-Reply-To: <1313629459-22937-2-git-send-email-vitor.hda@gmail.com>

vitor.hda@gmail.com wrote on Thu, 18 Aug 2011 02:04 +0100:
> Copy and rename detection arguments (-C and -M) allow setting a threshold value
> for the similarity ratio. If the similarity is below this threshold the rename
> or copy is ignored and the file is added as new.
> This patch allows setting git-p4.detectRenames and git-p4.detectCopies options
> to an integer value to set the respective threshold.
[..]
> -        if self.detectRenames:
> +        diffOpts = ""
> +        if self.detectRenames.lower() == "true":
>              diffOpts = "-M"
> -        else:
> -            diffOpts = ""
> +        elif self.detectRenames != "":
> +            self.detectRenames = int(self.detectRenames)
> +            if self.detectRenames >= 0 and self.detectRenames <= 100:
> +                diffOpts = "-M%d" % self.detectRenames
>  

This is a good idea.  But I notice that -M, -C, -B can take
a diverse set of option forms, including floating point and
optional "%".  "-B" can take two args, separated by a "/".

See diff_scoreopt_parse().  It would be weird for git-p4 to limit
that set to integers like you do here.

Would you be willing to change it to pass the string blindly?
Just filter out the special values, and pass everything else
unchanged.

    "true" -> "-M"
    "false" -> ""
    "" -> ""
    ".*" -> "\1"

Dunno if you care about --break-rewrites.  Just pointing to it
as a similar option to -M and -C in case you missed it.

		-- Pete