Yair Zaslavsky has posted comments on this change.

Change subject: utils: tar: support long file names within tar
......................................................................


Patch Set 2: Looks good to me, approved

Oops, did not notice that on previous reviews, maybe the package name should 
not be hostinstall, or it should be moved to somewhere else? don't you think 
that Tar util can help us in more (maybe future ) scenarios besides host 
install?

Thanks

(Besides that +2)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I364d67dc408af21e08a20014c6a0c99a413fe3be
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to