332954 - Update Win32 installation instructions about LDAP and Registry Key

Status: NEW

(Bugzilla :: Documentation, defect)

Description

[mige harimurti]

Change 'install Net::LDAP' to 'install Net-LDAP-Express'
Change 'HKEY_CLASSES_ROOT\.cgi\Shell\ExecCGI\Command' to 'HKEY_CLASSES_ROOT\.cgi\shell\execcgi\command'
Change 'C:\Perl\bin\perl.exe -T' to '"C:\Perl\bin\perl.exe" -T "%1"'

Comment 1

[mige harimurti]

Attachment: Change Install LDAP and Registry Key/Value

Comment 2

[Vlad Dascalu]

Comment on attachment 217393 [details] [diff] [review]
Change Install LDAP and Registry Key/Value

Hey Mige, can you update this patch?

-ppm&gt; <span class="in">install Net::LDAP</span>
+ppm&gt; <span class="in">install Net-LDAP-Express</span>

You're changing this reference, but you have a previous reference of Net::LDAP that you've left untouched. It's good to be consistent; if Net::LDAP has a different name in Windows, it's good to replace all its occurences.

+Create a key <b>HKEY_CLASSES_ROOT\.cgi\shell\execcgi\command</b> with the default value of the full path of perl.exe with a -T parameter.
+For example <b>"C:\Perl\bin\perl.exe" -T "%1"</b>

The second line is welcomed, since the current example probably fails if you install Bugzilla in a directory that contains spaces. Thanks for catching that!

However I don't understand why you changed capitalization. "ExecCGI" instead of "execcgi" is pretty standard in Windows Registry, and I'd like to keep that one.


It would be cool if you could reupload a second version, for checking it in (don't forget to ask review on it to keep it on the radar)

Comment 3

[Vlad Dascalu]

CCing glog since he wrote originally this.

Comment 4

[:glob ✱]

> +Create a key <b>HKEY_CLASSES_ROOT\.cgi\shell\execcgi\command</b> with the
> default value of the full path of perl.exe with a -T parameter.
> +For example <b>"C:\Perl\bin\perl.exe" -T "%1"</b>
> 
> The second line is welcomed, since the current example probably fails if you
> install Bugzilla in a directory that contains spaces.

the guide already says:

"Note: Installing Perl into a directory that contains a space (eg "C:\Program Files") will break the Template-Toolkit installer."

so you'll encounter other issues if you install perl into a directory with spaces.

that said, adding the quotes is a good idea.

you should also update the regedit screenshot to match -- http://www.bugzilla.org/img/regedit.png

> However I don't understand why you changed capitalization.

i agree -- for Shell and ExecCGI please.

Comment 5

[Frédéric Buclin]

gerv: any reason why https://wiki.mozilla.org/Bugzilla:Win32Install isn't merged with http://bugzilla.readthedocs.org/en/latest/installing/windows.html?

Comment 6

[Gervase Markham [:gerv]]

LpSolit: "redirect" added. I've also been through and done similar things for a load of other obsolete or semi-obsolete wiki pages where the canonical information is now in the real docs.

Gerv