Re: [Freeipa-devel] [PATCH] initial commit of log watcher (lwatch)

2009-07-22 Thread John Dennis
. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] initial commit of log watcher (lwatch)

2009-07-24 Thread John Dennis
documentation is on the to-do list. This is all for now. Thank you for your review and attention. I'm preparing a new patch. I've got all of Stephen's issues fixed and I'll move on now to folding in your suggestions, the patch will follow. -- John Dennis jden...@redhat.com Looking to carve out

Re: [Freeipa-devel] [PATCH] initial commit of log watcher (lwatch)

2009-07-25 Thread John Dennis
for threads is if there were some reason we couldn't use asynchronous non-blocking methods. Asynchronous non-blocking methods are to the best of my knowledge the preferred approach in the project. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

Re: [Freeipa-devel] [PATCH] 273 join a host to an IPA domain

2009-09-25 Thread John Dennis
. I'm going to wait until I've got my xmlrpc-c code working to my satisfaction before I post the issues. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel

Re: [Freeipa-devel] [PATCH] 290 set cert_t context on some files for selfsign plugin

2009-10-08 Thread John Dennis
to add a try/except block around that code in selfsign and return a non-zero status if it fails. Do we have predefined status codes I should be using? -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

[Freeipa-devel] fedora-ds problem with kickstart in virtlab for F11

2009-10-09 Thread John Dennis
. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] why doesn't ipapython.ipautil.run() log what it's running?

2009-10-13 Thread John Dennis
. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] why doesn't ipapython.ipautil.run() log what it's running?

2009-10-13 Thread John Dennis
On 10/13/2009 02:25 PM, Rob Crittenden wrote: John Dennis wrote: Is there a reason why the run command (ipapython.ipautil.run()) does not log what command it's running? It logs the stdout and stderr output of the command (but without indicating what the log output is, so a lot of time it just

[Freeipa-devel] add_global_options() vs. build_global_parser()

2009-10-13 Thread John Dennis
ipalib.util defines the function add_global_options() which is never called but seems nearly identical to API.build_global_parser(). Why? Should API.build_global_parser() be calling util.add_global_options() or should util.add_global_options() be nuked as cruft? -- John Dennis jden

Re: [Freeipa-devel] Integer parameters

2009-10-19 Thread John Dennis
On 10/19/2009 09:12 AM, Pavel Zuna wrote: John Dennis wrote: I wanted to assure myself if a command was expecting an integer value, it could be input in whatever radix the user desires and be correctly converted. If I understand correctly this code is in parameters.py and is implemented

Re: [Freeipa-devel] validating return values in XML-RPC

2009-10-22 Thread John Dennis
. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] validating return values in XML-RPC

2009-10-23 Thread John Dennis
On 10/22/2009 08:38 PM, Rob Crittenden wrote: John Dennis wrote: On 10/22/2009 10:45 AM, Jason Gerard DeRose wrote: So I've been thinking about this as I've been doing the UI tuning (extending meta-data and making the engine smarter). I agree with John that we need to describe the return

[Freeipa-devel] localized strings

2009-11-10 Thread John Dennis
? Am I missing something about how we're supposed to be coding this? This also raised the question in my mind about how we're binding _(), the only example I could find was this: from request import ugettext as _ Is this what all the modules are supposed to be doing to bind _()? -- John

Re: [Freeipa-devel] localized strings

2009-11-11 Thread John Dennis
On 11/11/2009 03:45 PM, Jason Gerard DeRose wrote: On Tue, 2009-11-10 at 20:42 -0500, John Dennis wrote: I was writing some new code and wanted to return an error message. Error messages need to be localized. Typically this is done via the _() method. But _() has to be defined

[Freeipa-devel] [PATCH] respect debug arg during server install

2009-11-19 Thread John Dennis
The debug flag (e.g. -d) was not being respected during server install. This patch corrects that. --- install/tools/ipa-server-install |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/install/tools/ipa-server-install b/install/tools/ipa-server-install index

[Freeipa-devel] [PATCH] error strings in documentation were missing unicode specifier

2009-11-19 Thread John Dennis
error strings in documentation were missing unicode specifier --- ipalib/errors.py |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/ipalib/errors.py b/ipalib/errors.py index 1c358cd..ca1851c 100644 --- a/ipalib/errors.py +++ b/ipalib/errors.py @@ -390,7 +390,7 @@

[Freeipa-devel] [PATCH] add new error class for certificate operations

2009-11-19 Thread John Dennis
add new error class for certificate operations --- ipalib/errors.py | 29 - 1 files changed, 28 insertions(+), 1 deletions(-) diff --git a/ipalib/errors.py b/ipalib/errors.py index 1c358cd..f23a636 100644 --- a/ipalib/errors.py +++ b/ipalib/errors.py @@ -89,7 +89,9

[Freeipa-devel] [PATCH] Reading Int parameter class should respect radix prefix

2009-11-19 Thread John Dennis
Signed-off-by: John Dennis jden...@redhat.com Reading INT parameter class should respect radix prefix The Int parameter class was not respecting any radix prefix (e.g. 0x) the user may have supplied. This patch implements _convert_scalar method for the Int class so that we can pass the special

[Freeipa-devel] [PATCH] If plugin fails to load log the traceback

2009-11-19 Thread John Dennis
Signed-off-by: John Dennis jden...@redhat.com If plugin fails to load log the traceback If a plugin fails to load due to some kind of error it would be nice if the error log contained the traceback so you can examine what went wrong rather than being left blind as to why it failed to load

[Freeipa-devel] [PATCH] along with stdout, stderr also log the initial command

2009-11-19 Thread John Dennis
Signed-off-by: John Dennis jden...@redhat.com along with stdout,stderr also log the initial command This implements better logging of external commands. Formerly we were just outputting stdout and stderr without labeling which was which. We also omitted the initial command and it's arguments

[Freeipa-devel] [PATCH] Make NotImplementedError in rabase return the correct function name

2009-11-19 Thread John Dennis
Signed-off-by: John Dennis jden...@redhat.com Make NotImplementedError in rabase return the correct function name The NotImplementedError's in rabase were returning the wrong function name, looks like a simple cut-n-paste bug. --- ipaserver/plugins/rabase.py |8 1 files changed

[Freeipa-devel] [PATCH] Require current versions of python-nss python-lxml

2009-11-19 Thread John Dennis
is dependent on a newer version of python-nss. To the best of my knowledge both python-lxml and python-nss.0-8 are currently available in the RHEL5 IPA compose, they are both definately available in fedora F-11, F-12, rawhide. --- From: John Dennis jden...@redhat.com Subject: [PATCH] Require current

[Freeipa-devel] [PATCH] [Resend] Reading INT parameter class should respect radix prefix

2009-11-20 Thread John Dennis
for the Int class so that we can pass the special radix base of zero to the int constructor telling it to determine the radix from the prefix (if present). --- From: John Dennis jden...@redhat.com Subject: [PATCH] Reading INT parameter class should respect radix prefix ipalib/parameters.py

Re: [Freeipa-devel] [PATCH] Fix raising nonexistent errros.NotImplemented

2009-11-20 Thread John Dennis
On 11/20/2009 02:50 PM, Rob Crittenden wrote: John Dennis wrote: - raise errors.NotImplementedError('%s.check_request_status' % self.name) + raise NotImplementedError('%s.check_request_status' % self.name) This will go to the client as an Internal Error. Is this what we want? I must've

Re: [Freeipa-devel] remove config files on uninstall?

2009-11-30 Thread John Dennis
be removed. But shouldn't this be handled by rpm? In other words shouldn't the files be %ghost and marked as %config? -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list

Re: [Freeipa-devel] [PATCH] jderose 028 Lossless datetime round-trip

2009-12-03 Thread John Dennis
imagine it would raise a ValueError just like any other ill formed parameter. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com

Re: [Freeipa-devel] [PATCH] jderose 028 Lossless datetime round-trip

2009-12-04 Thread John Dennis
On 12/04/2009 09:49 AM, Rob Crittenden wrote: John Dennis wrote: On 12/03/2009 11:56 AM, Rob Crittenden wrote: What should this do it if the incoming DateTime value is not parsed correctly by datetime.datetime()? Well in theory this should never happen if the parameter value complies

Re: [Freeipa-devel] Declarative tests

2009-12-04 Thread John Dennis
Cool, that looks really nice Jason. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] Re: [PATCH] dogtag clean-up

2009-12-04 Thread John Dennis
On 12/02/2009 07:01 PM, John Dennis wrote: Easy to browse HTML documentation for the dogtag plugin can be found here (for the time being) http://jdennis.fedorapeople.org/ipa/dogtag opps ... I just realized this html was not generated properly and most of the formatting, footnotes, and cross

Re: [Freeipa-devel] [PATCH] 327 pass debug option in client installer

2009-12-04 Thread John Dennis
On 12/04/2009 04:40 PM, Rob Crittenden wrote: Pass along the debug option in the client installer to the ipa-join command. ACK -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing

Re: [Freeipa-devel] [PATCH] 343 Fix CA installer

2010-01-06 Thread John Dennis
On 01/06/2010 05:22 PM, Rob Crittenden wrote: In the Fedora package reviews the service name was changed from having a separate service for each CA instance to a single pki-cad service. Change the service name and use the instance name more appropriately. ACK -- John Dennis jden...@redhat.com

[Freeipa-devel] [PATCH] pass DER flag to x509.get_serial_number()

2010-01-19 Thread John Dennis
From: jdennis jden...@vm-041.idm.lab.bos.redhat.com --- ipalib/plugins/cert.py |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/ipalib/plugins/cert.py b/ipalib/plugins/cert.py index a22d075..312ad39 100644 --- a/ipalib/plugins/cert.py +++ b/ipalib/plugins/cert.py @@

[Freeipa-devel] [PATCH] Create pkiuser before calling pkicreate, pkicreate depends on the user existing

2010-01-20 Thread John Dennis
--- ipaserver/install/cainstance.py |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/ipaserver/install/cainstance.py b/ipaserver/install/cainstance.py index a43809c..97ba833 100644 --- a/ipaserver/install/cainstance.py +++ b/ipaserver/install/cainstance.py @@ -445,9

Re: [Freeipa-devel] [PATCH] 350 improvements to cert plugin

2010-01-20 Thread John Dennis
can include the subject when retrieving a cert with cert_get. So now the csr file can be in pem format? Great, I was just about to file a bug for that. You're one step ahead of me. ACK -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

Re: [Freeipa-devel] [PATCH] 352 fix ipa-rmkeytab

2010-01-20 Thread John Dennis
On 01/20/2010 11:50 AM, Rob Crittenden wrote: On F-12 I noticed that ipa-rmkeytab failed trying to remove entries. Turned out I needed to suspend looping when doing the removal. I think it was a fluke that this worked on F-11 with an older krb5-server. ACK -- John Dennis jden...@redhat.com

[Freeipa-devel] autogen.sh script?

2010-01-25 Thread John Dennis
the wrong versions of some of the tools. I can fix the version comparison but my inclination is not to bother since autogen scripts have been replaced by the autoreconf tool. As for testing the minimum required tool version; is there an actual problem being solved by this? -- John Dennis jden

[Freeipa-devel] not ascii, not utf-8, what's a parser supposed to do?

2010-01-26 Thread John Dennis
' % \ (40,94,40,0,124,10,124,13,124,32,124,58,124,60,41,124,91,0,10,13,128,45,255,93,43,124,91,32,93,43,36,41) Which is pretty unreadable, but with sufficient comments could be acceptable. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https

[Freeipa-devel] Why do we have so much duplicated code?

2010-01-26 Thread John Dennis
for the duplication can we start moving some of this stuff to common libraries? -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com

Re: [Freeipa-devel] Why do we have so much duplicated code?

2010-01-27 Thread John Dennis
On 01/27/2010 06:00 AM, Jason Gerard DeRose wrote: On Tue, 2010-01-26 at 18:55 -0500, John Dennis wrote: I constantly find identical code spread across multiple files. Is there a reason for this code duplication? (Perhaps trying to keep import name spaces isolated?) It seems to me code

[Freeipa-devel] what's deprecated what's in use in our tree?

2010-01-27 Thread John Dennis
is deprecated (right?) that's not a problem, right? If ipapython and ipaclient are deprecated why are we still building the ipa-python and ipa-client rpms? -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

Re: [Freeipa-devel] [PATCH] 356 update dogtag configuration to work with NSS CVE-2009-3555 fixes

2010-01-27 Thread John Dennis
but will attempt to re-do the handshake to make this happen which will fail with newer versions of NSS. This fixed version of NSS is currently in Fedora updates-testing but this patch should work with either release. ACK -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com

Re: [Freeipa-devel] what's deprecated what's in use in our tree?

2010-01-27 Thread John Dennis
code outside the framework belongs. Is there a fine distinction being drawn between the two import areas such that they can't live together? Can you define the term framework? -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

[Freeipa-devel] [PATCH] convert SAFE_STRING_PATTERN from string to integer sequence to prevent xgettext from aborting when it sees a non-ascii string

2010-01-27 Thread John Dennis
--- ipapython/ipautil.py | 16 +++- ipaserver/ipautil.py | 16 +++- 2 files changed, 30 insertions(+), 2 deletions(-) diff --git a/ipapython/ipautil.py b/ipapython/ipautil.py index 1399c70..3768fa0 100644 --- a/ipapython/ipautil.py +++ b/ipapython/ipautil.py @@ -293,7

[Freeipa-devel] [PATCH] Remove (un)wrap_binary_data cruft from */ipautil.py

2010-01-28 Thread John Dennis
Remove SAFE_STRING_PATTERN, safe_string_re, needs_base64(), wrap_binary_data(), unwrap_binary_data() from both instances of ipautil.py. This code is no longer in use and the SAFE_STRING_PATTERN regular expression string was causing xgettext to abort because it wasn't a valid ASCII string. ---

[Freeipa-devel] [PATCH] fix error message to be i18n translator friendly

2010-01-28 Thread John Dennis
This error message was producing a warning from xgettext because there were multiple substations in the string. In some languages it may be necessary to reorder the substitutions for a proper translation, this is only possible if the substitutions use named values. ---

Re: [Freeipa-devel] [PATCH] 360 be smarter about decoding certs

2010-01-28 Thread John Dennis
and decode it and let the decoder decide if it's really base64, the decoder has much strong rules about the input, including assuring the padding is correct. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

Re: [Freeipa-devel] [PATCH] 359 allow cert renewal

2010-01-28 Thread John Dennis
this description goofed up. ACK to the correct patch -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa

Re: [Freeipa-devel] [PATCH] Fix File parameter validation when prompting.

2010-01-28 Thread John Dennis
the same value. In this scheme the you don't need to keep state, you don't need to special case any code, you can use the existing normalize and validate mechanisms. Thoughts? -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

Re: [Freeipa-devel] [PATCH] 360 be smarter about decoding certs

2010-01-28 Thread John Dennis
On 01/28/2010 10:30 PM, Rob Crittenden wrote: John Dennis wrote: On 01/28/2010 04:15 PM, Rob Crittenden wrote: Gah, got the description mixed up with the last patch :-( Be a bit smarter about decoding certificates that might be base64 encoded. First see if it only contains those characters

Re: [Freeipa-devel] [PATCH] Fix File parameter validation when prompting.

2010-01-29 Thread John Dennis
On 01/29/2010 07:53 AM, Pavel Zuna wrote: John Dennis wrote: In this scheme the you don't need to keep state, you don't need to special case any code, you can use the existing normalize and validate mechanisms. The problem here is that validate/normalize can only be used on final values passed

Re: [Freeipa-devel] [PATCH] 360 be smarter about decoding certs

2010-01-29 Thread John Dennis
On 01/29/2010 09:28 AM, Rob Crittenden wrote: John Dennis wrote: On 01/28/2010 10:30 PM, Rob Crittenden wrote: John Dennis wrote: On 01/28/2010 04:15 PM, Rob Crittenden wrote: Gah, got the description mixed up with the last patch :-( Be a bit smarter about decoding certificates that might

Re: [Freeipa-devel] [PATCH] 360 be smarter about decoding certs

2010-01-29 Thread John Dennis
On 01/29/2010 11:26 AM, Rob Crittenden wrote: John Dennis wrote: On 01/29/2010 09:28 AM, Rob Crittenden wrote: John Dennis wrote: On 01/28/2010 10:30 PM, Rob Crittenden wrote: John Dennis wrote: On 01/28/2010 04:15 PM, Rob Crittenden wrote: Gah, got the description mixed up with the last

Re: [Freeipa-devel] [PATCH] 364 base64-encode binary data for outputting to cli

2010-02-01 Thread John Dennis
On 02/01/2010 02:03 PM, Rob Crittenden wrote: The python type (str or unicode) tells us whether a variable is binary or text. Use this knowledge to base64-encode binary data so we stop hosing up terminals, especially when displaying certificates. rob ACK -- John Dennis jden...@redhat.com

Re: [Freeipa-devel] [PATCH] 365 be smarter about base64 decoding

2010-02-01 Thread John Dennis
. rob ACK -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 367 fix some encoding issues

2010-02-02 Thread John Dennis
On 02/02/2010 05:23 PM, Rob Crittenden wrote: I forgot the 'make test' step before submitting my base64 encoding in the cli patch. This fixes a few minor embedded documentation test failures and a limitation that it only accepted string types. rob ACK -- John Dennis jden...@redhat.com

Re: [Freeipa-devel] [PATCH] Remove (un)wrap_binary_data cruft from */ipautil.py

2010-02-03 Thread John Dennis
On 02/03/2010 05:07 AM, Jason Gerard DeRose wrote: Could you submit this again? Done, attached. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From 6d65b8faeb6ec69bfba4e7674ce1fa4379b959c9 Mon Sep 17 00:00:00 2001 From: John Dennis jden

Re: [Freeipa-devel] [PATCH] 370 set default log level

2010-02-03 Thread John Dennis
On 02/03/2010 04:27 PM, Rob Crittenden wrote: The ipa-*-manage programs had the default log level set to NOTSET which was causing copious logging to occur. Setting it to ERROR fixes this. ACK -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

Re: [Freeipa-devel] [PATCH] 368 remove some duplicate code

2010-02-08 Thread John Dennis
names, they should be updated to match the name of the function. If all you do is just edit the names I don't think you need to repost the patch, I'll ACK it now. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

[Freeipa-devel] [PATCH] add i18n translation support

2010-02-09 Thread John Dennis
figure out how to do it. I'll of course fix any build issue which might arise that I didn't catch earlier. Jason and I did discuss adding a test language to test translations in our test framework. This patch does not include that, it will follow. John -- John Dennis jden...@redhat.com Looking

[Freeipa-devel] [PATCH] add a test locale to our i18n support, used for testing translations

2010-02-09 Thread John Dennis
test_i18n.py. At some point in the future we need to add a real test for translations but that is dependent on Jason finishing the LazyText work. That test will follow the logic in test_i18n.py. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From

[Freeipa-devel] [PATCH] Pass target locale to msginit

2010-02-10 Thread John Dennis
in a fully translated .po instead of a fully untranslated .po. Add some comments to better explain some of the cryptic sed commands. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From 2f7b71b109699dd0b5aedb21631cc6839380b719 Mon Sep 17 00:00:00

[Freeipa-devel] [PATCH] Friendlier diagnostics for i18n test

2010-02-10 Thread John Dennis
The diagnostic messages in test_i18n.py were a bit confusing because it didn't indicate what the test string was and where it came from. Hopefully this is clearer now. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From

Re: [Freeipa-devel] [PATCH] Use unicode instead of str for environmental variables in Env.

2010-02-11 Thread John Dennis
encoding for everything (at least in our distribution). ACK -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo

Re: [Freeipa-devel] [PATCHES] Bring back old outputting functionality

2010-02-11 Thread John Dennis
in? -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] Calling for translation help

2010-02-11 Thread John Dennis
about Transifex, how to sign up and how to contribute please start with the Transifex main page: https://www.transifex.net Thank you for your help! The FreeIPA team. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

[Freeipa-devel] [PATCH] add Polish translation

2010-02-11 Thread John Dennis
Woo ho! We got our first translation, Polish. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From faa377d09b551b9b6f9ae8b6d5f1e8b919d0a872 Mon Sep 17 00:00:00 2001 From: John Dennis jden...@redhat.com Date: Thu, 11 Feb 2010 18:29:27 -0500 Subject

Re: [Freeipa-devel] [PATCH] add Polish translation

2010-02-11 Thread John Dennis
On 02/11/2010 06:36 PM, John Dennis wrote: Woo ho! We got our first translation, Polish. NAK Something got screwed up with the encoding when the patch was sent through email. I need to figure out what went wrong and resubmit it. -- John Dennis jden...@redhat.com Looking to carve out

[Freeipa-devel] So what's up with this new stream of email from Transifex on freeipa-devel?

2010-02-11 Thread John Dennis
will also appear on this list. For the time being I will handle all the emails coming from Transifex. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com

Re: [Freeipa-devel] [PATCH] 377 fix deprecation warning

2010-02-12 Thread John Dennis
modifications to a minimum. So I agree with your approach. ack once you fix the import. ;) FWIW, python-nss has: hash_buf (specify any hash algorithm supported my NSS) md5_digest sha1_digest sha256_digest sha512_digest -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com

[Freeipa-devel] git patch email issues

2010-02-12 Thread John Dennis
~/.gitconfig is set up like this: [sendemail] smtpserver = smtp.corp.redhat.com to = freeipa-devel@redhat.com from = John Dennis jden...@redhat.com confirm = never [format] headers = Content-Type: text/plain; charset=\utf-8\\nContent-Transfer-Encoding: 8bit\n

Re: [Freeipa-devel] [PATCH] 379 fix automountlocation-tofiles

2010-02-13 Thread John Dennis
On 02/12/2010 04:29 PM, Rob Crittenden wrote: The command automountlocation-tofiles hadn't been ported to the new return value format. It should work again. I also added a few labels to make the output more readable. Shouldn't the labels be localized? -- John Dennis jden...@redhat.com

Re: [Freeipa-devel] git patch email issues

2010-02-15 Thread John Dennis
On 02/15/2010 12:10 PM, Stephen Gallagher wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/12/2010 05:43 PM, John Dennis wrote: 1) Continue to send patches the way we have making sure Thunderbird is configured to base64 encode them. Accept the fact that when displayed in a mail

Re: [Freeipa-devel] [PATCH] 379 fix automountlocation-tofiles

2010-02-15 Thread John Dennis
On 02/15/2010 01:22 PM, Jason Gerard DeRose wrote: On Sat, 2010-02-13 at 08:55 -0500, John Dennis wrote: On 02/12/2010 04:29 PM, Rob Crittenden wrote: The command automountlocation-tofiles hadn't been ported to the new return value format. It should work again. I also added a few labels

Re: [Freeipa-devel] [PATCH] 379 fix automountlocation-tofiles

2010-02-15 Thread John Dennis
On 02/15/2010 01:33 PM, Jason Gerard DeRose wrote: On Mon, 2010-02-15 at 13:28 -0500, John Dennis wrote: On 02/15/2010 01:22 PM, Jason Gerard DeRose wrote: On Sat, 2010-02-13 at 08:55 -0500, John Dennis wrote: On 02/12/2010 04:29 PM, Rob Crittenden wrote: The command automountlocation

[Freeipa-devel] [PATCH] Add translation statistics

2010-02-16 Thread John Dennis
untranslated, 13 missing, 129 untranslated pl.po: 120/133 90.2%0 po untranslated, 13 missing, 13 untranslated Also update configure.ac to search for msgcmp, awk sed programs. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From

Re: [Freeipa-devel] [PATCH] Convert password policy integer values to unicode instead of str.

2010-02-17 Thread John Dennis
On 02/17/2010 10:25 AM, Pavel Zuna wrote: Password policy plugin was converting integers to str, which is unfriendly with our str==binary approach. Pavel ACK -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

Re: [Freeipa-devel] [PATCH] Convert integers and boolean values to unicode, don't leave them as str.

2010-02-17 Thread John Dennis
On 02/17/2010 09:56 AM, Pavel Zuna wrote: This solves the bug, where integer and boolean values are interpreted as binary. ACK -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing

[Freeipa-devel] [PATCH]

2010-02-19 Thread John Dennis
Add translation for Bengali India (bn_IN). -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From 8118b1b3268eda7aef5179d8408136e5b94d8d28 Mon Sep 17 00:00:00 2001 From: John Dennis jden...@redhat.com Date: Fri, 19 Feb 2010 08:42:50 -0500 Subject

[Freeipa-devel] [PATCH] strip .po file suffix in translation statistics

2010-02-19 Thread John Dennis
untranslated, 13 missing, 129 untranslated pl: 120/133 90.2%0 po untranslated, 13 missing, 13 untranslated -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From 2b25bb389c9f236947c98c6214858e8f88d7ff07 Mon Sep 17 00:00:00 2001 From: John

Re: [Freeipa-devel] [PATCH] 385 don't try to revoke revoked certs

2010-02-19 Thread John Dennis
to localize all labels. That's a 1 line change so it falls into the make the fix commit category. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https

[Freeipa-devel] [PATCH] Add new Russian translation, update Polish

2010-02-20 Thread John Dennis
missing, 13 untranslated -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From 258527df1230ca65c6350934a44ba8ba123fd358 Mon Sep 17 00:00:00 2001 From: John Dennis jden...@redhat.com Date: Sat, 20 Feb 2010 09:04:38 -0500 Subject: [PATCH] Add new

[Freeipa-devel] [PATCH] make update-pot will not touch pot file if msgid's are unmodified

2010-02-20 Thread John Dennis
if the pot file was current. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From 324734f811ef0a67a590282393b0ebb52927a35d Mon Sep 17 00:00:00 2001 From: John Dennis jden...@redhat.com Date: Sat, 20 Feb 2010 11:00:01 -0500 Subject: [PATCH] make update-pot

Re: [Freeipa-devel] [PATCH] make update-pot will not touch pot file if msgid's are unmodified

2010-02-20 Thread John Dennis
Resending modified patch. I discovered the original patch had a minor bug which might leave temporary files in the directory. Please use this patch instead. On 02/20/2010 11:15 AM, John Dennis wrote: Formerly running 'make update-pot' would write an insignificantly different pot file even

[Freeipa-devel] [PATCH] Add more Kannada translations

2010-02-21 Thread John Dennis
Add more Kannada translations -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From c42de1fc0708e2a99bdafe75b5222b1f772cd94a Mon Sep 17 00:00:00 2001 From: John Dennis jden...@redhat.com Date: Sun, 21 Feb 2010 16:00:10 -0500 Subject: [PATCH] Add more

[Freeipa-devel] [PATCH] minor makefile cleanup

2010-02-22 Thread John Dennis
. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From e957450602e58a026ebd4bb1ef33bfa6e0bc7bc9 Mon Sep 17 00:00:00 2001 From: John Dennis jden...@redhat.com Date: Mon, 22 Feb 2010 16:50:37 -0500 Subject: [PATCH] minor makefile cleanup Content-Type

[Freeipa-devel] [PATCH] Add more Spanish translations

2010-02-23 Thread John Dennis
-- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From fb5165f3253a75ebae073d0bd09aa7d7e6400c0d Mon Sep 17 00:00:00 2001 From: John Dennis jden...@redhat.com Date: Tue, 23 Feb 2010 11:11:27 -0500 Subject: [PATCH] Add more Spanish translations Content

Re: [Freeipa-devel] [PATCH] jderose 048 Translatable Param.label, Param.doc

2010-02-23 Thread John Dennis
this is completely independent of the label used for prompts and displaying a value. Also the text.FixMe class needs some documentation on how we plan on using it to find unstranslated strings. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

[Freeipa-devel] [PATCH] Add contributors file

2010-02-23 Thread John Dennis
Add contributors file. This gets installed along side the LICENSE and README files in the doc dir for each rpm package. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From 7e14f978f41f9d575511af469bdb5b7ca3509681 Mon Sep 17 00:00:00 2001 From

[Freeipa-devel] [PATCH] Add contributors file. (corrected)

2010-02-24 Thread John Dennis
Note: This is a corrected version, Stephen's name misspelled (also there were inconsistencies with the indentation which I fixed as well) This gets installed along side the LICENSE and README files in the doc dir for each rpm package. -- John Dennis jden...@redhat.com Looking to carve out

[Freeipa-devel] [PATCH] Install Contributors.txt

2010-02-24 Thread John Dennis
-- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From 9375fce583792b238a779a164e514443ed8c8eec Mon Sep 17 00:00:00 2001 From: John Dennis jden...@redhat.com Date: Wed, 24 Feb 2010 14:07:33 -0500 Subject: [PATCH] Install Contributors.txt Content-Type

Re: [Freeipa-devel] [PATCH] 391 make dogtag a required and default component

2010-02-24 Thread John Dennis
to --ca. rob ACK -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] [PATCH] Rework contributors list from Simo's input

2010-02-24 Thread John Dennis
O.K., let's try this one more time. The list has be modified according to Simo's suggestions. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From e3ed90875bdb2a856fbea9b4efebd2641d570258 Mon Sep 17 00:00:00 2001 From: John Dennis jden

Re: [Freeipa-devel] [PATCH] Code cleanup: remove unused stuff, take 1.

2010-02-25 Thread John Dennis
On 02/25/2010 08:23 AM, Pavel Zuna wrote: Removes most of the deprecated output code from plugins. take 1 because there might be more patches like this in the near future. Pavel ACK I love getting rid of cruft. Thanks! -- John Dennis jden...@redhat.com Looking to carve out IT costs

[Freeipa-devel] [PATCH] fixed the contributors patch

2010-02-25 Thread John Dennis
time around (which I did send a patch for separately). This should have everything all together and updated. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From efab2e92232f0d542b51a9122c625766ca18be2c Mon Sep 17 00:00:00 2001 From: John Dennis jden

[Freeipa-devel] [PATCH] full Spanish translations as of 20100225

2010-02-25 Thread John Dennis
kn: 20/133 15.0% 113 po untranslated,0 missing, 113 untranslated pl: 133/133 100.0%0 po untranslated,0 missing,0 untranslated ru: 120/133 90.2%0 po untranslated, 13 missing, 13 untranslated -- John Dennis jden...@redhat.com Looking to carve out

[Freeipa-devel] JSON problems (the woes of binary data)

2010-02-26 Thread John Dennis
of the robustness of allowing binary data at all in our implementation. Trying to support binary data has been nothing but a headache and a source of many many bugs. Do we really need it? -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts

[Freeipa-devel] [PATCH] update POT

2010-03-01 Thread John Dennis
This updates the POT file. This can be immediately committed via the .po files require no ACK rule. Explanation: the POT file is auto generated on demand, there is nothing to review. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From

[Freeipa-devel] [PATCH] documentation fix for json_decode_binary

2010-03-02 Thread John Dennis
This is a multi-part message in MIME format. --1.6.6.1 Content-Type: text/plain; charset=UTF-8; format=fixed Content-Transfer-Encoding: 8bit --- ipaserver/rpcserver.py |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) --1.6.6.1 Content-Type: text/x-patch;

[Freeipa-devel] [PATCH] 0003 Fix JSON binary encode and decode errors (rebase)

2010-03-02 Thread John Dennis
' replace that dict with the base64 decoded value of the key's value. -- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From d3627d87ff2b26b380a4a4d0a349ad2d432c1ca3 Mon Sep 17 00:00:00 2001 From: John Dennis jden...@redhat.com Date: Mon, 1 Mar 2010 18:55:39

Re: [Freeipa-devel] [PATCH] documentation fix for json_decode_binary

2010-03-03 Thread John Dennis
this: git format-patch -n -start-number 3 -1 FWIW, I have a script which automates this, it maintains the sequence number in a file in the same directory which all my patches are written to. It's also capable of emailing the patch. -- John Dennis jden...@redhat.com Looking to carve out

  1   2   3   4   5   >