Re: [Freeipa-devel] Fix removal of ipa-kdc-proxy.conf symlink

2015-06-29 Thread Christian Heimes
On 2015-06-29 17:28, Petr Vobornik wrote: > On 06/29/2015 03:22 PM, Fraser Tweedale wrote: >> On Mon, Jun 29, 2015 at 10:54:50AM +0200, Christian Heimes wrote: >>> Hello, >>> >>> the attached patch fixes the first bug, that was reported by Fraser >>> today. installutils.remove_file() uses os.path.e

Re: [Freeipa-devel] Fix removal of ipa-kdc-proxy.conf symlink

2015-06-29 Thread Petr Vobornik
On 06/29/2015 03:22 PM, Fraser Tweedale wrote: On Mon, Jun 29, 2015 at 10:54:50AM +0200, Christian Heimes wrote: Hello, the attached patch fixes the first bug, that was reported by Fraser today. installutils.remove_file() uses os.path.exists() to check if the file still exists, which in turn us

Re: [Freeipa-devel] Fix removal of ipa-kdc-proxy.conf symlink

2015-06-29 Thread Fraser Tweedale
On Mon, Jun 29, 2015 at 10:54:50AM +0200, Christian Heimes wrote: > Hello, > > the attached patch fixes the first bug, that was reported by Fraser > today. installutils.remove_file() uses os.path.exists() to check if the > file still exists, which in turn uses stat(2). I have modified the > functi

[Freeipa-devel] Fix removal of ipa-kdc-proxy.conf symlink

2015-06-29 Thread Christian Heimes
Hello, the attached patch fixes the first bug, that was reported by Fraser today. installutils.remove_file() uses os.path.exists() to check if the file still exists, which in turn uses stat(2). I have modified the function to use os.path.lexists() instead. It doesn't follow symlinks. Because http