On Wed, Apr 06, 2011 at 12:52:49PM +0200, Bjoern Ricks wrote: > Hi Guido, > > On Tuesday 05 April 2011 14:21:36 Guido Günther wrote: > > This code isn't in the tree, it's: > > if repo.find_version(options.debian_tag, src.version): > > > > src.debian_verfsion only contains the last part of the version number. > > > > However using the full version is correct since that's what you're > > looking at. Can you provide a test case that fails for you? > > Ups, that debian_version change got committed accidentially. It was not > intended. It should be src.version.
> > An easy test case is to import a source package with an epoch version twice. > The second time it should be caught by > if repo.find_version > but because the src.version contains the epoch and on the first run the tag > was created without epoch it's not. Currently I am not sure if it's better to > include the epoch in the tag or change the find_version statement accordingly > to my patch. The bug here is that we dropped the epoch silently. I've fixed this and pushed the fix to git. Thanks for your prompt feedback! -- Guido -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
