[PATCH] management: Create 'rehash' command
Finucane, Stephen
stephen.finucane at intel.com
Mon Nov 30 08:44:03 AEDT 2015
On 24 Nov 21:22, Finucane, Stephen wrote:
> On 24 Nov 14:15, Jeremy Kerr wrote:
> > Hi Stephen,
> >
> > >> The rehash script, though undocumented and possibly unused at the
> > >> moment, likely has some value to some users. Howver, it makes more
> > >> sense to provide this command as a management command like 'retag'.
> > >> Do this.
> > >>
> > >> Signed-off-by: Stephen Finucane <stephen.finucane at intel.com>
> > >
> > > I'm still not sure if this should be converted to a command or just
> > > deleted. Does anyone use it? It seems like the hash could never really
> > > get out of date without manual DB modification (in which case you have
> > > more than one problem) thanks to the 'save' method.
> >
> > This was useful when the hashes were first introduced, but wouldn't be
> > used much now. If we did need to change the hashing algorithm, (eg, move
> > to SHA1, or change which parts of a patch are included in the hashed
> > text), then it'd be required for that.
> >
> > I'd say keep it in, since you've already done the work converting it :)
>
> ...and that's exactly what I'll do. Thanks, Jeremy.
>
> Cheers,
> Stephen
Merged.
More information about the Patchwork
mailing list