Bug 1813737 - Review Request: azote - Wallpaper and colour manager for Sway, i3 and some other WMs
Summary: Review Request: azote - Wallpaper and colour manager for Sway, i3 and some ot...
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: FE-NEEDSPONSOR
TreeView+ depends on / blocked
 
Reported: 2020-03-15 23:25 UTC by Bob Hepple
Modified: 2020-04-25 02:16 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-04-11 22:21:01 UTC
Type: Bug
Embargoed:
zebob.m: fedora-review+


Attachments (Terms of Use)

Description Bob Hepple 2020-03-15 23:25:37 UTC
Source URL: https://download.copr.fedorainfracloud.org/results/wef/azote/fedora-32-x86_64/01305908-azote/azote-1.7.10-1.fc32.wef.src.rpm
SPEC URL: https://download.copr.fedorainfracloud.org/results/wef/azote/fedora-32-x86_64/01305908-azote/azote.spec

FAS Username: wef

Description:

Azote is a GTK+3 - based picture browser and background setter, as the
frontend to the swaybg (sway/Wayland) and feh (X windows) commands. It
also includes several colour management tools.

The program is confirmed to work on sway, i3, Openbox, Fluxbox and dwm
window managers, on Arch Linux, Void Linux and Debian.

Comment 2 Artur Frenszek-Iwicki 2020-03-16 00:37:02 UTC
>License:  GPL
The proper identifier here would be "GPLv3".
https://fedoraproject.org/wiki/Licensing:Main#Good_Licenses

>Source0:  ${url}/archive/v%{version}.tar.gz
percent-URL, not dollar-URL - the latter is a bash variable, not an rpm variable.

>Requires: python-send2trash
Unversioned python is a big no-no in Fedora; use "python3-send2trash" here.
https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_dependencies

>install -p -D -m 0644 -t %{buildroot}/%{_datadir}/applications dist/%{name}.desktop
Add a BuildRequires for "desktop-file-utils" and run desktop-file-validate on the installed desktop file.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_desktop_file_install_usage

Comment 4 Aleksei Bavshin 2020-03-16 09:55:19 UTC
Bundled colorthief.py is BSD licensed.
You either need to remove it and replace with dependency to 'python3-colorthief' or change license to 'GPLv3 and BSD'. Using dependency is a preferred way.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#bundling

> Source0:  %{url}/archive/v%{version}.tar.gz
I believe the standard for github sources is %{url}/archive/v%{version}/%{name}-%{version}.tar.gz. Although it is a cosmetic change that only affects file name in lookaside cache.

> %autosetup -n %{name}-%{version}
%{name}-%{version} is already default for %autosetup. You only need to use `-n` when the toplevel directory of source archive does not match %{name}-%{version} pattern.

Comment 5 Aleksei Bavshin 2020-03-16 10:16:16 UTC
(In reply to Aleksei Bavshin from comment #4)
> Bundled colorthief.py is BSD licensed.
> You either need to remove it and replace with dependency to
> 'python3-colorthief' or change license to 'GPLv3 and BSD'. Using dependency is a preferred way.

Sorry, 'python3-colorthief' is actually not available in Fedora repository. I had it installed from COPR.
The guidelines section on bundling still applies here and it's preferable to have separate package.

Comment 6 Bob Hepple 2020-03-17 00:08:11 UTC
Thanks for your help, Aleksei.

>> Source0:  %{url}/archive/v%{version}.tar.gz
> I believe the standard for github sources is %{url}/archive/v%{version}/%{name}-%{version}.tar.gz. 

This has been troubling me for a while - the upstream releases page (https://github.com/nwg-piotr/azote/releases) has this url for the source:

https://github.com/nwg-piotr/azote/archive/v1.7.10.tar.gz

I would dearly love to know how to get github to serve %{name}-%{version}.tar.gz but I've come up with nothing, even trying %forgeurl.

2/ I've taken the 'GPLv3 and BSD' option as the colorthief dependency is not available. This implies bundling so I've included this in the spec:

Provides: bundled(python3-colorthief) = 0.2.1

... or is that only for compiled libraries?

Rebuilds:

Source URL: https://download.copr.fedorainfracloud.org/results/wef/azote/fedora-32-x86_64/01309304-azote/azote-1.7.10-3.fc32.wef.src.rpm
Spec URL:   https://download.copr.fedorainfracloud.org/results/wef/azote/fedora-32-x86_64/01309304-azote/azote.spec

Comment 7 Till Hofmann 2020-03-17 06:40:08 UTC
(In reply to Bob Hepple from comment #6)
> Thanks for your help, Aleksei.
> 
> >> Source0:  %{url}/archive/v%{version}.tar.gz
> > I believe the standard for github sources is %{url}/archive/v%{version}/%{name}-%{version}.tar.gz. 
> 
> This has been troubling me for a while - the upstream releases page
> (https://github.com/nwg-piotr/azote/releases) has this url for the source:
> 
> https://github.com/nwg-piotr/azote/archive/v1.7.10.tar.gz
> 
> I would dearly love to know how to get github to serve
> %{name}-%{version}.tar.gz but I've come up with nothing, even trying
> %forgeurl.
> 

https://github.com/nwg-piotr/azote/archive/v1.7.10/azote-1.7.10.tar.gz works, see
https://fedoraproject.org/wiki/Packaging:SourceURL?rd=Packaging/SourceURL#Github

So the source URL suggested by Aleksei should work.

Comment 9 Robert-André Mauchin 🐧 2020-03-26 17:21:03 UTC
 - Remove your wef thingie in Release:

 - Add a newline between changelog entries

 - No:

%global debug_package %{nil}

the package should be noarch instead:

BuildArch: noarch

 - Add a comment explaining the license breakdown:

# GPLv3: main program
# BSD: colorthief.py
License:  GPLv3 and BSD

Comment 11 Robert-André Mauchin 🐧 2020-03-31 14:56:20 UTC
Package approved.

Comment 12 Gwyn Ciesla 2020-04-01 13:54:24 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/azote

Comment 13 Fedora Update System 2020-04-02 04:32:53 UTC
FEDORA-2020-8a0d86a630 has been submitted as an update to Fedora 32. https://bodhi.fedoraproject.org/updates/FEDORA-2020-8a0d86a630

Comment 14 Fedora Update System 2020-04-02 04:44:17 UTC
FEDORA-2020-f5187a56b4 has been submitted as an update to Fedora 31. https://bodhi.fedoraproject.org/updates/FEDORA-2020-f5187a56b4

Comment 15 Fedora Update System 2020-04-02 04:59:24 UTC
FEDORA-EPEL-2020-a9d5b16ad2 has been submitted as an update to Fedora EPEL 8. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2020-a9d5b16ad2

Comment 16 Fedora Update System 2020-04-03 19:55:57 UTC
FEDORA-2020-8a0d86a630 has been pushed to the Fedora 32 testing repository.
In short time you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2020-8a0d86a630 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-8a0d86a630

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

Comment 17 Fedora Update System 2020-04-03 20:20:02 UTC
FEDORA-EPEL-2020-a9d5b16ad2 has been pushed to the Fedora EPEL 8 testing repository.

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

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

Comment 18 Fedora Update System 2020-04-03 20:49:27 UTC
FEDORA-2020-f5187a56b4 has been pushed to the Fedora 31 testing repository.
In short time you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2020-f5187a56b4 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-f5187a56b4

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

Comment 19 Fedora Update System 2020-04-11 22:21:01 UTC
FEDORA-2020-f5187a56b4 has been pushed to the Fedora 31 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 20 Fedora Update System 2020-04-18 00:47:21 UTC
FEDORA-EPEL-2020-a9d5b16ad2 has been pushed to the Fedora EPEL 8 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 21 Fedora Update System 2020-04-25 02:16:27 UTC
FEDORA-2020-8a0d86a630 has been pushed to the Fedora 32 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.