getPathList doesn't work with newer rpm

Bug #263762 reported by Anders F Björklund
4
Affects Status Importance Assigned to Milestone
Smart Package Manager
Fix Released
Undecided
Gustavo Niemeyer
smart (Fedora)
Fix Released
Medium

Bug Description

RPMTAG_OLDFILENAMES is deprecated, having been replaced with RPMTAG_DIRNAMES+RPMTAG_BASENAMES

Related branches

description: updated
Changed in smart:
milestone: none → 1.2
Revision history for this message
In , John (john-redhat-bugs) wrote :

Description of problem:The 'Content' pane normally shows the result of 'rpm -ql' for the highlighted package. In fc10 it is empty.

Version-Release number of selected component (if applicable):1.1-56.fc10@x86_64

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual results:

Expected results:

Additional info: This is the reply to my posting about this on the smart list

From: =?ISO-8859-1?Q?Anders_F_Bj=F6rklund?= <email address hidden>
Subject: Re: smart-gui 'Content' tab empty in Fedora 10
Date: Mon, 15 Dec 2008 17:33:37 +0100

> Hi: I've just installed Fedora 10 and have the x86_64 release version of smart and smart-gui in the KDE desktop. I haven't found a config option to show the useful 'rpm -ql' info that I see in the Centos 5.2 version: the panel is always empty. The other tabs work as usual. Have I missed something?

It's the newer RPM version, an old deprecated tag is now gone.

See https://bugs.launchpad.net/smart/+bug/263762 for patch...

--anders

Revision history for this message
In , John (john-redhat-bugs) wrote :

It didn't at first occur to me that I could simply edit in the 6-line patch to the .py file and expect it to work: but it works-for-me :-)

File is /usr/lib64/python2.5/site-packages/smart/backends/rpm/header.py

Revision history for this message
In , Fedora (fedora-redhat-bugs) wrote :

smart-1.1-58.fc8 has been pushed to the Fedora 8 stable repository. If problems still persist, please make note of it in this bug report.

Revision history for this message
In , Fedora (fedora-redhat-bugs) wrote :

smart-1.1-58.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report.

Revision history for this message
In , Fedora (fedora-redhat-bugs) wrote :

smart-1.1-58.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.

Revision history for this message
In , John (john-redhat-bugs) wrote :

I just installed 1.1-58.fc10@x86-64. The 'Content' tab is still empty.

After making Comment #1 above I found that although the pane was no longer empty after editing in the suggested patch to 1.1-56, its content was _not_ always identical with that given by 'rpm -ql'. I reported this to Anders, but not elsewhere.

Revision history for this message
In , Fedora (fedora-redhat-bugs) wrote :

smart-1.1-58.0.1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/smart-1.1-58.0.1.fc10

Revision history for this message
In , Axel (axel-redhat-bugs) wrote :

(In reply to comment #6)
> I just installed 1.1-58.fc10@x86-64. The 'Content' tab is still empty.

This was a bad build, please try 1.1-58.0.1.fc10. Thanks!

Revision history for this message
In , John (john-redhat-bugs) wrote :

Downloaded and installed smart, smart-update, smart-gui 58.0.1 from the koji url.

The patch (if it's this one)

http://bazaar.launchpad.net/~afb/smart/sandbox/revision/936

still isn't here

/usr/lib64/python2.5/site-packages/smart/backends/rpm/header.py

and the pane is still empty :-(

Happy New Year!

Revision history for this message
In , Axel (axel-redhat-bugs) wrote :

OK, I thought this patch had made it into

https://code.launchpad.net/~smartpm/smart/bugfix

but I was wrong. So 1.1-58 will not fix it. I'll make another build, but I do want to get 1.1-58 out the door asap to fix all other issues.

Revision history for this message
Axel Thimm (axel-thimm) wrote :

Could this patch be merged into https://code.launchpad.net/~smartpm/smart/bugfix

Thanks!

Revision history for this message
Anders F Björklund (afb) wrote :

I wanted to refactor both this and #244771 into proper helper functions,
instead of duplicating the same code whenever it's needed. But sure.

Revision history for this message
In , John (john-redhat-bugs) wrote :

OK, it's not a major issue, although it would be nice to have it back - and perhaps the 'Changelog' tab that Anders mentioned, too. I didn't try that.

FYI, when I inserted the 'revision/936' patch as before, smart produced an error message and exited.

Revision history for this message
Anders F Björklund (afb) wrote :

Now merged from sandbox.

Changed in smart:
milestone: 1.2 → 1.1.2
Revision history for this message
Anders F Björklund (afb) wrote :
Changed in smart:
status: Unknown → In Progress
Revision history for this message
Anders F Björklund (afb) wrote :

Stupid thinko when I retrieved the lost patch made it not use RPMTAG_DIRINDEXES like it should. Fixed now, thanks to niemeyer for noticing.

Revision history for this message
In , Fedora (fedora-redhat-bugs) wrote :

smart-1.1-59.0.1.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report.

Revision history for this message
In , Fedora (fedora-redhat-bugs) wrote :

smart-1.1-59.0.1.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.

Changed in smart:
status: In Progress → Fix Released
Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

This was fixed in trunk with a similar implementation and full tests.

Changed in smart:
assignee: nobody → niemeyer
status: New → Fix Committed
Revision history for this message
Anders F Björklund (afb) wrote :

Released, 2009-03-04

Changed in smart:
status: Fix Committed → Fix Released
Changed in smart (Fedora):
importance: Unknown → Medium
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.