Bug 1483963
Summary: | Review Request: nuvola-app-jupiter-broadcasting - Jupiter Broadcasting plugin for Nuvola Player | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | MartinKG <mgansser> |
Component: | Package Review | Assignee: | Robert-André Mauchin 🐧 <zebob.m> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | package-review, vondruch, zebob.m |
Target Milestone: | --- | Flags: | zebob.m:
fedora-review+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-08-24 09:03:26 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
MartinKG
2017-08-22 11:44:25 UTC
Hello, - You should use the %configure macro instead of ./configure --prefix=%{_prefix} - DESTDIR=%{buildroot} is not needed after %make_install - You can actually fix the typo: linux → Linux - Use this for Source0: Source0: https://github.com/tiliado/%{name}/archive/%{version}/%{name}-%{version}.tar.gz Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/nuvola-app-jupiter-broadcasting.spec SRPM URL: https://martinkg.fedorapeople.org/Review/SRPMS/nuvola-app-jupiter-broadcasting-1.1-2.fc26.src.rpm %changelog * Wed Aug 23 2017 Martin Gansser <martinkg> - 1.1-2 - use the %%configure macro - DESTDIR=%%{buildroot} is not needed after %%make_install - fix typo - Use correct Source0 All good, package accepted. Thanks for the review. (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/nuvola-app-jupiter-broadcasting package has been built successfully on f25, f26, f27 and rawhide. |