https://bugzilla.redhat.com/show_bug.cgi?id=1397643
Honggang LI changed:
What|Removed |Added
Status|ASSIGNED|CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1397643
--- Comment #10 from Honggang LI ---
(In reply to Michal Schmidt from comment #8)
> (In reply to Honggang LI from comment #4)
> > (In reply to Michal Schmidt from comment #3)
> > > In %description of utils use plural
https://bugzilla.redhat.com/show_bug.cgi?id=1397643
--- Comment #9 from Jon Ciesla ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/dapl
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always
https://bugzilla.redhat.com/show_bug.cgi?id=1397643
Michal Schmidt changed:
What|Removed |Added
Flags||fedora-review+
https://bugzilla.redhat.com/show_bug.cgi?id=1397643
--- Comment #7 from Honggang LI ---
Please review updated SPEC and SRPM files:
Spec URL:
http://people.redhat.com/honli/.419dc6a07891d4da7bc03168d5613fcz/dapl.spec
SRPM URL:
https://bugzilla.redhat.com/show_bug.cgi?id=1397643
--- Comment #6 from Honggang LI ---
(In reply to Doug Ledford from comment #5)
> > [!]: Package complies to the Packaging Guidelines
> > >>> Dir ownership issue noted above.
>
> dapl will co-own the /etc/rdma directory.
https://bugzilla.redhat.com/show_bug.cgi?id=1397643
Doug Ledford changed:
What|Removed |Added
CC||dledf...@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1397643
--- Comment #4 from Honggang LI ---
(In reply to Michal Schmidt from comment #3)
> Issues:
> ===
> [!]: Package must own all directories that it creates.
> Note: Directories without known owners: /etc/rdma
> >>> I
https://bugzilla.redhat.com/show_bug.cgi?id=1397643
--- Comment #3 from Michal Schmidt ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
Issues:
===
[!]: Package must own all directories that it creates.
https://bugzilla.redhat.com/show_bug.cgi?id=1397643
--- Comment #2 from Honggang LI ---
(In reply to Michal Schmidt from comment #1)
> I suggest to drop all the Obsoletes lines:
Removed.
>
> Before the ExcludeArch line:
> ExcludeArch: s390, armv7hl
> maybe add a comment:
https://bugzilla.redhat.com/show_bug.cgi?id=1397643
Michal Schmidt changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://bugzilla.redhat.com/show_bug.cgi?id=1397643
Honggang LI changed:
What|Removed |Added
CC||mschm...@redhat.com,
12 matches
Mail list logo