Juan Hernandez has posted comments on this change. Change subject: tools: engine-config bug fixes ......................................................................
Patch Set 6: (1 inline comment) .................................................... File backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/ConfigurationProvider.java Line 64: temp.setWritable(true, true); Sorry to insist, but it is not only the setExecutable method, setReadable and setWritable don't work either the way you want. They can only add permissions, but not remove them. Only alternative before Java 7 is chmod, or some native code. -- To view, visit http://gerrit.ovirt.org/6096 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3003f20a51bb5a62387593d914894d62b012c70b Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shav...@redhat.com> Gerrit-Reviewer: Doron Fediuck <dfedi...@redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com> Gerrit-Reviewer: Shahar Havivi <shav...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
