[PATCH] use createrepo_c instead of createrepo to make repos if it is available

[Dennis Gilmore]

Signed-off-by: Dennis Gilmore <dennis(a)ausil.us&gt;
---
 builder/kojid | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/builder/kojid b/builder/kojid
index 3a92ec3..4e53fb5 100755
--- a/builder/kojid
+++ b/builder/kojid
@@ -4252,7 +4252,11 @@ class CreaterepoTask(BaseTaskHandler):
 
     def create_local_repo(self, rinfo, arch, pkglist, groupdata, oldrepo):
         koji.ensuredir(self.outdir)
-        cmd = ['/usr/bin/createrepo', '-vd', '-o', self.outdir]
+        if os.path.isfile('/usr/bin/createrepo_c'):
+            cmd = ['/usr/bin/createrepo_c']
+        else:
+            cmd = ['/usr/bin/createrepo']
+        cmd.extend(['-vd', '-o', self.outdir])
         if pkglist is not None:
             cmd.extend(['-i', pkglist])
         if os.path.isfile(groupdata):
-- 
2.4.0

[Dan Horák]

On Thu,  7 May 2015 11:20:16 -0500
Dennis Gilmore <dennis(a)ausil.us&gt; wrote:

>  Signed-off-by: Dennis Gilmore <dennis(a)ausil.us&gt;
>  ---
>   builder/kojid | 6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
>  
>  diff --git a/builder/kojid b/builder/kojid
>  index 3a92ec3..4e53fb5 100755
>  --- a/builder/kojid
>  +++ b/builder/kojid
>  @@ -4252,7 +4252,11 @@ class CreaterepoTask(BaseTaskHandler):
>   
>       def create_local_repo(self, rinfo, arch, pkglist, groupdata,
>  oldrepo): koji.ensuredir(self.outdir)
>  -        cmd = ['/usr/bin/createrepo', '-vd', '-o', self.outdir]
>  +        if os.path.isfile('/usr/bin/createrepo_c'):
>  +            cmd = ['/usr/bin/createrepo_c']
>  +        else:
>  +            cmd = ['/usr/bin/createrepo']
>  +        cmd.extend(['-vd', '-o', self.outdir])
>           if pkglist is not None:
>               cmd.extend(['-i', pkglist])
>           if os.path.isfile(groupdata): 
ACK, the same logic as in my patch from
https://lists.fedoraproject.org/pipermail/buildsys/2014-July/004310.html


		Dan

[Dennis Gilmore]

On Thursday, May 07, 2015 11:20:16 AM you wrote:
>  Signed-off-by: Dennis Gilmore <dennis(a)ausil.us&gt;
>  ---
>   builder/kojid | 6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
>  
>  diff --git a/builder/kojid b/builder/kojid
>  index 3a92ec3..4e53fb5 100755
>  --- a/builder/kojid
>  +++ b/builder/kojid
>  @@ -4252,7 +4252,11 @@ class CreaterepoTask(BaseTaskHandler):
>  
>       def create_local_repo(self, rinfo, arch, pkglist, groupdata, oldrepo):
>           koji.ensuredir(self.outdir)
>  -        cmd = ['/usr/bin/createrepo', '-vd', '-o', self.outdir]
>  +        if os.path.isfile('/usr/bin/createrepo_c'):
>  +            cmd = ['/usr/bin/createrepo_c']
>  +        else:
>  +            cmd = ['/usr/bin/createrepo']
>  +        cmd.extend(['-vd', '-o', self.outdir])
>           if pkglist is not None:
>               cmd.extend(['-i', pkglist])
>           if os.path.isfile(groupdata): 
ping

[Dan Horák]

On Sat, 06 Jun 2015 15:38:06 -0500
Dennis Gilmore <dennis(a)ausil.us&gt; wrote:

>  On Thursday, May 07, 2015 11:20:16 AM you wrote:
>  > Signed-off-by: Dennis Gilmore <dennis(a)ausil.us&gt;
>  > ---
>  >  builder/kojid | 6 +++++-
>  >  1 file changed, 5 insertions(+), 1 deletion(-)
>  > 
>  > diff --git a/builder/kojid b/builder/kojid
>  > index 3a92ec3..4e53fb5 100755
>  > --- a/builder/kojid
>  > +++ b/builder/kojid
>  > @@ -4252,7 +4252,11 @@ class CreaterepoTask(BaseTaskHandler):
>  > 
>  >      def create_local_repo(self, rinfo, arch, pkglist, groupdata,
>  > oldrepo): koji.ensuredir(self.outdir)
>  > -        cmd = ['/usr/bin/createrepo', '-vd', '-o',
> self.outdir]
>  > +        if os.path.isfile('/usr/bin/createrepo_c'):
>  > +            cmd = ['/usr/bin/createrepo_c']
>  > +        else:
>  > +            cmd = ['/usr/bin/createrepo']
>  > +        cmd.extend(['-vd', '-o', self.outdir])
>  >          if pkglist is not None:
>  >              cmd.extend(['-i', pkglist])
>  >          if os.path.isfile(groupdata):
>  ping 
ping^2

I have realized that our new s390 hub is calling the old createrepo :-(


		Dan