Re: [SSSD] [PATCH] sss_groupshow - a utility to print properties of a local group

2010-01-20 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/19/2010 08:28 AM, Sumit Bose wrote: On Tue, Jan 19, 2010 at 01:47:13PM +0100, Jakub Hrozek wrote: ouch, sorry, fixed.. ACK bye, Sumit Pushed to master. - -- Stephen Gallagher RHCE 804006346421761 Delivering value year after year.

Re: [SSSD] [PATCH] sss_groupshow - a utility to print properties of a local group

2010-01-19 Thread Jakub Hrozek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/18/2010 05:28 PM, Sumit Bose wrote: On Fri, Jan 15, 2010 at 06:55:29PM +0100, Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/15/2010 06:24 PM, Jakub Hrozek wrote: On 01/15/2010 06:22 PM, Jakub Hrozek wrote: This

Re: [SSSD] [PATCH] sss_groupshow - a utility to print properties of a local group

2010-01-19 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/19/2010 06:14 AM, Jakub Hrozek wrote: On 01/18/2010 05:28 PM, Sumit Bose wrote: Works nice, but can you add _() to the default output so it can get translated? bye, Sumit Sure, thanks for the review. New patch is attached. Nack.

Re: [SSSD] [PATCH] sss_groupshow - a utility to print properties of a local group

2010-01-19 Thread Jakub Hrozek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/19/2010 01:33 PM, Stephen Gallagher wrote: On 01/19/2010 06:14 AM, Jakub Hrozek wrote: On 01/18/2010 05:28 PM, Sumit Bose wrote: Works nice, but can you add _() to the default output so it can get translated? bye, Sumit Sure, thanks

Re: [SSSD] [PATCH] sss_groupshow - a utility to print properties of a local group

2010-01-19 Thread Sumit Bose
On Tue, Jan 19, 2010 at 01:47:13PM +0100, Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/19/2010 01:33 PM, Stephen Gallagher wrote: On 01/19/2010 06:14 AM, Jakub Hrozek wrote: On 01/18/2010 05:28 PM, Sumit Bose wrote: Works nice, but can you add _() to the

Re: [SSSD] [PATCH] sss_groupshow - a utility to print properties of a local group

2010-01-18 Thread Sumit Bose
On Fri, Jan 15, 2010 at 06:55:29PM +0100, Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/15/2010 06:24 PM, Jakub Hrozek wrote: On 01/15/2010 06:22 PM, Jakub Hrozek wrote: This patch adds a utility called sss_groupshow that allows user to print properties of a

Re: [SSSD] [PATCH] sss_groupshow - a utility to print properties of a local group

2010-01-18 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/18/2010 11:28 AM, Sumit Bose wrote: On Fri, Jan 15, 2010 at 06:55:29PM +0100, Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/15/2010 06:24 PM, Jakub Hrozek wrote: On 01/15/2010 06:22 PM, Jakub Hrozek wrote: This

[SSSD] [PATCH] sss_groupshow - a utility to print properties of a local group

2010-01-15 Thread Jakub Hrozek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 This patch adds a utility called sss_groupshow that allows user to print properties of a group in the local domain. Fixes: #306 -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.10 (GNU/Linux) Comment: Using GnuPG with Fedora -

Re: [SSSD] [PATCH] sss_groupshow - a utility to print properties of a local group

2010-01-15 Thread Jakub Hrozek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/15/2010 06:22 PM, Jakub Hrozek wrote: This patch adds a utility called sss_groupshow that allows user to print properties of a group in the local domain. Fixes: #306 Self-nack, I realized it is missing changes to specfile, sorry for the

Re: [SSSD] [PATCH] sss_groupshow - a utility to print properties of a local group

2010-01-15 Thread Jakub Hrozek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/15/2010 06:24 PM, Jakub Hrozek wrote: On 01/15/2010 06:22 PM, Jakub Hrozek wrote: This patch adds a utility called sss_groupshow that allows user to print properties of a group in the local domain. Fixes: #306 Self-nack, I realized it