short koji repo question

[Jonathan Dieter]

On Thu, 2007-10-04 at 13:40 +0200, Oliver Falk wrote:
> Well. Sooner or later you'll have sqlite support for createrepo on your
> machines and then:
> 
> --- /usr/sbin/kojid     2007-10-04 13:36:32.000000000 +0200
> +++ /tmp/kojid  2007-10-04 13:36:15.000000000 +0200
> @@ -2244,7 +2244,7 @@
>          outdir = "%s/repo" % self.workdir
>          datadir = "%s/repodata" % outdir
>          koji.ensuredir(outdir)
> -        cmd = ['/usr/bin/createrepo', '-vpd', '--outputdir', outdir]
> +        cmd = ['/usr/bin/createrepo', '-vp', '--outputdir', outdir]
>          if os.path.exists("%s/comps.xml" % repodir):
>              cmd.extend(['-g', 'comps.xml'])
>          #attempt to recycle repodata from last repo
> @@ -2272,7 +2272,7 @@
> 
>          files = []
>          for f in os.listdir(datadir):
> -            if f.endswith('.xml') or f.endswith('.xml.gz') or
> f.endswith('.bz2'):
> +            if f.endswith('.xml') or f.endswith('.xml.gz'):
>                  files.append(f)
>                  session.uploadWrapper("%s/%s" % (datadir, f),
> uploadpath, f)
> 
> 
> I can do more digging and check if I can add some config key and >if<
> that two lines of code... Or we simply check which version of createrepo
> introduced -d and BR it in koji spec...
> 
> I can also open a ticket on hosted, if you prefer!?
> 
> Best,
>  Oliver
> 

I think your patch is backward.  Shouldn't the -'s be +'s and visa
versa?

Jonathan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://listman.redhat.com/archives/fedora-devel-list/attachments/20071004/e045882b/attachment.sig>