Bug 2074406 - Review Request: golang-github-a8m-envsubst - Environment variables substitution for Go
Summary: Review Request: golang-github-a8m-envsubst - Environment variables substituti...
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:
Depends On:
Blocks: 2074467
TreeView+ depends on / blocked
 
Reported: 2022-04-12 07:58 UTC by Julien Rische
Modified: 2022-07-17 00:57 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-07-05 14:32:22 UTC
Type: ---
Embargoed:
maxwell: fedora-review+


Attachments (Terms of Use)

Comment 2 Maxwell G 2022-06-25 03:15:46 UTC
I am not going to go through the whole fedora-review template, as this package uses go2rpm. For standard packages (or if you're a newer packager), you should go through the whole template.

- [x] The specfile is sane.
- [x] License is correct
- [x] Builds successfully in mock
- [!] Package is installable
See below
- [x] All rpmlint errors are trivial
- [x] %check section passes
- [x] The latest version is packaged
- [x] `%goipath` is set correctly
- [!] Binaries don't conflict with binaries already in the distribution
/usr/bin/envsubst conflicts with gettext. I would recommend renaming the binary to goenvsubst or something else so that this doesn't cause conflicts. At least on my system, it is uninstallable, because gettext is required by grub and multiple other packages. Here is a patch[1] to fix the issue.
- [!] The package complies with the Packaging Guidelines.
- [!] Package does not use globs in `%files` for shared directories

[1]: https://paste.sr.ht/~gotmax23/7e204a3cf40d507a68d7cbc4fc0877a798979a47

Comment 4 Maxwell G 2022-07-01 18:58:33 UTC
Thanks, Julien. I have approved your package and sponsored you into the packager group! Look out for an email from me about how to import your package into Fedora, along with some other pertinent information.

Comment 5 Kevin Fenzi 2022-07-04 17:11:49 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-a8m-envsubst

Comment 6 Fedora Update System 2022-07-04 17:30:45 UTC
FEDORA-2022-f8f69a2e43 has been submitted as an update to Fedora 37. https://bodhi.fedoraproject.org/updates/FEDORA-2022-f8f69a2e43

Comment 7 Fedora Update System 2022-07-04 22:18:33 UTC
FEDORA-2022-8104e059ec has been submitted as an update to Fedora 36. https://bodhi.fedoraproject.org/updates/FEDORA-2022-8104e059ec

Comment 8 Fedora Update System 2022-07-04 22:18:34 UTC
FEDORA-2022-53fdc3a7d3 has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2022-53fdc3a7d3

Comment 9 Fedora Update System 2022-07-05 14:32:22 UTC
FEDORA-2022-f8f69a2e43 has been pushed to the Fedora 37 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 10 Fedora Update System 2022-07-06 01:24:17 UTC
FEDORA-2022-53fdc3a7d3 has been pushed to the Fedora 35 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-53fdc3a7d3 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-53fdc3a7d3

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

Comment 11 Fedora Update System 2022-07-06 02:15:45 UTC
FEDORA-2022-8104e059ec has been pushed to the Fedora 36 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-8104e059ec \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-8104e059ec

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

Comment 12 Fedora Update System 2022-07-11 02:41:08 UTC
FEDORA-2022-3969b64d4b has been pushed to the Fedora 35 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-3969b64d4b`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-3969b64d4b

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

Comment 13 Fedora Update System 2022-07-14 01:45:15 UTC
FEDORA-2022-8104e059ec has been pushed to the Fedora 36 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 14 Fedora Update System 2022-07-17 00:57:30 UTC
FEDORA-2022-3969b64d4b 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.