On 06/19/2014 03:52 PM, Tomas Babej wrote:
On 06/19/2014 12:52 PM, Tomas Babej wrote:
On 06/18/2014 10:52 AM, Petr Viktorin wrote:
On 06/17/2014 02:15 PM, Tomas Babej wrote:
On 06/17/2014 12:03 PM, Timo Aaltonen wrote:
On 17.06.2014 11:16, Martin Kosek wrote:
Attached is a new version of patch 226, and a new patch 228, which moves
the paths from installers to the paths module.
In patch 226, there's another "certificated" typo in
I greped the repository, and I do not see many paths lurking around any
more, there are only some in the error messages (as these can't be
reliably replaced automatically, and will need some manual love).
If you see any forgotten paths, which should be added to the module, let
Well, since you asked...
/var/lib/sss/db -name '*.ldb' | "
"/usr/sbin/ipa-replica-conncheck " +
print_info("/usr/sbin/ipa-replica-conncheck " + " ".join(remote_check_opts))
ipapython/ipautil.py:296: env["PATH"] =
I guess it'll be a while before we catch them all, but now it's at least
clear where these paths should be, so anyone porting to another distro
can send patches (or tickets) upstream.
I see another duplicate:
SSS_KRB5_INCLUDE_D = "/var/lib/sss/pubconf/krb5.include.d"
Could you just pick one instead? Would ipa_backup.py break if it had a
trailing slash here?
In ipa-client-install, if you set:
NSSWITCH_CONF = paths.NSSWITCH_CONF
then you should only use one of those later. (Preferably paths.*, to get
rid of the redundant constants.)
Perhaps this is for another patch that would clean up all the cases
where these trivial module variables are used.
Fixed all mentioned issues. I also attached a patch 230, which removes
the base Authconfig class.
Attaching one additional patch, which removes unnecessary build warnings.
226, 230, 231 look good
Freeipa-devel mailing list