Bug 112794

Summary:

[RFE] [rpm-python] Return [] for header types that return lists

Product:

[Retired] Red Hat Raw Hide

Reporter:

Paul Nasrat <nobody+pnasrat>

Component:

rpm-python

Assignee:

Jeff Johnson <jbj>

Status:

CLOSED CURRENTRELEASE

QA Contact:

David Lawrence <dkl>

Severity:

medium

Docs Contact:

Priority:

medium

Version:

1.0

CC:

wtogami

Target Milestone:

---

Keywords:

FutureFeature

Target Release:

---

Hardware:

All

OS:

Linux

Whiteboard:

Fixed In Version:

4.3.0-7.1

Doc Type:

Enhancement

Doc Text:

Story Points:

---

Clone Of:

Environment:

Last Closed:

2004-01-26 10:55:25 UTC

Type:

---

Regression:

---

Mount Type:

---

Documentation:

---

CRM:

Verified Versions:

Category:

---

oVirt Team:

---

RHEL 7.3 requirements from Atomic Host:

Cloudforms Team:

---

Target Upstream Version:

Embargoed:

Attachments:

Description	Flags
test_obs.py	none

Description Paul Nasrat 2004-01-01 12:08:00 UTC

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4.1)
Gecko/20031114

Description of problem:
Second part of bug
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=110226#c3

Return empty list for header tags that can return lists.

Version-Release number of selected component (if applicable):
rpm-4.2.2-0.8

How reproducible:
Always

Steps to Reproduce:
See test attachment
    

Actual Results:  Returns None

Expected Results:  Returns []

Additional info:

Comment 1 Paul Nasrat 2004-01-01 12:10:06 UTC

Created attachment 96737 [details]
test_obs.py

Test case for bindings, I'm going to check in working patch against HEAD, but
would be grateful for sanity check.

Comment 2 Jeff Johnson 2004-01-26 10:02:47 UTC

Hmmm, this looks fishy:
    ts.setFlags(-1)
    vsflags = ts.setVSFlags(~(rpm._RPMVSF_NOSIGNATURES))
I don't think you should do either.

If you *must* disable signature checking, then
    ts.setVSFlags(-1)
is all that is needed.

ts.setFlags() should have sane defaults, and is really used
to control ts.run() operation.

NEEDINFO so's I don't have to stare at the bug.

Comment 3 Paul Nasrat 2004-01-26 10:55:25 UTC

I got rid of both those lines a while back I think I uploaded an older
version of the script.

Basically this is fixed using the forceArray and switch for ENVRAO in
HEAD and rpm-4.3-0.7.1 and above