Discussion:
[LIBREPORT PATCH] can now use custom repo filters for enabling repos, related abrt/abrt#688
Petr Kubat
2013-09-02 13:47:42 UTC
Permalink
Signed-off-by: Petr Kubat <pkubat-H+wXaHxf7aLQT0dZR+***@public.gmane.org>
---
src/client-python/debuginfo.py | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/client-python/debuginfo.py b/src/client-python/debuginfo.py
index 07e54c2..86fd8da 100644
--- a/src/client-python/debuginfo.py
+++ b/src/client-python/debuginfo.py
@@ -165,13 +165,15 @@ def downloadErrorCallback(callBackObj):
return {'fail':0}

class DebugInfoDownload(YumBase):
- def __init__(self, cache, tmp, keep_rpms=False, noninteractive=True):
+ def __init__(self, cache, tmp, repo_pattern, keep_rpms=False,
+ noninteractive=True,):
self.cachedir = cache
self.tmpdir = tmp
global tmpdir
tmpdir = tmp
self.keeprpms = keep_rpms
self.noninteractive = noninteractive
+ self.repo_pattern = repo_pattern
YumBase.__init__(self)
mute_stdout()
#self.conf.cache = os.geteuid() != 0
@@ -240,7 +242,7 @@ class DebugInfoDownload(YumBase):
# setting-up repos one-by-one, so we can skip the broken ones...
# this helps when users are using 3rd party repos like rpmfusion
# in rawhide it results in: Can't find valid base url...
- for r in self.repos.findRepos(pattern="*debug*"):
+ for r in self.repos.findRepos(pattern=self.repo_pattern):
try:
rid = self.repos.enableRepo(r.id)
self.repos.doSetup(thisrepo=str(r.id))
--
1.8.3.1
Jiri Moskovcak
2013-09-03 10:27:01 UTC
Permalink
Post by Petr Kubat
---
src/client-python/debuginfo.py | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/client-python/debuginfo.py b/src/client-python/debuginfo.py
index 07e54c2..86fd8da 100644
--- a/src/client-python/debuginfo.py
+++ b/src/client-python/debuginfo.py
return {'fail':0}
+ def __init__(self, cache, tmp, repo_pattern, keep_rpms=False,
- please use keyword argument repo_pattern="*debug*", so abrt works even
without the next patch
Post by Petr Kubat
self.cachedir = cache
self.tmpdir = tmp
global tmpdir
tmpdir = tmp
self.keeprpms = keep_rpms
self.noninteractive = noninteractive
+ self.repo_pattern = repo_pattern
YumBase.__init__(self)
mute_stdout()
#self.conf.cache = os.geteuid() != 0
# setting-up repos one-by-one, so we can skip the broken ones...
# this helps when users are using 3rd party repos like rpmfusion
# in rawhide it results in: Can't find valid base url...
rid = self.repos.enableRepo(r.id)
self.repos.doSetup(thisrepo=str(r.id))
Petr Kubat
2013-09-03 11:04:54 UTC
Permalink
fixed default repository search pattern
this patch now also depends on previous patches "[LIBREPORT PATCH 1/2] fixed exception handling - closes #173" and "[LIBREPORT PATCH 2/2 v2] add docstrings, remove global variable - related #171"

Signed-off-by: Petr Kubat <pkubat-H+wXaHxf7aLQT0dZR+***@public.gmane.org>
---
src/client-python/debuginfo.py | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/client-python/debuginfo.py b/src/client-python/debuginfo.py
index 698a769..00dce5f 100644
--- a/src/client-python/debuginfo.py
+++ b/src/client-python/debuginfo.py
@@ -228,7 +228,8 @@ class DebugInfoDownload(YumBase):
This class is used to manage download of debuginfos.
"""

- def __init__(self, cache, tmp, keep_rpms=False, noninteractive=True):
+ def __init__(self, cache, tmp, repo_pattern="*debug*", keep_rpms=False,
+ noninteractive=True):
self.old_stdout = -1
self.cachedir = cache
self.tmpdir = tmp
@@ -236,6 +237,7 @@ class DebugInfoDownload(YumBase):
TMPDIR = tmp
self.keeprpms = keep_rpms
self.noninteractive = noninteractive
+ self.repo_pattern=repo_pattern
YumBase.__init__(self)
self.mute_stdout()
#self.conf.cache = os.geteuid() != 0
@@ -336,7 +338,7 @@ class DebugInfoDownload(YumBase):
# setting-up repos one-by-one, so we can skip the broken ones...
# this helps when users are using 3rd party repos like rpmfusion
# in rawhide it results in: Can't find valid base url...
- for r in self.repos.findRepos(pattern="*debug*"):
+ for r in self.repos.findRepos(pattern=self.repo_pattern):
try:
rid = self.repos.enableRepo(r.id)
self.repos.doSetup(thisrepo=str(r.id))
--
1.8.3.1
Jiri Moskovcak
2013-09-05 09:13:23 UTC
Permalink
- pushed, thanks
Post by Petr Kubat
fixed default repository search pattern
this patch now also depends on previous patches "[LIBREPORT PATCH 1/2] fixed exception handling - closes #173" and "[LIBREPORT PATCH 2/2 v2] add docstrings, remove global variable - related #171"
---
src/client-python/debuginfo.py | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/client-python/debuginfo.py b/src/client-python/debuginfo.py
index 698a769..00dce5f 100644
--- a/src/client-python/debuginfo.py
+++ b/src/client-python/debuginfo.py
This class is used to manage download of debuginfos.
"""
+ def __init__(self, cache, tmp, repo_pattern="*debug*", keep_rpms=False,
self.old_stdout = -1
self.cachedir = cache
self.tmpdir = tmp
TMPDIR = tmp
self.keeprpms = keep_rpms
self.noninteractive = noninteractive
+ self.repo_pattern=repo_pattern
YumBase.__init__(self)
self.mute_stdout()
#self.conf.cache = os.geteuid() != 0
# setting-up repos one-by-one, so we can skip the broken ones...
# this helps when users are using 3rd party repos like rpmfusion
# in rawhide it results in: Can't find valid base url...
rid = self.repos.enableRepo(r.id)
self.repos.doSetup(thisrepo=str(r.id))
Loading...