Spec URL: https://pagure.io/audiotube_rpm/raw/main/f/audiotube.spec SRPM URL: https://pagure.io/audiotube_rpm/raw/main/f/audiotube-21.12-1.fc35.src.rpm Description: AudioTube can search YouTube Music, list albums and artists, play automatically generated playlists, albums and allows to put your own playlist together. Fedora Account System Username: thunderbirdtr
Initial spec review: > BuildRequires: yt-dlp > BuildRequires: python3-ytmusicapi These also need to be declared as "Requires" too.
(In reply to Neal Gompa from comment #1) > Initial spec review: > > > BuildRequires: yt-dlp > > BuildRequires: python3-ytmusicapi > > These also need to be declared as "Requires" too. Done !
> Requires: yt-dlp Requires: python3-ytmusicapi Indentation is wacky here...
> %description > %{summary} This should be "%{summary}." in the description simply to make rpmlint shut up. > %prep > %autosetup -n %{name}-%{version} "-n %{name}-%{version}" is superfluous, since that's the default. Drop it. Also, license files are not captured with a "%license" stanza. Add "%license LICENSES/*" to the file list.
All done!
> %description > %{summary} Still not fixed.
(In reply to Neal Gompa from comment #6) > > %description > > %{summary} > > Still not fixed. Fixed.
Review notes: [x]: Package follows Fedora Packaging Guidelines [x]: Package builds and installs (when its BRs are installed) [x]: No serious issues from rpmlint [x]: License is correctly identified and installed PACKAGE APPROVED.
(fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/audiotube
Package is now in repositories, closing review.