Closed
Bug 423963
Opened 16 years ago
Closed 15 years ago
OS-specific installation notes should be clearer in how to use ScriptInterpreterSource
Categories
(Bugzilla :: Documentation, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.0
People
(Reporter: Wurblzap, Assigned: LpSolit)
References
Details
Attachments
(1 file, 2 obsolete files)
2.94 KB,
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
The OS-specific installation notes mention ScriptInterpreterSource and link to the Apache documentation, but that's rather generic and can be clearer and specific to Bugzilla. Patch along the lines as the change proposed by robbe on IRC.
Attachment #310575 -
Flags: review?(documentation)
The value "Registry-Strict" is apache 2.x minimum. In apache 1.x this would be "Registry", which has a lot of security implications, as described in the apache doc. I'm not sure to what extent apache 1.x users should be considerated and how much security warnings there should be. (this is user robbe on irc, after pondering this again)
Comment 2•16 years ago
|
||
Don't worry about Apache 1.x users on Windows. By the time that Bugzilla supported Windows, Apache 2.x was the standard and 1.x was quite old.
Comment 3•16 years ago
|
||
I've just read the patch, and I'm confused... The <para></para> block before this one tells me to set it to something involving the path to Perl, then this one tells me to go registry hacking and set it to something different. I think this needs to be more consistent?
@Colin: Right. The para-block before is a reduced description. Fact is: you do not set the "ScriptInterpreterSource" directly to the path to Perl. Instead, there are exactly three values it can be set to: "Registry", "Registry-Strict" and "Script". If you set it to values "Registry" or "Registry-Strict", it will look up the path to Perl in the registry. This is a bit confusing, but since it is covered by the indicated apache doc, there is no need to explain that in bugzilla doc. Furthermore I'm not a native speaker, so it's difficult for me to express complex facts in an accurate way with few words. After all I would propose to change both paragraphs to something like the following. Please note that it is importent to add the tags in the first paragraph again, which I omitted. ------- Note If using Apache on windows, you can set the ScriptInterpreterSource directive in your Apache config to avoid having to modify the first line of every script to contain your path to Perl instead of /usr/bin/perl. Do not forget to specify the -T flag to enable the taint mode. To use the <programlisting>ScriptInterpreterSource</programlisting> directive with bugzilla, insert the line <programlisting>ScriptInterpreterSource Registry-Strict</programlisting> into your <filename>httpd.conf</filename> file, and create the key <option>HKEY_CLASSES_ROOT\.cgi\Shell\ExecCGI\Command</option> with the value <option>C:\Perl\bin\perl.exe -T</option> (adapt to your path if needed) in the registry. Be sure to know about the security implications as described in the apache doc. -------
Assignee | ||
Comment 5•16 years ago
|
||
Comment on attachment 310575 [details] [diff] [review] Patch I like Robbe's suggestion in comment 4 better. It's confusing and useless to repeat things twice. The first paragraph should be a quick note, then you immediately describe in details how to use ScriptInterpreterSource.
Attachment #310575 -
Flags: review?(documentation) → review-
Assignee | ||
Comment 7•16 years ago
|
||
Marc, are you still working on it?
Reporter | ||
Comment 8•16 years ago
|
||
Not atm. Not being a native speaker, I don't feel comfortable with any current options... It may be I'll pick it up again some time, but for now, I bail.
Assignee: wurblzap → documentation
Status: ASSIGNED → NEW
Assignee | ||
Comment 9•15 years ago
|
||
Assignee: documentation → LpSolit
Attachment #310575 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #391188 -
Flags: review?(wurblzap)
Assignee | ||
Comment 10•15 years ago
|
||
Note that the removal of the "win32-code-changes" section as part of this patch is intentional as we no longer support Bugzilla 2.x.
Reporter | ||
Comment 11•15 years ago
|
||
Comment on attachment 391188 [details] [diff] [review] patch, v2 Mostly all right, r=Wurblzap on an updated patch: >Index: docs/en/xml/installation.xml >@@ -1687,14 +1685,16 @@ C:\perl> <command>ppm install <mod >+ Bugzilla looks at <filename>/usr/bin/perl</filename> to It's not Bugzilla, it's the web server. I think that's an important distinction, because if it goes wrong, it's the web server's fault, not Bugzilla's :)
Attachment #391188 -
Flags: review?(wurblzap) → review+
Assignee | ||
Comment 12•15 years ago
|
||
s/Bugzilla looks at/The web server looks at/. Carrying forward Marc's r+.
Attachment #391188 -
Attachment is obsolete: true
Attachment #391314 -
Flags: review+
Assignee | ||
Comment 13•15 years ago
|
||
tip: Checking in docs/en/xml/installation.xml; /cvsroot/mozilla/webtools/bugzilla/docs/en/xml/installation.xml,v <-- installation.xml new revision: 1.168; previous revision: 1.167 done 3.4: Checking in docs/en/xml/installation.xml; /cvsroot/mozilla/webtools/bugzilla/docs/en/xml/installation.xml,v <-- installation.xml new revision: 1.165.2.3; previous revision: 1.165.2.2 done 3.2.4: Checking in docs/en/xml/installation.xml; /cvsroot/mozilla/webtools/bugzilla/docs/en/xml/installation.xml,v <-- installation.xml new revision: 1.157.2.10; previous revision: 1.157.2.9 done 3.0.8: Checking in docs/xml/installation.xml; /cvsroot/mozilla/webtools/bugzilla/docs/xml/Attic/installation.xml,v <-- installation.xml new revision: 1.136.2.18; previous revision: 1.136.2.17 done
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•