Bug #79045 for RT-Extension-LDAPImport: Documentation consistency

Fri Aug 17 09:24:44 2012 bpeccatte [...] dictao.com - Ticket created

Subject:	Documentation consistency
Date:	Fri, 17 Aug 2012 13:24:27 +0000
To:	"bug-RT-Extension-LDAPImport [...] rt.cpan.org" <bug-RT-Extension-LDAPImport [...] rt.cpan.org>
From:	Benoît Peccatte <bpeccatte [...] dictao.com>

The documentation indicate thqt you must import the plugin, but the plugin is only used from rtldapimport. So these lines are not necessary : # Add to any existing plugins Set(@Plugins, qw(RT::Extension::LDAPImport));

Fri Aug 17 11:40:31 2012 tsibley [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #79045] Documentation consistency
Date:	Fri, 17 Aug 2012 08:40:13 -0700
To:	bug-RT-Extension-LDAPImport [...] rt.cpan.org
From:	Thomas Sibley <tsibley [...] cpan.org>

On 08/17/2012 06:24 AM, Benoît Peccatte via RT wrote: Show quoted text

> The documentation indicate thqt you must import the plugin, > > but the plugin is only used from rtldapimport. > > So these lines are not necessary : > > # Add to any existing plugins > Set(@Plugins, qw(RT::Extension::LDAPImport));

Without being in @Plugins, RT won't set lib paths correctly. It is also useful to see if LDAPImport is used from the System Configuration page.

Fri Aug 17 11:40:32 2012 The RT System itself - Status changed from 'new' to 'open'

Fri Aug 17 11:47:17 2012 bpeccatte [...] dictao.com - Correspondence added

Subject:	RE: [rt.cpan.org #79045] Documentation consistency
Date:	Fri, 17 Aug 2012 15:46:58 +0000
To:	"bug-RT-Extension-LDAPImport [...] rt.cpan.org" <bug-RT-Extension-LDAPImport [...] rt.cpan.org>
From:	Benoît Peccatte <bpeccatte [...] dictao.com>

Show quoted text

> From : Thomas Sibley via RT [mailto:bug-RT-Extension- > LDAPImport@rt.cpan.org] > > <URL: https://rt.cpan.org/Ticket/Display.html?id=79045 > > > On 08/17/2012 06:24 AM, Benoît Peccatte via RT wrote:

> > The documentation indicate thqt you must import the plugin, > > > > but the plugin is only used from rtldapimport. > > > > So these lines are not necessary : > > > > # Add to any existing plugins > > Set(@Plugins, qw(RT::Extension::LDAPImport));

> > Without being in @Plugins, RT won't set lib paths correctly. It is also useful to > see if LDAPImport is used from the System Configuration page. >

RT doesn't need lib paths since it doesn't use it. rtldapimport script does and does include the lib path in the binary.

Fri Aug 17 11:51:37 2012 tsibley [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #79045] Documentation consistency
Date:	Fri, 17 Aug 2012 08:51:04 -0700
To:	bug-RT-Extension-LDAPImport [...] rt.cpan.org
From:	Thomas Sibley <tsibley [...] cpan.org>

On 08/17/2012 08:47 AM, Benoît Peccatte via RT wrote: Show quoted text

> RT doesn't need lib paths since it doesn't use it. > rtldapimport script does and does include the lib path in the binary.

Where does rtldapimport include the lib path to RT::Extension::LDAPImport?

Fri Aug 17 11:55:55 2012 bpeccatte [...] dictao.com - Correspondence added

Subject:	RE: [rt.cpan.org #79045] Documentation consistency
Date:	Fri, 17 Aug 2012 15:55:39 +0000
To:	"bug-RT-Extension-LDAPImport [...] rt.cpan.org" <bug-RT-Extension-LDAPImport [...] rt.cpan.org>
From:	Benoît Peccatte <bpeccatte [...] dictao.com>

Show quoted text

> From: Thomas Sibley via RT [mailto:bug-RT-Extension- > LDAPImport@rt.cpan.org] > > <URL: https://rt.cpan.org/Ticket/Display.html?id=79045 > > > On 08/17/2012 08:47 AM, Benoît Peccatte via RT wrote:

> > RT doesn't need lib paths since it doesn't use it. > > rtldapimport script does and does include the lib path in the binary.

> > Where does rtldapimport include the lib path to RT::Extension::LDAPImport? >

I don't know where it comes from. Mine contains : use lib qw(/usr/local/share/request-tracker4/lib /usr/share/request-tracker4/lib /usr/local/share/request-tracker4/plugins/RT-Extension-DictaoLDAPImport/lib); It seems to be generated from : ### after: use lib qw(@RT_LIB_PATH@); use lib '/opt/rt3/local/lib /opt/rt3/lib';

Fri Aug 17 12:08:03 2012 tsibley [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #79045] Documentation consistency
Date:	Fri, 17 Aug 2012 09:07:32 -0700
To:	bug-RT-Extension-LDAPImport [...] rt.cpan.org
From:	Thomas Sibley <tsibley [...] cpan.org>

On 08/17/2012 08:55 AM, Benoît Peccatte via RT wrote: Show quoted text

> I don't know where it comes from. Mine contains : > use lib qw(/usr/local/share/request-tracker4/lib /usr/share/request-tracker4/lib /usr/local/share/request-tracker4/plugins/RT-Extension-DictaoLDAPImport/lib);

That last lib path was added manually by someone in your install. It is not generated normally. Show quoted text

> It seems to be generated from : > ### after: use lib qw(@RT_LIB_PATH@); > use lib '/opt/rt3/local/lib /opt/rt3/lib';

@RT_LIB_PATH@ comes from Makefile.PL, and doesn't include a plugin lib path.

Fri Aug 17 13:02:28 2012 bpeccatte [...] dictao.com - Correspondence added

Subject:	RE: [rt.cpan.org #79045] Documentation consistency
Date:	Fri, 17 Aug 2012 17:02:04 +0000
To:	"bug-RT-Extension-LDAPImport [...] rt.cpan.org" <bug-RT-Extension-LDAPImport [...] rt.cpan.org>
From:	Benoît Peccatte <bpeccatte [...] dictao.com>

Show quoted text

> From: Thomas Sibley via RT [mailto:bug-RT-Extension- > LDAPImport@rt.cpan.org] > > <URL: https://rt.cpan.org/Ticket/Display.html?id=79045 > > > On 08/17/2012 08:55 AM, Benoît Peccatte via RT wrote:

> > I don't know where it comes from. Mine contains : > > use lib qw(/usr/local/share/request-tracker4/lib > > /usr/share/request-tracker4/lib > > /usr/local/share/request-tracker4/plugins/RT-Extension-DictaoLDAPImpor > > t/lib);

> > That last lib path was added manually by someone in your install. It is not > generated normally.

I added the "Dictao" word because I made a patch, but I thought the remaining part was already there. But the path may have been added before. Sorry for the inconvenience.

Fri Aug 17 13:07:39 2012 tsibley [...] cpan.org - Status changed from 'open' to 'rejected'