https://bugzilla.redhat.com/show_bug.cgi?id=1744132

Robert-André Mauchin <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |POST
           Assignee|[email protected]    |[email protected]
              Flags|                            |fedora-review+



--- Comment #3 from Robert-André Mauchin <[email protected]> ---
(In reply to Andreas Gerstmayr from comment #2)
> Thanks for the review!
> 
> (In reply to Robert-André Mauchin from comment #1)
> >  - This shouldn't be needed as it is a noarch package:
> > 
> > %global         _debugsource_template %{nil} # avoid empty
> > debugsourcefiles.list
> 
> I removed this line now.
> 
> > 
> >  - Please add a comment or include a script as Source which explains how
> > grafana-pcp-deps-%{version}.tar.xz was generated.
> 
> Ok, I've updated the spec and the SRPM at the same location.
> 
Ok

> > grafana-pcp.noarch: E: world-writable
> > /var/lib/grafana/plugins/grafana-pcp/components/config/config.html 666
> 
> Not sure what's wrong here, when I run
> $ rpm -qp --dump ~/rpmbuild/RPMS/noarch/grafana-pcp-0.0.7-1.fc30.noarch.rpm
> | grep config/config.html
> 
> it shows me
> 
> /var/lib/grafana/plugins/grafana-pcp/components/config/config.html 613
> 1566824688 6c8a49f249880ccbc8ebb19c9097a0944b58229cc0b71948011f419839deeca6
> 0100644 root root 0 0 0 X
Ok

> 
> > grafana-pcp.src: W: invalid-url Source1: grafana-pcp-deps-0.0.7.tar.xz
> 
> Anything I can / should do about that one?
>

Nothing, just add the recipe so that someone needing to reproduce the build
know how to.


LGTM, package approved.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]

Reply via email to