Alon Bar-Lev has posted comments on this change.

Change subject: backend: upgrade - process symbolic link files
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.ovirt.org/#/c/27395/1/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetoVirtISOsQuery.java
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetoVirtISOsQuery.java:

Line 32
Line 33
Line 34
Line 35
Line 36
> We can replace but it's not related to this bugzilla. Another patch can ach
it is.

as if you chase symlinks you should get rid of duplicates.


Line 58
Line 59
Line 60
Line 61
Line 62
> import java.io.File;
.getCanonicalPath()?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If4084095c40848dc5573d434c1c7ca497883b280
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to