Bug 2031239 - Review Request: golang-github-bep-gowebp - Go library for Webp images
Summary: Review Request: golang-github-bep-gowebp - Go library for Webp images
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Fabio Alessandro Locati
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1930952
TreeView+ depends on / blocked
 
Reported: 2021-12-10 20:35 UTC by W. Michael Petullo
Modified: 2022-05-07 04:26 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-04-18 17:58:36 UTC
Type: ---
Embargoed:
me: fedora-review+


Attachments (Terms of Use)

Description W. Michael Petullo 2021-12-10 20:35:33 UTC
Spec URL: https://www.flyn.org/SRPMS/golang-github-bep-gowebp.spec
SRPM URL: https://www.flyn.org/SRPMS/golang-github-bep-gowebp-0.1.0-1.fc35.src.rpm
Description: Provides C bindings and an API for encoding Webp images using Google's libwebp.
Fedora Account System Username: mikep

Comment 2 Mikel Olasagasti Uranga 2022-03-15 15:21:52 UTC
I'm not able to build this one against rawhide. A test fails because swig is not installed:

> github.com/bep/gowebp/libwebp_src/swig
> go build github.com/bep/gowebp/libwebp_src/swig: swig: exec: "swig": executable file not found in $PATH

After adding it it fails with missing runtime.h

> github.com/bep/gowebp/libwebp_src/swig
> # github.com/bep/gowebp/libwebp_src/swig
> libwebp_gc.c:14:10: fatal error: runtime.h: No such file or directory
>    14 | #include "runtime.h"
>       |          ^~~~~~~~~~~
> compilation terminated.

Comment 3 W. Michael Petullo 2022-03-18 03:25:29 UTC
I am trying to figure this out. The build on mock Rawhide deviates from the build on mock Fedora 35. Rawhide appears to run more tests and fails. Fedora 35 succeeds.



Rawhide:
      command: go test -buildmode pie -compiler gc -ldflags " -X github.com/bep/gowebp/version=0.1.0 -extldflags '-Wl,-z,relro -Wl,--as-needed  -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1  -Wl,--build-id=sha1 -Wl,-dT,/builddir/build/BUILD/gowebp-0.1.0/.package_note-golang-github-bep-gowebp-0.1.0-1.fc37.x86_64.ld '"
      testing: github.com/bep/gowebp
github.com/bep/gowebp/internal/libwebp
?   	github.com/bep/gowebp/internal/libwebp	[no test files]
github.com/bep/gowebp/libwebp
PASS
ok  	github.com/bep/gowebp/libwebp	0.491s
github.com/bep/gowebp/libwebp
PASS
ok  	github.com/bep/gowebp/libwebp	0.504s
github.com/bep/gowebp/libwebp/webpoptions
?   	github.com/bep/gowebp/libwebp/webpoptions	[no test files]
github.com/bep/gowebp/libwebp_src/swig
# github.com/bep/gowebp/libwebp_src/swig
libwebp_gc.c:14:10: fatal error: runtime.h: No such file or directory
   14 | #include "runtime.h"
      |          ^~~~~~~~~~~
compilation terminated.
error: Bad exit status from /var/tmp/rpm-tmp.ZNIJjQ (%check)
    Bad exit status from /var/tmp/rpm-tmp.ZNIJjQ (%check)




Fedora 35:
            command: go test -buildmode pie -compiler gc -ldflags " -X github.com/bep/gowebp/version=0.1.0 -extldflags '-Wl,-z,relro -Wl,--as-needed  -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1  '"
      testing: github.com/bep/gowebp
github.com/bep/gowebp/libwebp
PASS
ok  	github.com/bep/gowebp/libwebp	0.574s

Comment 4 Fabio Alessandro Locati 2022-04-10 16:26:20 UTC
I would like to share with you https://fale.fedorapeople.org/golang-github-bep-gowebp/golang-github-bep-gowebp.spec that is my take to this SPEC file. Maybe it can help with it.
As for that specific test, I've skipped it. My understanding is that it would be ok for the package even if the test fails

Comment 5 W. Michael Petullo 2022-04-12 20:16:02 UTC
Spec URL: https://www.flyn.org/SRPMS/golang-github-bep-gowebp.spec
SRPM URL: https://www.flyn.org/SRPMS/golang-github-bep-gowebp-0.1.0-1.fc35.src.rpm

Apply suggestions from Fabio.

Comment 6 Fabio Alessandro Locati 2022-04-18 08:20:46 UTC
go2rpm package, fedora-review is correct:

- The specfile is sane.
- License is correct
- Builds successfully in mock
- No rpmlint errors
- %check section passes
- The latest version is packaged
- The package complies with the Packaging Guidelines.

Package approved! On import, don't forget to do the following:

- Add package to release-monitoring.org
- Add package to Koschei.
- Give go-sig privileges on package
- Close the review bug by referencing it in the rpm changelog and/or the Bodhi ticket. (rhbz#BUG_ID)

Thanks!

PS: Due to the fact that rpmbuild does not care abou folders in the documentation, the last file (libwebp_src/swig/README) will replace libwebp_src/README. I would suggest renaming or dropping libwebp_src/swig/README.

Comment 7 Gwyn Ciesla 2022-04-18 15:38:05 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-bep-gowebp

Comment 8 Fedora Update System 2022-04-18 17:58:11 UTC
FEDORA-2022-34cc54bd63 has been submitted as an update to Fedora 37. https://bodhi.fedoraproject.org/updates/FEDORA-2022-34cc54bd63

Comment 9 Fedora Update System 2022-04-18 17:58:36 UTC
FEDORA-2022-34cc54bd63 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-04-18 18:05:35 UTC
FEDORA-2022-8700bb8fb1 has been submitted as an update to Fedora 36. https://bodhi.fedoraproject.org/updates/FEDORA-2022-8700bb8fb1

Comment 11 Fedora Update System 2022-04-18 18:16:16 UTC
FEDORA-2022-90631629ed has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2022-90631629ed

Comment 12 Fedora Update System 2022-04-19 17:27:12 UTC
FEDORA-2022-8700bb8fb1 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 --advisory=FEDORA-2022-8700bb8fb1 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-8700bb8fb1

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

Comment 13 Fedora Update System 2022-04-20 20:25:48 UTC
FEDORA-2022-90631629ed 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 --advisory=FEDORA-2022-90631629ed \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-90631629ed

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

Comment 14 Fedora Update System 2022-04-28 05:50:23 UTC
FEDORA-2022-90631629ed has been pushed to the Fedora 35 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 15 Fedora Update System 2022-05-07 04:26:45 UTC
FEDORA-2022-8700bb8fb1 has been pushed to the Fedora 36 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.