Bug 2224163 - Review Request: blackbox-terminal - Elegant and customizable terminal for GNOME
Summary: Review Request: blackbox-terminal - Elegant and customizable terminal for GNOME
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Tom "spot" Callaway
QA Contact: Fedora Extras Quality Assurance
URL: https://gitlab.gnome.org/raggesilver/...
Whiteboard:
Depends On: 2224161
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-07-20 03:49 UTC by Carl George 🤠
Modified: 2023-08-17 01:16 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-08-08 21:34:34 UTC
Type: ---
Embargoed:
spotrh: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 6188848 to 6188993 (868 bytes, patch)
2023-07-20 05:24 UTC, Fedora Review Service
no flags Details | Diff

Comment 1 Fedora Review Service 2023-07-20 03:53:50 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6188848
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2224163-blackbox-terminal/fedora-rawhide-x86_64/06188848-blackbox-terminal/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
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 3 Fedora Review Service 2023-07-20 05:24:05 UTC
Created attachment 1976649 [details]
The .spec file difference from Copr build 6188848 to 6188993

Comment 4 Fedora Review Service 2023-07-20 05:24:07 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6188993
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2224163-blackbox-terminal/fedora-rawhide-x86_64/06188993-blackbox-terminal/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
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 5 Carl George 🤠 2023-07-27 02:48:11 UTC
pqmarble (a build requirement for this package) has been reviewed [0] and built in rawhide [1].  It's available now for mock builds if you enable the "local" repo in the rawhide mock template.  That's enough to satisfy a local run of fedora-review, but I expect the review service build won't work for a few more days.

[0] bug 2224161
[1] https://bodhi.fedoraproject.org/updates/FEDORA-2023-dccb399c14

Comment 6 Tom "spot" Callaway 2023-08-04 14:59:56 UTC
Nothing blocking. Approved.

= Review =
- rpmlint checks return:
blackbox-terminal.x86_64: W: no-manual-page-for-binary blackbox-terminal
blackbox-terminal.x86_64: W: no-documentation
blackbox-terminal.x86_64: W: files-duplicate /usr/share/icons/hicolor/scalable/actions/com.raggesilver.BlackBox-fullscreen-symbolic.svg /usr/share/blackbox/icons/hicolor/scalable/actions/com.raggesilver.BlackBox-fullscreen-symbolic.svg
blackbox-terminal.x86_64: W: files-duplicate /usr/share/icons/hicolor/scalable/actions/com.raggesilver.BlackBox-show-headerbar-symbolic.svg /usr/share/blackbox/icons/hicolor/scalable/actions/com.raggesilver.BlackBox-show-headerbar-symbolic.svg
blackbox-terminal.x86_64: W: files-duplicate /usr/share/icons/hicolor/scalable/actions/external-link-symbolic.svg /usr/share/blackbox/icons/hicolor/scalable/actions/external-link-symbolic.svg
blackbox-terminal.x86_64: W: files-duplicate /usr/share/icons/hicolor/scalable/actions/settings-symbolic.svg /usr/share/blackbox/icons/hicolor/scalable/actions/settings-symbolic.svg
blackbox-terminal.x86_64: W: files-duplicate /usr/share/icons/hicolor/scalable/apps/com.raggesilver.BlackBox.svg /usr/share/blackbox/icons/hicolor/scalable/apps/com.raggesilver.BlackBox.svg

Maybe consider symlinking those svg files to cut down on install footprint a bit. Not a MUST.

- package meets naming guidelines
- package meets packaging guidelines
- license (GPL-3.0-or-later AND (MIT OR Apache-2.0) AND MPL-2.0) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream (0a8b76666ac71a6dc9e05b8084e1c6c53cda880ad1f8f778d9d5de9a6644ad0d)
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc <- Should README.md be in there?
- Desktop file validated

Comment 7 Fedora Admin user for bugzilla script actions 2023-08-08 16:37:34 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/blackbox-terminal

Comment 8 Carl George 🤠 2023-08-08 19:46:47 UTC
> blackbox-terminal.x86_64: W: files-duplicate ...

It looks like cause of this is the upstream meson config installs the icons twice into separate directories.  I've opened a merge request upstream to fix it, and applied it as a patch in the package.

https://gitlab.gnome.org/raggesilver/blackbox/-/merge_requests/126

> - nothing in %doc <- Should README.md be in there?

I added README.md and CHANGELOG.md.

Comment 9 Fedora Update System 2023-08-08 21:34:22 UTC
FEDORA-2023-828d2a8c2d has been submitted as an update to Fedora 40. https://bodhi.fedoraproject.org/updates/FEDORA-2023-828d2a8c2d

Comment 10 Fedora Update System 2023-08-08 21:34:34 UTC
FEDORA-2023-828d2a8c2d has been pushed to the Fedora 40 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 11 Fedora Update System 2023-08-08 22:41:59 UTC
FEDORA-2023-e785787322 has been submitted as an update to Fedora 39. https://bodhi.fedoraproject.org/updates/FEDORA-2023-e785787322

Comment 12 Fedora Update System 2023-08-08 22:43:35 UTC
FEDORA-2023-e785787322 has been pushed to the Fedora 39 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 13 Fedora Update System 2023-08-09 02:01:03 UTC
FEDORA-2023-e9e4c3992c 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-e9e4c3992c \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-e9e4c3992c

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

Comment 14 Fedora Update System 2023-08-17 01:16:59 UTC
FEDORA-2023-e9e4c3992c 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.