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

Phil Wyett <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |needinfo?([email protected])



--- Comment #20 from Phil Wyett <[email protected]> ---
(In reply to Stansoft from comment #19)
> Spec URL:
> https://download.copr.fedorainfracloud.org/results/stansoft/pilotlog/fedora-
> rawhide-x86_64/10172178-pilotlog/pilotlog.spec
> SRPM URL:
> https://download.copr.fedorainfracloud.org/results/stansoft/pilotlog/fedora-
> rawhide-x86_64/10172178-pilotlog/pilotlog-11-1.fc45.src.rpm
> 
> Updated Pilotlog version and fixed the spec file issues.

If this is the first time the package would be in Fedora, then I would dro all
bu the latest %changelog entry as the initial RPM packaging.

Are you intending to ExclusivArch this package? I will do a fuller review when
this is is a yes or no. :-)

It is also the norm to put the BuildRequires and Requires in alphabetical
order, but this one is up to you.

Regards

Phil


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2117046

Report this comment as SPAM: 
https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202117046%23c20

-- 
_______________________________________________
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]
Do not reply to spam, report it: 
https://forge.fedoraproject.org/infra/tickets/issues/new

Reply via email to