[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 Mikolaj Izdebski changed: What|Removed |Added Blocks|652183 (FE-JAVASIG) | Referenced Bugs:

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 Martin Perina changed: What|Removed |Added Status|NEW |CLOSED Resolution|---

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 Piotr Kliczewski changed: What|Removed |Added CC||mper...@redhat.com

needinfo canceled: [Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

Product: Fedora Version: rawhide Component: Package Review Piotr Kliczewski has canceled Package Review 's request for Piotr Kliczewski 's needinfo: Bug 1149176: Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt https://bugzilla.redhat.com/show_bug.cgi?id=1149176

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 automat...@ovirt.org changed: What|Removed |Added External Bug ID||oVirt gerrit 63673 -- You are

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 --- Comment #19 from Piotr Kliczewski --- Michael thank you for your feedback. Will follow your suggestions. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 --- Comment #18 from Michael Simacek --- Looks much better now. Issues I found so far: - There are missing BuildRequires, so the package doesn't build in mock. Most notably "maven-local" package which provides mvn_build.

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 --- Comment #17 from Piotr Kliczewski --- I removed old spec and srpm and uploaded updated schema and newly created srpm: Spec URL: https://drive.google.com/open?id=0ByDkUDLXsT0YYTZqTGc2YTJnSDQ To download: wget

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 --- Comment #16 from Michael Simacek --- Great. Most of the links to relevant documentation are available in https://fedoraproject.org/wiki/Join_the_package_collection_maintainers Java packaging HowTo is available at

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 --- Comment #15 from Piotr Kliczewski --- Will do that. -- 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 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 --- Comment #14 from Michael Simacek --- I only see the comment saying that you updated it, but not the updated specfile. I'd like to note that while Fedora and RHEL based products use RPM specfiles, their usage and

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 Piotr Kliczewski changed: What|Removed |Added Flags|needinfo?(pkliczew@redhat.c |

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 Mikolaj Izdebski changed: What|Removed |Added Flags|

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 --- Comment #11 from Piotr Kliczewski --- The spec was updated long time ago and still waiting on review. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 Douglas Schilling Landgraf changed: What|Removed |Added Assignee|dougsl...@redhat.com

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 Douglas Schilling Landgraf changed: What|Removed |Added Assignee|nob...@fedoraproject.org

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 --- Comment #9 from Piotr Kliczewski --- (In reply to Douglas Schilling Landgraf from comment #6) > Hi Piotr, > > I see you updated the package. Let's start with the below, please also > consider the previous comments. >

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 --- Comment #8 from Piotr Kliczewski --- El6 is supported as well and we build it every time there is change in the code within this package. -- You are receiving this mail because: You are on the CC list for the bug.

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 --- Comment #6 from Douglas Schilling Landgraf --- Hi Piotr, I see you updated the package. Let's start with the below, please also consider the previous comments. - Point the spec and srpm to a different url, so it

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 --- Comment #7 from Douglas Schilling Landgraf --- Also, please also double check if vdsm-jsonrpc-java spec should keep the EL6 support, looks like the current vdsm only supports EL7 in current days. -- You are

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 Mikolaj Izdebski changed: What|Removed |Added CC|

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 --- Comment #5 from Douglas Schilling Landgraf --- Hi Piotr, As mentioned in comment#3 it's recommended to provide the spec and sprm in it can be downloaded directly with tools like wget. The Source is pointing to

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 Michael Simacek changed: What|Removed |Added Status|ASSIGNED|NEW

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 gil cattaneo changed: What|Removed |Added Blocks||201449

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 Michael Simacek changed: What|Removed |Added CC|

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 Piotr Kliczewski changed: What|Removed |Added Flags|needinfo?(pkliczew@redhat.c |

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 Michael Simacek changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

https://bugzilla.redhat.com/show_bug.cgi?id=1149176 gil cattaneo changed: What|Removed |Added Blocks||652183 (FE-JAVASIG)