Bug 1966393 - Review Request: rust-stderrlog - Logger that aims to provide a simple case of env_logger
Summary: Review Request: rust-stderrlog - Logger that aims to provide a simple case of...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Robert-André Mauchin 🐧
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1966394 1966395
Blocks: 1966385
TreeView+ depends on / blocked
 
Reported: 2021-06-01 04:05 UTC by Rémi Lauzier
Modified: 2021-06-16 20:48 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2021-06-16 20:48:22 UTC
Type: ---
Embargoed:
eclipseo: fedora-review+


Attachments (Terms of Use)

Description Rémi Lauzier 2021-06-01 04:05:45 UTC
Spec URL: https://download.copr.fedorainfracloud.org/results/remilauzier/rust-oxipng/fedora-rawhide-x86_64/02220386-rust-stderrlog/rust-stderrlog.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/remilauzier/rust-oxipng/fedora-rawhide-x86_64/02220386-rust-stderrlog/rust-stderrlog-0.5.1-1.fc35.src.rpm
Description: Logger that aims to provide a simple case of env_logger that just logs to stderr based on verbosity.
Fedora Account System Username: remilauzier

Comment 1 Robert-André Mauchin 🐧 2021-06-05 18:25:47 UTC
 - You need to relax the docopt and thread_local deps (done with rust2rpm --stdout -p stderrlog):

# stderrlog-fix-metadata.diff
--- stderrlog-0.5.1/Cargo.toml  2021-01-10T00:15:24+00:00
+++ stderrlog-0.5.1/Cargo.toml  2021-06-05T18:19:08.223731+00:00
@@ -37,12 +37,12 @@
 version = "~1.1"
 
 [dependencies.thread_local]
-version = "~1.0"
+version = "1.0"
 [dev-dependencies.clap]
 version = "~2.33.0"
 
 [dev-dependencies.docopt]
-version = "~1.0"
+version = "1.0"
 
 [dev-dependencies.libc]
 version = "0.2.18"


Send a patch upstream to fix this and include the PR as a comment:

# Upstream license specification: MIT/Apache-2.0
License:        MIT or ASL 2.0
URL:            https://crates.io/crates/stderrlog
Source:         %{crates_source}
# Initial patched metadata
# - Relax thread_local https://link.to.pr
# - Relax docopt
Patch0:         stderrlog-fix-metadata.diff

 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Comment 3 Robert-André Mauchin 🐧 2021-06-06 14:07:16 UTC
Package approved.

Comment 4 Gwyn Ciesla 2021-06-07 13:38:56 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-stderrlog

Comment 5 Fedora Update System 2021-06-07 20:32:34 UTC
FEDORA-2021-3078e07b79 has been submitted as an update to Fedora 34. https://bodhi.fedoraproject.org/updates/FEDORA-2021-3078e07b79

Comment 6 Fedora Update System 2021-06-09 01:17:03 UTC
FEDORA-2021-3078e07b79 has been pushed to the Fedora 34 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2021-3078e07b79 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-3078e07b79

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

Comment 7 Fedora Update System 2021-06-16 20:48:22 UTC
FEDORA-2021-3078e07b79 has been pushed to the Fedora 34 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.