[Bug 1337120] Review Request: gyazo - Fast screen capture tool

2016-06-18 Thread bugzilla
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 mail because: You are on the C

[Bug 1337120] Review Request: gyazo - Fast screen capture tool

2016-06-15 Thread bugzilla
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 mail because: You are on the C

[Bug 1337120] Review Request: gyazo - Fast screen capture tool

2016-06-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 Fedora Update System changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|---

[Bug 1337120] Review Request: gyazo - Fast screen capture tool

2016-06-03 Thread bugzilla
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: You are on the CC list for th

[Bug 1337120] Review Request: gyazo - Fast screen capture tool

2016-06-03 Thread bugzilla
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: You are on the CC list for th

[Bug 1337120] Review Request: gyazo - Fast screen capture tool

2016-06-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 Fedora Update System changed: What|Removed |Added Status|POST|MODIFIED -- You are receiving

[Bug 1337120] Review Request: gyazo - Fast screen capture tool

2016-05-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 Jon Ciesla changed: What|Removed |Added Flags|needinfo?(limburgher@gmail. | |com)

[Bug 1337120] Review Request: gyazo - Fast screen capture tool

2016-05-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 Patrik Opravil changed: What|Removed |Added CC||limburg...@gmail.com Flag

[Bug 1337120] Review Request: gyazo - Fast screen capture tool

2016-05-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 Patrik Opravil changed: What|Removed |Added Summary|Review Request: Gyazo - |Review Request: gyazo -

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-20 Thread bugzilla
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 notified about changes

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 jiri vanek changed: What|Removed |Added Flags||fedora-review+ -- You are receiving thi

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 jiri vanek changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | --- Comment #22 from jiri vanek --- sp

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 jiri vanek changed: What|Removed |Added Status|ASSIGNED|POST --- Comment #21 from jiri vanek --

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-20 Thread bugzilla
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? > > > > > > Without it, the

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
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 debuginfo. > > Deb

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
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 f23 and f25 packages

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
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 /usr/share/ruby/gyazo/ directory

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
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 on the CC list for

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
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 component

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
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! > > Ruby: > [x]: Test

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
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. [x]: Test suite of t

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
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 modified. Thanx! Crap. Ho

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 Dmitrij S. Kryzhevich changed: What|Removed |Added CC||kryz...@ispms.ru --- Comment

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
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 notified about changes

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
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 %{?dist} to add to rel

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
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 are on the CC list fo

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
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! All problems fixed and fi

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 Patrik Opravil changed: What|Removed |Added Blocks||177841 (FE-NEEDSPONSOR) Referenced

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-18 Thread bugzilla
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 for the bug. You are a

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-18 Thread bugzilla
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 to ruby by this mac

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-18 Thread bugzilla
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 product and componen

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 Dmitrij S. Kryzhevich changed: What|Removed |Added CC||kryz...@ispms.ru --- Comment

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-18 Thread bugzilla
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 lets you instantly gr

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 jiri vanek changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #1 from jiri vanek

[Bug 1337120] Review Request: Gyazo - Fast screen capture tool

2016-05-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1337120 jiri vanek changed: What|Removed |Added CC||jva...@redhat.com Assignee|nob..