Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1155878 - "dnf history list all" has different output then "dnf history list"
Summary: "dnf history list all" has different output then "dnf history list"
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: dnf
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Packaging Maintenance Team
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-10-23 04:58 UTC by Vít Ondruch
Modified: 2014-10-23 07:58 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-10-23 07:46:28 UTC
Type: Bug
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1155918 0 unspecified CLOSED let "history list" show all records 2022-05-16 11:32:56 UTC

Internal Links: 1155918

Description Vít Ondruch 2014-10-23 04:58:51 UTC 
Description of problem:
"dnf history list" shows "Command line" column while "dnf history list all" displays useless "Login user". Please always show the "Command line" column.

Version-Release number of selected component (if applicable):
$ rpm -q dnf
dnf-0.6.2-1.fc22.noarch


How reproducible:
Always

Steps to Reproduce:
1. dnf history list
2. dnf history list all
3.

Actual results:
# sudo dnf history list all
ID     | Login user               | Date a | Action | Altere
-------------------------------------------------------------------------------


Expected results:
# sudo dnf history list all
ID     | Command line             | Date a | Action | Altere


Additional info:
It seems that the columns might be configurable somehow, but I can't find anything about it in man pages :/
Comment 1 Radek Holy 2014-10-23 07:21:11 UTC 
Given that you already looked at man pages you must know that DNF does not support a command called "history list all".
Comment 2 Vít Ondruch 2014-10-23 07:33:47 UTC 
Excuse me? So what does the "dnf history list all" does then? What does the "all" do? Trying some other "random" parameters, DNF fails, so it definitely understands to parameter "all" and it definitely gives me different results (although trying now on different machine with less entries in history, I can't see difference).
Comment 3 Radek Holy 2014-10-23 07:46:28 UTC 
BTW, code ```import dnf; dnf.Base().install('package', reponame='fedora')``` also does not produce a ```TypeError``` although the "reponame" parameter is not in the documentation.

It's simple. If it's not in the documentation, it isn't supported. The thing that DNF *accepts* an undocumented argument does not mean that it *understands* it. If it's not in the documentation, then there is no definition of what given command does. Then you cannot expect any behavior and you cannot say that the behavior is buggy (= differs from the expectation).

Maybe you want to file a bug reporting that DNF does not throw an error in this case.
Comment 4 Radek Holy 2014-10-23 07:50:31 UTC 
Or maybe you want to file an RFE asking for the support of this command although I don't understand why would be need two commands doing the same thing.
Comment 5 Vít Ondruch 2014-10-23 07:58:12 UTC 
Here it goes: bug #1155918

Thanks.
Note You need to log in before you can comment on or make changes to this bug.