https://bugzilla.redhat.com/show_bug.cgi?id=1739816
Mattia Verga changed:
What|Removed |Added
Status|ASSIGNED|NEW
Assignee|zebo...@gmail.com
https://bugzilla.redhat.com/show_bug.cgi?id=1739816
Robert-André Mauchin changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|nob.
https://bugzilla.redhat.com/show_bug.cgi?id=1739816
--- Comment #8 from Ege Güneş ---
(In reply to Robert-André Mauchin from comment #7)
> (In reply to Ege Güneş from comment #6)
> > (In reply to Robert-André Mauchin from comment #5)
> > > - That's not needed, the file should be in licensedir
https://bugzilla.redhat.com/show_bug.cgi?id=1739816
--- Comment #7 from Robert-André Mauchin ---
(In reply to Ege Güneş from comment #6)
> (In reply to Robert-André Mauchin from comment #5)
> > - That's not needed, the file should be in licensedir
> >
> > %{_docdir}/redminecli/LICENSE
> >
>
https://bugzilla.redhat.com/show_bug.cgi?id=1739816
--- Comment #6 from Ege Güneş ---
(In reply to Robert-André Mauchin from comment #5)
> - That's not needed, the file should be in licensedir
>
> %{_docdir}/redminecli/LICENSE
>
The setup.py script tries to install license to `sys.prefix` (
https://bugzilla.redhat.com/show_bug.cgi?id=1739816
--- Comment #5 from Robert-André Mauchin ---
- That's not needed, the file should be in licensedir
%{_docdir}/redminecli/LICENSE
- Add a nemline between your changelog entry
- You haven't added the tests?
--
You are receiving this mail
https://bugzilla.redhat.com/show_bug.cgi?id=1739816
--- Comment #4 from Ege Güneş ---
(In reply to Robert-André Mauchin from comment #3)
I updated the spec according to your review.
Spec URL:
https://copr-be.cloud.fedoraproject.org/results/egegunes/redminecli/fedora-30-x86_64/01022248-redmine
https://bugzilla.redhat.com/show_bug.cgi?id=1739816
Robert-André Mauchin changed:
What|Removed |Added
CC||zebo...@gmail.com
--- Comment
https://bugzilla.redhat.com/show_bug.cgi?id=1739816
--- Comment #2 from J. Scheurich ---
This is only a informal review, i am not in the packager group
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= M
https://bugzilla.redhat.com/show_bug.cgi?id=1739816
J. Scheurich changed:
What|Removed |Added
CC||muft...@web.de
--- Comment #1 from J.
https://bugzilla.redhat.com/show_bug.cgi?id=1739816
Ege Güneş changed:
What|Removed |Added
Blocks||177841 (FE-NEEDSPONSOR)
Referenced Bugs:
11 matches
Mail list logo