Juan Hernandez has posted comments on this change.

Change subject: Shell-In-A-Box UI plugin - adding RPM spec file
......................................................................


Patch Set 2: (1 inline comment)

....................................................
File shellbox-plugin/ovirt-engine-shellinabox-uiplugin.spec
Line 5: License: GPL
Line 6: BuildArch: noarch
Line 7: Source0: %{name}-%{version}.tar
Line 8: 
Line 9: %define pluginspath /usr/share/ovirt-engine/ui-plugins
We can skip the dependency completely, and things will work anyway, only that 
in older versions of ovirt-engine (less than 3.3, the change to create the 
ui-plugins directory is in [1]) the ui-plugins directory will be created but 
not owned by any package, so it won't be removed when the ovirt-engine packages 
are removed. Not a big deal.

[1] http://gerrit.ovirt.org/11200
Line 10: 
Line 11: %description
Line 12: ShellBox UI plugin - provides SSH connection to a host using
Line 13: shellinabox - a Web based AJAX terminal emulator


--
To view, visit http://gerrit.ovirt.org/11198
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If916c7cf022e9e903cd06e55f7b8ff34851ab9c8
Gerrit-PatchSet: 2
Gerrit-Project: samples-uiplugins
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Vojtech Szocs <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to