https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #29 from Fedora Update System ---
gyazo-1.2-1.fc24 has been pushed to the Fedora 24 stable repository. If
problems still persist, please make note of it in this bug report.
--
You are receiving this
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #28 from Fedora Update System ---
gyazo-1.2-1.fc23 has been pushed to the Fedora 23 stable repository. If
problems still persist, please make note of it in this bug report.
--
You are receiving this
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
Fedora Update System changed:
What|Removed |Added
Status|MODIFIED
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #27 from Fedora Update System ---
gyazo-1.2-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-3193c9df0d
--
You are receiving this mail because:
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #26 from Fedora Update System ---
gyazo-1.2-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-b6f000cc62
--
You are receiving this mail because:
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
Fedora Update System changed:
What|Removed |Added
Status|POST
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
Jon Ciesla changed:
What|Removed |Added
Flags|needinfo?(limburgher@gmail. |
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
Patrik Opravil changed:
What|Removed |Added
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
Patrik Opravil changed:
What|Removed |Added
Summary|Review Request: Gyazo - |Review
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #23 from Jon Ciesla ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/Gyazo
--
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=1337120
jiri vanek changed:
What|Removed |Added
Flags||fedora-review+
--
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
jiri vanek changed:
What|Removed |Added
Blocks|177841 (FE-NEEDSPONSOR) |
--- Comment #22
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
jiri vanek changed:
What|Removed |Added
Status|ASSIGNED|POST
--- Comment #21
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #20 from Patrik Opravil ---
(In reply to jiri vanek from comment #19)
> (In reply to Dmitrij S. Kryzhevich from comment #18)
> > > > Not issue but, what is the reason for debug_package variable in spec?
>
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #19 from jiri vanek ---
(In reply to Dmitrij S. Kryzhevich from comment #18)
> > > Not issue but, what is the reason for debug_package variable in spec?
> >
> > Without it, the rpm build fail trying to generate
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #18 from Dmitrij S. Kryzhevich ---
> > Not issue but, what is the reason for debug_package variable in spec?
>
> Without it, the rpm build fail trying to generate debuginfo.
Debuginfo? In noarch package? Both
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #17 from jiri vanek ---
You have left one nasty rpm build warning in:
File listed twice: /usr/share/ruby/gyazo/gyazo.rb
Remove this file from %files, it is included automaticay, when you added the
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #16 from jiri vanek ---
Spec URL: https://jvanek.fedorapeople.org/gyazo/v3/gyazo.spec
SRPM URL: https://jvanek.fedorapeople.org/gyazo/v3/gyazo-1.2-1.fc24.src.rpm
--
You are receiving this mail because:
You are
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #15 from Patrik Opravil ---
All fixed. New version on Drive.
--
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
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #14 from jiri vanek ---
(In reply to jiri vanek from comment #13)
> (In reply to jiri vanek from comment #10)
> > Please fix above 3 issues, oonce done, I will rerun the review with ruby
> > plugin on. Thanx!
>
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #13 from jiri vanek ---
(In reply to jiri vanek from comment #10)
> Please fix above 3 issues, oonce done, I will rerun the review with ruby
> plugin on. Thanx!
Ruby:
[x]: Test suite should not be run by rake.
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #12 from jiri vanek ---
(In reply to Dmitrij S. Kryzhevich from comment #11)
> There are two more issues.
>
> 1. There is a %{_bindir} macro for /usr/bin. There are several entries in
> spec for it to be
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
Dmitrij S. Kryzhevich changed:
What|Removed |Added
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #10 from jiri vanek ---
Please fix above 3 issues, oonce done, I will rerun the review with ruby plugin
on. Thanx!
--
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=1337120
--- Comment #9 from jiri vanek ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
[0] Dist tag is missing . You need
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #8 from jiri vanek ---
Thanx!
Spec URL: https://jvanek.fedorapeople.org/gyazo/v2/gyazo.spec
SRPM URL: https://jvanek.fedorapeople.org/gyazo/v2/gyazo-1.2-1.src.rpm
--
You are receiving this mail because:
You
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #7 from Patrik Opravil ---
(In reply to jiri vanek from comment #6)
> desktop-file-validate %{buildroot}/usr/share/applications/gyazo.desktop
>
> belongs to %check section (if it works for you)
Hi jiri!
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
Patrik Opravil changed:
What|Removed |Added
Blocks||177841
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #6 from jiri vanek ---
desktop-file-validate %{buildroot}/usr/share/applications/gyazo.desktop
belongs to %check section (if it works for you)
--
You are receiving this mail because:
You are on the CC list
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #5 from jiri vanek ---
Hi Patrik!
First few nits:
There is license in:
https://github.com/gyazo/Gyazo-for-Linux/blob/master/debian/copyright
Please pack it as %license
You should replace ahrdocded paths
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #4 from jiri vanek ---
Hi Dimirij, I will sponsor him on the way through this package.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
Dmitrij S. Kryzhevich changed:
What|Removed |Added
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
--- Comment #2 from jiri vanek ---
Spec URL: https://jvanek.fedorapeople.org/gyazo/v1/gyazo.spec
SRPM URL: https://jvanek.fedorapeople.org/gyazo/v1/gyazo-1.2-1.src.rpm
Description: Seriously Instant Screen-Grabbing Gyazo
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
jiri vanek changed:
What|Removed |Added
Status|NEW |ASSIGNED
--- Comment
https://bugzilla.redhat.com/show_bug.cgi?id=1337120
jiri vanek changed:
What|Removed |Added
CC||jva...@redhat.com
35 matches
Mail list logo