Closed
Bug 399823
Opened 17 years ago
Closed 17 years ago
nonroot installation instructions tell you to modify files that don't exist
Categories
(Bugzilla :: Documentation, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.2
People
(Reporter: justdave, Assigned: sam.folkwilliams)
References
()
Details
Attachments
(1 file)
8.27 KB,
patch
|
cso
:
review+
|
Details | Diff | Splinter Review |
-----8<----- If you had to install Perl modules as a non-root user (Section 2.6.4) or to non-standard directories, you will need to change the scripts, setting the correct location of the Perl modules: perl -pi -e 's@use strict\;@use strict\; use lib \"/home/foo/perl/lib\"\;@' *cgi *pl Bug.pm processmail syncshadowdb -----8<----- a) Bug.pm, processmail, and syncshadowdb no longer exist, and haven't, in a few versions. b) If you copy/paste that code directly, it won't work without a \ added on the end of the first two lines.
Assignee | ||
Comment 1•17 years ago
|
||
Dave - do you know what files *do* need to be modified, if any? Or should we just take this out?
Flags: needsinfo?(justdave)
Reporter | ||
Comment 2•17 years ago
|
||
pm files don't need to be touched, and there are no executable files left that don't have suffixes. It should be safe to just do .cgi and .pl files. And I'd include the dot. perl -pi -e \ 's@use strict\;@use strict\; use lib \"/home/foo/perl/lib\"\;@' *.cgi *.pl
Flags: needsinfo?(justdave)
Reporter | ||
Comment 3•17 years ago
|
||
14:31:43 <@LpSolit> errr... justdave, wait 14:31:50 <@LpSolit> we now have ./lib/ 14:31:59 <@LpSolit> all scripts look into it 14:32:03 <@LpSolit> no need to change anything 14:32:39 <@justdave> ah, just have to tell cpan to install there then 14:32:57 <@justdave> or use that installer thingie mkanat cooked up 14:33:21 <@LpSolit> justdave: bug 399954 14:33:21 <+bugbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=399954 Installation & Upgrading, enhancement, --, Bugzilla 3.2, mkanat@bugzilla.org, RESOLVED FIXED, Make Bugzilla able to hold its dependencies in a local directory
Assignee | ||
Comment 4•17 years ago
|
||
max - with the module-install.pl script, does that basically replace the non-root perl installation process described here: http://www.bugzilla.org/docs/tip/html/nonroot.html#install-perlmodules-nonroot Or do we still need to tell CPAN where to install the modules or install a separate perl in $HOME?
Assignee | ||
Updated•17 years ago
|
Status: NEW → ASSIGNED
Comment 5•17 years ago
|
||
It replaces those instructions entirely and nobody should ever have to do a manual nonroot CPAN installation anymore.
Flags: needsinfo?(mkanat) → needsinfo+
Comment 6•17 years ago
|
||
install-module.pl only exists in 3.2, so I don't think we want to update the doc in older versions. Dave, as you reported the bug, is that fine for you?
Target Milestone: --- → Bugzilla 3.2
Assignee | ||
Comment 7•17 years ago
|
||
This might be too drastic - took out the cpan stuff and simplified this down to running the script (and providing link to its documentation). Also fixed some formatting problems. Thoughts?
Attachment #298689 -
Flags: review?(documentation)
Comment 8•17 years ago
|
||
I don't think there is, but someone else might thing there is merit in the 'install your own personal version of perl' bit... If there isn't, then it's fine with me as is, if there is then it needs that reinstating.
Comment 9•17 years ago
|
||
I'm sure there's documentation elsewhere on the Internet about how to install your own version of Perl--we don't have to duplicate that information here.
Comment 10•17 years ago
|
||
Comment on attachment 298689 [details] [diff] [review] draft 1 r=me (inspection) based on my previous comment and MAx.
Attachment #298689 -
Flags: review?(documentation) → review+
Comment 11•17 years ago
|
||
No answer from justdave about my question in comment 6. So considering this bug as fixed: Checking in docs/xml/installation.xml; /cvsroot/mozilla/webtools/bugzilla/docs/xml/installation.xml,v <-- installation.xml new revision: 1.151; previous revision: 1.150 done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•