Bug 2184737 - Review Request: R-discretization - Data Preprocessing, Discretization for Classification
Summary: Review Request: R-discretization - Data Preprocessing, Discretization for Cla...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Iñaki Ucar
QA Contact: Fedora Extras Quality Assurance
URL: https://CRAN.R-project.org/package=%{...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-04-05 14:40 UTC by Iztok Fister Jr.
Modified: 2023-04-15 03:18 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2023-04-15 02:07:30 UTC
Type: ---
Embargoed:
i.ucar86: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 5746002 to 5746716 (993 bytes, patch)
2023-04-05 18:12 UTC, Jakub Kadlčík
no flags Details | Diff

Description Iztok Fister Jr. 2023-04-05 14:40:48 UTC
Spec URL: https://raw.githubusercontent.com/firefly-cpp/rpm-reviews/main/R-discretization.spec
SRPM URL: https://github.com/firefly-cpp/rpm-reviews/raw/main/R-discretization-1.0.1.1-1.fc37.src.rpm
Description: A collection of supervised discretization algorithms. It can also
be grouped in terms of top-down or bottom-up, implementing the discretization algorithms.
Fedora Account System Username: iztokf

Comment 1 Jakub Kadlčík 2023-04-05 14:49:48 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/5746002
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2184737-R-discretization/fedora-rawhide-x86_64/05746002-R-discretization/fedora-review/review.txt

Please take a look if any issues were found.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 2 Iñaki Ucar 2023-04-05 15:03:30 UTC
The package does not require testthat for checks. Please remove those bits. Relevant issues:

> [!]: Sources can be downloaded from URI in Source: tag
>      Note: Could not download Source0:
>      https://CRAN.R-project.org/package=discretization&version=1.0.1.1#/discretization_1.0-1.1.tar.gz

Note that you should use %{ver} instead of %{version} in the URL.

> R-discretization.noarch: W: invalid-license GPL

See https://docs.fedoraproject.org/en-US/packaging-guidelines/LicensingGuidelines/#_valid_license_short_names

> R-discretization.noarch: E: description-line-too-long be grouped in terms of top-down or bottom-up, implementing the discretization algorithms.

See https://fedoraproject.org/wiki/Common_Rpmlint_issues#description-line-too-long

Comment 3 Iztok Fister Jr. 2023-04-05 15:14:59 UTC
Thanks.

Fixed now. Changes: https://github.com/firefly-cpp/rpm-reviews/commit/79a135ac347ea828149ffe809e4863571038dd5f

Comment 4 Iñaki Ucar 2023-04-05 15:25:49 UTC
Note that:

- GPL v2 is also allowed
- Identifiers from the third column are legacy ones. SPDX identifiers are in the first column of the list.

So the license tag should be: GPL-2.0-or-later

Also, please always provide new Spec URL and SRPM URL for any change, so that a new check is automatically triggered.

Comment 6 Jakub Kadlčík 2023-04-05 18:12:58 UTC
Created attachment 1955948 [details]
The .spec file difference from Copr build 5746002 to 5746716

Comment 7 Jakub Kadlčík 2023-04-05 18:13:00 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/5746716
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2184737-R-discretization/fedora-rawhide-x86_64/05746716-R-discretization/fedora-review/review.txt

Please take a look if any issues were found.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 8 Iñaki Ucar 2023-04-05 21:15:37 UTC
Thanks, package approved.

Comment 9 Fedora Admin user for bugzilla script actions 2023-04-06 08:19:47 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/R-discretization

Comment 10 Fedora Update System 2023-04-06 10:23:09 UTC
FEDORA-2023-6508fabab6 has been submitted as an update to Fedora 37. https://bodhi.fedoraproject.org/updates/FEDORA-2023-6508fabab6

Comment 11 Fedora Update System 2023-04-06 10:23:10 UTC
FEDORA-2023-b8b5382510 has been submitted as an update to Fedora 38. https://bodhi.fedoraproject.org/updates/FEDORA-2023-b8b5382510

Comment 12 Fedora Update System 2023-04-07 01:52:47 UTC
FEDORA-2023-6508fabab6 has been pushed to the Fedora 37 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2023-6508fabab6`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-6508fabab6

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

Comment 13 Fedora Update System 2023-04-07 02:00:19 UTC
FEDORA-2023-b8b5382510 has been pushed to the Fedora 38 testing repository.

You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-b8b5382510

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

Comment 14 Fedora Update System 2023-04-15 02:07:30 UTC
FEDORA-2023-b8b5382510 has been pushed to the Fedora 38 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 15 Fedora Update System 2023-04-15 03:18:40 UTC
FEDORA-2023-6508fabab6 has been pushed to the Fedora 37 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.