347453 - Checksetup needs Perl-LDAP for Net::LDAP module on Windows

Status: RESOLVED FIXED

(Bugzilla :: Installation & Upgrading, defect)

Description

[A. Karl Kornel]

Checksetup looks for the Net::LDAP module as an optional module for those who wish to use LDAP-based authentication.  When it is unable to find the module, it asks the user to run `ppm install Net-LDAP` to get the module.  However, the user should be running `ppm install Perl-LDAP`, which gets Net::LDAP and some additional modules, to satisfy checksetup.  I was able to confirm this by running `ppm install Perl-LDAP`; during the next run of checksetup I was no longer asked for the Net::LDAP module.

Comment 1

[Frédéric Buclin]

If this problem also exists in 2.22, it should block 2.22.1. Being unable to log in because one module cannot be installed using the information displayed makes the software pretty useless.

Comment 2

[bigstijn]

I can't simulate this on my installation: Bugzilla 2.22 .
Checksetup doesn't mention anything about Net::LDAP as optional module, and I don't have Net-LDAP-express or perl-ldap installed. 


(I don't use LDAP authentication, but that shouldn't bother checksetup)

Comment 3

[bigstijn]

Attachment: output of PPM test for modules

(see comment 2)

Comment 4

[Max Kanat-Alexander]

(In reply to comment #2)
> (I don't use LDAP authentication, but that shouldn't bother checksetup)

  Oh 2.22, it does matter that you don't use LDAP auth, since 2.22 checksetup doesn't check for it otherwise.

Comment 5

[bigstijn]

Comment on attachment 237822 [details]
output of PPM test for modules

(obsolete: see comment 4)

Comment 6

[Max Kanat-Alexander]

Attachment: v1 - tip

Granting myself review as module owner.

Comment 7

[Max Kanat-Alexander]

This isn't a regression, so I'm not going to backport it. 2.22 didn't print out an installation command at all, for Net::LDAP.

Comment 8

[Dave Miller [:justdave]]

this probably won't affect the qa tests much

Comment 9

[Max Kanat-Alexander]

Checking in Bugzilla/Install/Requirements.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/Requirements.pm,v  <--  Requirements.pm
new revision: 1.20; previous revision: 1.19
done