poldek fails to see packages from all repos if one repository is broken

Bug #813213 reported by Elan Ruusamäe
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
poldek
Fix Released
Undecided
Cactus
PLD Linux
Fix Released
Undecided
Cactus

Bug Description

say, if i have one repo broken:

# poldek -u grub2
Loading [pndir]th...
Loading [pndir]th...
Loading [dir]local-pld...
error: opendir /home/glen/rpm/pld/RPMS: No such file or directory
error: /home/glen/rpm/pld/RPMS: load failed
Loading [pndir]dsl-delfi...
Loading [pndir]dsl-delfi...
warn: no packages loaded
error: grub2: no such package

workarounded can be to specify repos manually:

# poldek -u grub2 --sn th
Loading [pndir]th...
Loading [pndir]th...
16749 packages read
grub2-1.97.2-2.x86_64: equal version installed, skipped
Nothing to do

# cat /etc/poldek/source.conf
[source]
name = local-pld
type = dir
path = /home/glen/rpm/pld/RPMS
auto = yes
autoup = no

Revision history for this message
Cactus (uzi18) wrote :

Elan check this patch: http://pastebin.com/c90E5pth
is it ok for You?

Revision history for this message
Cactus (uzi18) wrote :

diff --git a/pkgset-load.c b/pkgset-load.c
index 470dae5..000e4df 100644
--- a/pkgset-load.c
+++ b/pkgset-load.c
@@ -98,7 +98,7 @@ int pkgset_load(struct pkgset *ps, int ldflags, tn_array *sources)
         if ((pkgdir->flags & PKGDIR_LOADED) == 0) {
             if (!pkgdir_load(pkgdir, ps->depdirs, ldflags)) {
                 logn(LOGERR, _("%s: load failed"), pkgdir->idxpath);
- iserr = 1;
+ //iserr = 1;
             }
         }
         MEMINF("after load %s", pkgdir_idstr(pkgdir));

Cactus (uzi18)
Changed in poldek:
assignee: nobody → Cactus (uzi18)
status: New → Confirmed
status: Confirmed → In Progress
Cactus (uzi18)
Changed in pld-linux:
assignee: nobody → Cactus (uzi18)
status: New → In Progress
Cactus (uzi18)
Changed in poldek:
status: In Progress → Fix Committed
Revision history for this message
Elan Ruusamäe (glen666) wrote :

i guess it is ok. but i'd be happy to test with new poldek than guess is it ok or not :)

Revision history for this message
Cactus (uzi18) wrote :
Revision history for this message
Cactus (uzi18) wrote :

Elan please set "Heat" for the rest poldek bugs to know what to fix first ;)

Revision history for this message
Jan Rękorajski (baggins-pld-linux) wrote :

Fixed in poldek-0.30-1.rc3 (commit f8e81a61c6ba3c5977e0f6bba1cd065ee919def1)

Changed in pld-linux:
status: In Progress → Fix Released
Changed in poldek:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.