Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-12-11 Thread Martin Kosek
...@redhat.com Cc: freeipa-devel@redhat.com Sent: Friday, November 9, 2012 3:25:20 PM Subject: Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors On 8.11.2012 17:22, Lynn Root wrote: Hmm I hope I understand well enough this time around. However, when I run the tests, there's

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-11-12 Thread Lynn Root
Sent: Thursday, November 8, 2012 8:46:42 AM Subject: Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors On 11/07/2012 06:46 PM, Jan Cholasta wrote: On 7.11.2012 16:08, Lynn Root wrote: Third time is a charm? Lynn Root Associate Software Engineer Red Hat

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-11-12 Thread Jan Cholasta
Sent: Thursday, November 8, 2012 8:46:42 AM Subject: Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors On 11/07/2012 06:46 PM, Jan Cholasta wrote: On 7.11.2012 16:08, Lynn Root wrote: Third time is a charm? Lynn Root Associate Software Engineer Red Hat - Original

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-11-12 Thread Lynn Root
:25:20 PM Subject: Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors On 8.11.2012 17:22, Lynn Root wrote: Hmm I hope I understand well enough this time around. However, when I run the tests, there's this one error message I come across from `test_user[97

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-11-12 Thread Jan Cholasta
: Friday, November 9, 2012 3:25:20 PM Subject: Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors On 8.11.2012 17:22, Lynn Root wrote: Hmm I hope I understand well enough this time around. However, when I run the tests, there's this one error message I come across from `test_user

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-11-09 Thread Jan Cholasta
:42 AM Subject: Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors On 11/07/2012 06:46 PM, Jan Cholasta wrote: On 7.11.2012 16:08, Lynn Root wrote: Third time is a charm? Lynn Root Associate Software Engineer Red Hat - Original Message - From: Jan Cholasta jchol

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-11-08 Thread Lynn Root
...@redhat.com Cc: freeipa-devel@redhat.com Sent: Monday, November 5, 2012 10:25:32 AM Subject: Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors On 5.11.2012 09:43, Lynn Root wrote: Here's try #2! Adjusted patch attached. Let me know if there's anything else I've missed

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-11-07 Thread Martin Kosek
Subject: Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors On 5.11.2012 09:43, Lynn Root wrote: Here's try #2! Adjusted patch attached. Let me know if there's anything else I've missed. Switched %r specifiers to '%s' in Public errors, and adjusted tests to expect

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-11-07 Thread Jan Cholasta
@redhat.com Sent: Tuesday, October 30, 2012 9:08:33 AM Subject: Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors On 10/30/2012 09:04 AM, Jan Cholasta wrote: Hi, On 29.10.2012 19:54, Lynn Root wrote: Hi all! This switch drops the preceding 'u' from strings in public error messages

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-11-07 Thread Martin Kosek
: Monday, November 5, 2012 10:25:32 AM Subject: Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors On 5.11.2012 09:43, Lynn Root wrote: Here's try #2! Adjusted patch attached. Let me know if there's anything else I've missed. Switched %r specifiers to '%s' in Public errors

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-11-05 Thread Lynn Root
Root Associate Software Engineer Red Hat - Original Message - From: Martin Kosek mko...@redhat.com To: Jan Cholasta jchol...@redhat.com Cc: Lynn Root lr...@redhat.com, freeipa-devel@redhat.com Sent: Tuesday, October 30, 2012 9:08:33 AM Subject: Re: [Freeipa-devel] [PATCH] Switch %r

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-11-05 Thread Jan Cholasta
: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors On 10/30/2012 09:04 AM, Jan Cholasta wrote: Hi, On 29.10.2012 19:54, Lynn Root wrote: Hi all! This switch drops the preceding 'u' from strings in public error messages. Ticket: https://fedorahosted.org/freeipa/ticket/3121

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-10-30 Thread Jan Cholasta
Hi, On 29.10.2012 19:54, Lynn Root wrote: Hi all! This switch drops the preceding 'u' from strings in public error messages. Ticket: https://fedorahosted.org/freeipa/ticket/3121 This patch also addresses the unfriendly 'u' from re-raising errors from the external call to netaddr.IPAddress

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-10-30 Thread Martin Kosek
On 10/30/2012 09:04 AM, Jan Cholasta wrote: Hi, On 29.10.2012 19:54, Lynn Root wrote: Hi all! This switch drops the preceding 'u' from strings in public error messages. Ticket: https://fedorahosted.org/freeipa/ticket/3121 This patch also addresses the unfriendly 'u' from re-raising

Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors

2012-10-30 Thread Lynn Root
, 2012 9:08:33 AM Subject: Re: [Freeipa-devel] [PATCH] Switch %r specifiers to %s in Public errors On 10/30/2012 09:04 AM, Jan Cholasta wrote: Hi, On 29.10.2012 19:54, Lynn Root wrote: Hi all! This switch drops the preceding 'u' from strings in public error messages. Ticket: https