Bug 2254717 - Review Request: python-expandvars - Expand system variables Unix style
Summary: Review Request: python-expandvars - Expand system variables Unix style
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Maxwell G
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard: Trivial
Depends On:
Blocks: 2254700
TreeView+ depends on / blocked
 
Reported: 2023-12-15 12:55 UTC by Ben Beasley
Modified: 2023-12-27 03:17 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-12-18 15:57:09 UTC
Type: ---
Embargoed:
maxwell: fedora-review+


Attachments (Terms of Use)

Description Ben Beasley 2023-12-15 12:55:41 UTC
Spec URL: https://music.fedorapeople.org/python-expandvars.spec
SRPM URL: https://music.fedorapeople.org/python-expandvars-0.12.0-1.fc39.src.rpm
Description:

This module is inspired by GNU bash’s variable expansion features. It can be
used as an alternative to Python’s os.path.expandvars function.

A good use case is reading config files with the flexibility of reading values
from environment variables using advanced features like returning a default
value if some variable is not defined.

Fedora Account System Username: music

This is a trivial pure-Python package required for the current version of python-frozenlist, which in turn is a dependency for python-aiohttp and python-aiosignal.

I plan to add the new “-l” option for %pyproject_save_files to assert the presence of a properly-packaged license file once pyproject-rpm-macros 1.11.0 reaches stable in all current releases (https://bodhi.fedoraproject.org/updates/?packages=pyproject-rpm-macros). For now, I have manually confirmed it with “rpm -qL -p /python3-expandvars-0.12.0-1.fc40.noarch.rpm” (expected output: /usr/lib/python3.12/site-packages/expandvars-0.12.0.dist-info/licenses/LICENSE).

Koji scratch builds:
F40: https://koji.fedoraproject.org/koji/taskinfo?taskID=110378314
F39: https://koji.fedoraproject.org/koji/taskinfo?taskID=110378315
F38: https://koji.fedoraproject.org/koji/taskinfo?taskID=110378316

Comment 1 Fedora Review Service 2023-12-15 13:01:09 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6760098
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2254717-python-expandvars/fedora-rawhide-x86_64/06760098-python-expandvars/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 Maxwell G 2023-12-18 04:58:31 UTC
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated

- [x] The License tag reflects the package contents and uses the correct identifiers.
- [x] The license text is included and marked with %license.
- [x] The package builds successfully in mock.
- [x] The package is installable (checked by fedora-review).
- [x] There are no relevant rpmlint errors.
- [x] The package runs tests in %check.
- [x] The latest version is packaged or packaging an earlier version is justified.
- [x] The packager considers avoiding confusing `%foo_name` macros. (Not a blocker)
- [x] Libraries: The package name has a `python3-` prefix and uses the canonical project name
- [x] The pyproject macros are used.
- [x] There are no bundled libraries.
- [x] The package complies with the Python and general Packaging Guidelines.

Package approved! On import, don't forget to add the package to release-monitoring.org. You can give @python-packagers-sig `commit` if you wish`

Thanks!

Comment 3 Ben Beasley 2023-12-18 15:39:01 UTC
Thank you for the review!

I’ve waded into simple Rust reviews lately, so I’ll try to take a look at bug 2254855 when I have some time.

Comment 4 Fedora Admin user for bugzilla script actions 2023-12-18 15:40:01 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/python-expandvars

Comment 5 Ben Beasley 2023-12-18 15:40:36 UTC
Added to release-monitoring.org: https://release-monitoring.org/project/95228/

Comment 6 Ben Beasley 2023-12-18 15:50:19 UTC
Added python-sig with commit privileges and configured Koschei (https://koschei.fedoraproject.org/package/python-expandvars?collection=f40).

Comment 7 Fedora Update System 2023-12-18 15:52:14 UTC
FEDORA-2023-d7fbfb0d0d has been submitted as an update to Fedora 40. https://bodhi.fedoraproject.org/updates/FEDORA-2023-d7fbfb0d0d

Comment 8 Fedora Update System 2023-12-18 15:57:09 UTC
FEDORA-2023-d7fbfb0d0d has been pushed to the Fedora 40 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 9 Fedora Update System 2023-12-18 16:56:46 UTC
FEDORA-EPEL-2023-c4d698b7e9 has been submitted as an update to Fedora EPEL 9. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2023-c4d698b7e9

Comment 10 Fedora Update System 2023-12-18 18:58:56 UTC
FEDORA-2023-8d65575379 has been submitted as an update to Fedora 39. https://bodhi.fedoraproject.org/updates/FEDORA-2023-8d65575379

Comment 11 Fedora Update System 2023-12-18 19:08:51 UTC
FEDORA-2023-5a8d22d786 has been submitted as an update to Fedora 38. https://bodhi.fedoraproject.org/updates/FEDORA-2023-5a8d22d786

Comment 12 Fedora Update System 2023-12-19 01:42:38 UTC
FEDORA-2023-5a8d22d786 has been pushed to the Fedora 38 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2023-5a8d22d786 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-5a8d22d786

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

Comment 13 Fedora Update System 2023-12-19 01:52:12 UTC
FEDORA-EPEL-2023-c4d698b7e9 has been pushed to the Fedora EPEL 9 testing repository.

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

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

Comment 14 Fedora Update System 2023-12-19 01:54:17 UTC
FEDORA-2023-8d65575379 has been pushed to the Fedora 39 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2023-8d65575379 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-8d65575379

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

Comment 15 Fedora Update System 2023-12-27 00:39:20 UTC
FEDORA-EPEL-2023-c4d698b7e9 has been pushed to the Fedora EPEL 9 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 16 Fedora Update System 2023-12-27 01:29:39 UTC
FEDORA-2023-8d65575379 has been pushed to the Fedora 39 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 17 Fedora Update System 2023-12-27 03:17:33 UTC
FEDORA-2023-5a8d22d786 has been pushed to the Fedora 38 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.