[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 Miroslav Suchý changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | Referenced

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 --- Comment #16 from Fedora Update System --- hexer-0.2.3-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

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 Fedora Update System changed: What|Removed |Added Status|ON_QA

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 --- Comment #14 from Fedora Update System --- hexer-0.2.3-1.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-7d69fa003b -- You are receiving this mail because:

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 Fedora Update System changed: What|Removed |Added Status|POST

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 --- Comment #13 from jiri vanek --- (In reply to Alex Kashchenko from comment #12) > Thanks for the review, on the comments: > > > There are sources of some "calc" even with manpage. Is it intentionally > > not-packed? >

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 --- Comment #12 from Alex Kashchenko --- Thanks for the review, on the comments: > There are sources of some "calc" even with manpage. Is it intentionally > not-packed? Yes, it is a separate utility myc that is

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 jiri vanek changed: What|Removed |Added Status|ASSIGNED|POST

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 --- Comment #10 from jiri vanek --- There are sources of some "calc" even with manpage. Is it intentionally not-packed? It seems that makefile is supporting setupable bindir and mandir. Maybe it is better to set them in

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 --- Comment #9 from jiri vanek --- Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed = MUST items = C/C++: [x]: Package does not

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 --- Comment #7 from Alex Kashchenko --- Updated package to upstream version 0.2.3: Spec URL: https://bitbucket.org/akasko/share/downloads/hexer.spec SRPM URL:

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 jiri vanek changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 jiri vanek changed: What|Removed |Added CC||jva...@redhat.com

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 --- Comment #6 from Upstream Release Monitoring --- akasko's scratch build of hexer-0.1.8-1.fc23.src.rpm for f23 completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12184492 -- You

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 --- Comment #5 from Michael Schwendt (Fedora Packager Sponsors Group) bugs.mich...@gmx.net --- Try pointing the fedora-review tool at this ticket: fedora-review -b 1213065 It evaluates the Spec URL: and SRPM URL: lines, downloads the latest

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 --- Comment #2 from Pranav Kant pranav...@gmail.com --- There is a superfluous space before the start of the changelog entry. * Fri Apr 17 2015 Alex Kashchenko m...@alexkasko.com 0.1.8-1 - initial package, in sigaction struct sa_mask field

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 --- Comment #3 from Pranav Kant pranav...@gmail.com --- rm -rf %{buildroot} at the beginning of the install is not required. You can remove it. -- You are receiving this mail because: You are on the CC list for the bug. You are always

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 Pranav Kant pranav...@gmail.com changed: What|Removed |Added CC||pranav...@gmail.com

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 --- Comment #4 from Alex Kashchenko m...@alexkasko.com --- Addressed comments 1, 2 and 3 (please remove _v1 postfix after download): Spec URL: https://bitbucket.org/akasko/share/downloads/hexer.spec_v1 SRPM URL:

[Bug 1213065] Review Request: hexer - ncurses-based binary editor

https://bugzilla.redhat.com/show_bug.cgi?id=1213065 Alex Kashchenko m...@alexkasko.com changed: What|Removed |Added Blocks||177841