Bug 1810210 - fedora-review: clean at start of install test passes when it should not
Summary: fedora-review: clean at start of install test passes when it should not
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: fedora-review
Version: 33
Hardware: All
OS: Linux
unspecified
unspecified
Target Milestone: ---
Assignee: Stanislav Ochotnicky
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-03-04 17:45 UTC by Jerry James
Modified: 2020-11-20 01:29 UTC (History)
5 users (show)

Fixed In Version: fedora-review-0.7.6-1.fc33 fedora-review-0.7.6-1.fc32 fedora-review-0.7.6-1.fc31
Clone Of:
Environment:
Last Closed: 2020-11-13 03:47:09 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Jerry James 2020-03-04 17:45:14 UTC
Description of problem:
I am reviewing bug 1804529.  The spec file contains this:

%install
rm -rf $RPM_BUILD_ROOT
%make_install

but fedora-review produced this:

[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
     beginning of %install.

Version-Release number of selected component (if applicable):
fedora-review-0.7.5-1.fc31.noarch

How reproducible:
Always

Steps to Reproduce:
1. Create a spec file with an %install section as shown above
2. Run fedora-review on it

Actual results:
The test passes.

Expected results:
The test should fail.

Additional info:

Comment 1 Jerry James 2020-03-04 17:50:47 UTC
In /usr/share/fedora-review/plugins/generic.py, line 230:

        regex = r"rm\s+\-[rf][rf]\s+(@buildroot@|$RPM_BUILD_ROOT)[^/]"

the $ should be escaped; i.e.,

        regex = r"rm\s+\-[rf][rf]\s+(@buildroot@|\$RPM_BUILD_ROOT)[^/]"

Comment 2 Ben Cotton 2020-11-03 16:25:49 UTC
This message is a reminder that Fedora 31 is nearing its end of life.
Fedora will stop maintaining and issuing updates for Fedora 31 on 2020-11-24.
It is Fedora's policy to close all bug reports from releases that are no longer
maintained. At that time this bug will be closed as EOL if it remains open with a
Fedora 'version' of '31'.

Package Maintainer: If you wish for this bug to remain open because you
plan to fix it in a currently maintained version, simply change the 'version' 
to a later Fedora version.

Thank you for reporting this issue and we are sorry that we were not 
able to fix it before Fedora 31 is end of life. If you would still like 
to see this bug fixed and are able to reproduce it against a later version 
of Fedora, you are encouraged  change the 'version' to a later Fedora 
version prior this bug is closed as described in the policy above.

Although we aim to fix as many bugs as possible during every release's 
lifetime, sometimes those efforts are overtaken by events. Often a 
more recent Fedora release includes newer upstream software that fixes 
bugs or makes them obsolete.

Comment 3 Jerry James 2020-11-03 20:52:23 UTC
I submitted a pull request to make the change in comment 1, and it was merged, but there has been no new release of fedora-review since.  Please, can we have another release?  Several bugs have been fixed in master.  It would be good to get the fixes into the hands of package reviewers.

Comment 4 Fedora Update System 2020-11-11 02:45:58 UTC
FEDORA-2020-20c4ebf164 has been submitted as an update to Fedora 31. https://bodhi.fedoraproject.org/updates/FEDORA-2020-20c4ebf164

Comment 5 Fedora Update System 2020-11-11 02:46:06 UTC
FEDORA-2020-af607360e8 has been submitted as an update to Fedora 32. https://bodhi.fedoraproject.org/updates/FEDORA-2020-af607360e8

Comment 6 Fedora Update System 2020-11-11 02:46:13 UTC
FEDORA-2020-219835ab79 has been submitted as an update to Fedora 33. https://bodhi.fedoraproject.org/updates/FEDORA-2020-219835ab79

Comment 7 Fedora Update System 2020-11-12 04:07:59 UTC
FEDORA-2020-20c4ebf164 has been pushed to the Fedora 31 testing repository.
In short time you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2020-20c4ebf164`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-20c4ebf164

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 8 Fedora Update System 2020-11-12 04:24:33 UTC
FEDORA-2020-219835ab79 has been pushed to the Fedora 33 testing repository.
In short time you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2020-219835ab79`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-219835ab79

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 9 Fedora Update System 2020-11-12 04:42:25 UTC
FEDORA-2020-af607360e8 has been pushed to the Fedora 32 testing repository.
In short time you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2020-af607360e8`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-af607360e8

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 10 Fedora Update System 2020-11-13 03:47:09 UTC
FEDORA-2020-219835ab79 has been pushed to the Fedora 33 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 11 Fedora Update System 2020-11-20 01:26:11 UTC
FEDORA-2020-af607360e8 has been pushed to the Fedora 32 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 12 Fedora Update System 2020-11-20 01:29:11 UTC
FEDORA-2020-20c4ebf164 has been pushed to the Fedora 31 stable repository.
If problem still persists, please make note of it in this bug report.


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