Error in man sssd-ldap ?

[Marco Pizzoli]

Hi, I hope to be of help in asking/reporting this kind of things.

In man sssd-ldap:

- ldap_enumeration_refresh_timeout (integer)

The LDAP attribute that contains how many seconds SSSD has to wait before
refreshing its cache of enumerated records.

Default: 300

Probably this is not a LDAP attribute. Am I right?

---------------------------------

- ldap_purge_cache_timeout  <missing parameter type>

Marco

[Jakub Hrozek]

On Mon, Feb 06, 2012 at 05:08:02PM +0100, Marco Pizzoli wrote:
> Hi, I hope to be of help in asking/reporting this kind of things.
>
> In man sssd-ldap:
>
> ldap_enumeration_refresh_timeout (integer)
>   The LDAP attribute that contains how many seconds SSSD has to wait
>   before refreshing its cache of enumerated records.
>
>   Default: 300
>
> Probably this is not a LDAP attribute. Am I right?

You're right, it's not an attribute.
> ldap_purge_cache_timeout  <missing parameter type>

Yes, it's supposed to be saying "integer".
> Marco

Thank you for reporting these. Feel free to send git-formatted patches
for bugs you see in manual pages (or elsewhere).

[Marco Pizzoli]

Hi Jakub,

On Mon, Feb 6, 2012 at 5:25 PM, Jakub Hrozek  wrote:
> On Mon, Feb 06, 2012 at 05:08:02PM +0100, Marco Pizzoli wrote:
> > Hi, I hope to be of help in asking/reporting this kind of things.
> >
> > In man sssd-ldap:
> >
> > ldap_enumeration_refresh_timeout (integer)
> >   The LDAP attribute that contains how many seconds SSSD has to
>
> wait
> >        before refreshing its cache of enumerated records.
> >
> >        Default: 300
> >
> > Probably this is not a LDAP attribute. Am I right?
>
> You're right, it's not an attribute.
> > ldap_purge_cache_timeout  <missing parameter type>
>
> Yes, it's supposed to be saying "integer".
> > Marco
>
> Thank you for reporting these. Feel free to send git-formatted patches
> for bugs you see in manual pages (or elsewhere).

I would be happy to do my best, but I admit to be total ignorant of how to
git-format anything :-/
Please, could you point me to an easy tutorial on how to do this?

I see in your patch file this command:

diff --git a/src/man/sssd-ldap.5.xml b/src/man/sssd-ldap.5.xml

Are there other tasks I need to complete before or after?

Thanks
Marco

[Jakub Hrozek]

On Mon, Feb 06, 2012 at 06:46:45PM +0100, Marco Pizzoli wrote:
> Hi Jakub,
>
> On Mon, Feb 6, 2012 at 5:25 PM, Jakub Hrozek  wrote:
>  On Mon, Feb 06, 2012 at 05:08:02PM +0100, Marco Pizzoli wrote:
>  >    Hi, I hope to be of help in asking/reporting this kind of things.
>  >
>  >    In man sssd-ldap:
>  >
>  >    - ldap_enumeration_refresh_timeout (integer)
>  >
>  >            The LDAP attribute that contains how many seconds SSSD has
>  to wait
>  >            before refreshing its cache of enumerated records.
>  >
>  >            Default: 300
>  >
>  >    Probably this is not a LDAP attribute. Am I right?
>
>  You're right, it's not an attribute.
>  >
>  >    ---------------------------------
>  >
>  >    - ldap_purge_cache_timeout  <missing parameter type>
>
>  Yes, it's supposed to be saying "integer".
>
>  >
>  >    Marco
>
>  Thank you for reporting these. Feel free to send git-formatted patches
>  for bugs you see in manual pages (or elsewhere).
>
> I would be happy to do my best, but I admit to be total ignorant of how to
>    git-format anything :-/

No problem, git can be hard to learn.
> Please, could you point me to an easy tutorial on how to do this?
>
> I see in your patch file this command:

The routine would be:
    * edit the files that need editing
    * git add the changed files
    * git commit -m "Commit message"
    * git log/git log -p to review the commits
    * git format-patch -1 (or hash-id) to produce a git-formatted patch

I found this tutorial nice when I started with git:
http://schacon.github.com/git/everyday.html
> diff --git a/src/man/sssd-ldap.5.xml b/src/man/sssd-ldap.5.xml
>
> Are there other tasks I need to complete before or after?
>    Thanks
>    Marco

[Stephen Gallagher]

On Mon, 2012-02-06 at 20:13 +0100, Jakub Hrozek wrote:
> On Mon, Feb 06, 2012 at 06:46:45PM +0100, Marco Pizzoli wrote:
> > Hi Jakub,
> >
> > On Mon, Feb 6, 2012 at 5:25 PM, Jakub Hrozek  wrote:
> >  On Mon, Feb 06, 2012 at 05:08:02PM +0100, Marco Pizzoli wrote:
> >  >    Hi, I hope to be of help in asking/reporting this kind of things.
> >  >
> >  >    In man sssd-ldap:
> >  >
> >  >    - ldap_enumeration_refresh_timeout (integer)
> >  >
> >  >            The LDAP attribute that contains how many seconds SSSD has
> >  to wait
> >  >            before refreshing its cache of enumerated records.
> >  >
> >  >            Default: 300
> >  >
> >  >    Probably this is not a LDAP attribute. Am I right?
> >
> >  You're right, it's not an attribute.
> >  >
> >  >    ---------------------------------
> >  >
> >  >    - ldap_purge_cache_timeout  <missing parameter type>
> >
> >  Yes, it's supposed to be saying "integer".
> >
> >  >
> >  >    Marco
> >
> >  Thank you for reporting these. Feel free to send git-formatted patches
> >  for bugs you see in manual pages (or elsewhere).
> >
> > I would be happy to do my best, but I admit to be total ignorant of how to
> >    git-format anything :-/
>
> No problem, git can be hard to learn.
> > Please, could you point me to an easy tutorial on how to do this?
> >
> > I see in your patch file this command:
>
> The routine would be:
>     * edit the files that need editing
>     * git add the changed files
>     * git commit -m "Commit message"
>     * git log/git log -p to review the commits
>     * git format-patch -1 (or hash-id) to produce a git-formatted patch
>
> I found this tutorial nice when I started with git:
> http://schacon.github.com/git/everyday.html

It's also worth mentioning that you would want to clone the repository
first with 
git clone git://git.fedorahosted.org/git/sssd.git

[Simo Sorce]

On Mon, 2012-02-06 at 14:15 -0500, Stephen Gallagher wrote:
> > No problem, git can be hard to learn.
> > > Please, could you point me to an easy tutorial on how to do
>
> this?
> > > I see in your patch file this command:
> >
> > The routine would be:
> >     * edit the files that need editing
> >     * git add the changed files
> >     * git commit -m "Commit message"
> >     * git log/git log -p to review the commits
> >     * git format-patch -1 (or hash-id) to produce a git-formatted
>
> patch
> > I found this tutorial nice when I started with git:
> > http://schacon.github.com/git/everyday.html
>
> It's also worth mentioning that you would want to clone the repository
> first with 
> git clone git://git.fedorahosted.org/git/sssd.git

I would also like to add that although we *strongly* prefer to receive
git formatted patches, that we accept also normal unidiff patches, if
you do not know git well enough and the patch is relatively simple.

But we strongly prefer proper git patches :)

Simo.