Hi Guilhem, On Mon, Mar 06, 2023 at 01:37:01PM +0100, Guilhem Moulin wrote: > Hi Guido, > > On Wed, 01 Mar 2023 at 12:14:51 +0100, Guido Günther wrote: > > On Tue, Aug 09, 2022 at 01:07:34PM +0200, Guilhem Moulin wrote: > >> That'd work for me, thanks! Some ideas to cover other use-cases if > >> desired: > >> > >> - Always strip ‘+ds(\.\d*)?’ and ‘+dfsg(\.\d*)?’ repack suffixes > >> *after* version mangling. After all, if upstream uses such suffixes > >> in its tags or version number, then the revision has to be mangled so > >> it doesn't collide with a repack suffix no? > >> - New option --upstream-vcs-tag-strip='\+ds(\.\d*)?$' > > > > I wonder if a simple: > > > > --upstream-vcs-tag-strip > > > > that defaults to 'on' would already do the trick (as stripping this > > should be the norm (as you elaborate in your first point) and the option > > is only there if people want the old behavior. > > That would cover my use-case nicely, at least :-)
Sorry for the delay. I've pushed something based on the patch in #968329. Maybe we can even get away without another option this way. Cheers, -- Guido > > -- > Guilhem.