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 1948133 - Review Request: rust-configparser - Simple configuration parsing utility with no dependencies
Summary: Review Request: rust-configparser - Simple configuration parsing utility with...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Robert-André Mauchin 🐧
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1948159
TreeView+ depends on / blocked
 
Reported: 2021-04-10 10:20 UTC by Fabio Valentini
Modified: 2021-04-13 13:47 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-04-13 13:47:34 UTC
Type: ---
Embargoed:
zebob.m: fedora-review+

Attachments (Terms of Use)

Description Fabio Valentini 2021-04-10 10:20:56 UTC 
Spec URL: https://decathorpe.fedorapeople.org/rust-configparser.spec
SRPM URL: https://decathorpe.fedorapeople.org/rust-configparser-2.0.0-1.fc33.src.rpm

Description:
Simple configuration parsing utility with no dependencies that allows you to
parse INI and ini-style syntax. You can use this to write Rust programs which
can be customized by end users easily.

Fedora Account System Username: decathorpe
Comment 1 Fabio Valentini 2021-04-10 10:21:00 UTC 
This package built on koji:  https://koji.fedoraproject.org/koji/taskinfo?taskID=65640783
Comment 2 Robert-André Mauchin 🐧 2021-04-11 20:31:35 UTC 
 - Bump to 2.0.1

 - Please remove the executable bits of:

rust-configparser-devel.noarch: W: spurious-executable-perm /usr/share/doc/rust-configparser-devel/CHANGELOG.md
rust-configparser-devel.noarch: W: spurious-executable-perm /usr/share/doc/rust-configparser-devel/README.md

in %prep and send a patch upstream.


 - License ok
 - Builds in mock
 - Conforms to Packaging Guidelines

Package approved. Please fix the aforementioned issue before import.
Comment 3 Fabio Valentini 2021-04-12 09:46:27 UTC 
I've updated to 2.0.1 and added "chmod -x" ahead of importing.

The issue with executable bits (and file mtimes of unix epoch 0 ???) has already been reported upstream:
https://github.com/QEDK/configparser-rs/issues/15

Looks like the crate author has weird problems with their machine and / or publishing workflow.

https://pagure.io/releng/fedora-scm-requests/issue/33467
https://pagure.io/releng/fedora-scm-requests/issue/33468
Comment 4 Gwyn Ciesla 2021-04-12 14:32:36 UTC 
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-configparser
Comment 5 Fedora Update System 2021-04-13 13:47:34 UTC 
FEDORA-2021-c1fecf829f has been pushed to the Fedora 35 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.