1. Explanations does not consider IIS for file structure and permissions (chmod o+w ...)
2. My beta2 install, like the previous beta1, creates the \sites\default\files and \sites\default\files\styles directories, but not the \sites\default\private directory, although it writes so in the the document.
3. Under "5. CONFIGURE DRUPAL" section, "setting the $base_url variable" solution does not work for "Page Not Found" errors. See also "Page not found" after new install and other broken links on IIS 5.1 and its comments.
(The anomalies at 2. and 3. are observed during D7beta2 installation with IIS 5.x + PHP 5.x + MySQL 5.1.x.)
Comment | File | Size | Author |
---|---|---|---|
#31 | INSTALL.txt | 17.38 KB | jhodgdon |
#31 | 952606-31.patch | 29.59 KB | jhodgdon |
#22 | INSTALL.txt | 17.45 KB | jhodgdon |
#22 | 952606-22.patch | 29.6 KB | jhodgdon |
#19 | INSTALL.txt | 17.51 KB | jhodgdon |
Comments
Comment #1
arianek CreditAttribution: arianek commentedmoving to the Core queue
Comment #2
jhodgdonThanks for reporting this.
RE #1 - In the UPGRADE.txt file, we are changing the wording so that rather than saying you need to run certain commands, it says what the objective is, like "Change file permissions, and if you're on Unix/Linux you can use this command" rather than "Run this command". We need to do that in INSTALL.txt as well.
RE #2 - I think that section is just plain wrong. On Apache, the sites/default/files directory is created, but no sites/default/private, at installation (at least with the Minimal install profile -- maybe Standard creates sites/default/private?) The INSTALL.txt shouldn't mention sites/default/private at all, in my opinion.
RE #3 - We should link to this Handbook page instead of giving the $base_url instruction -- there are many other reasons that clean URLs might not work, and this page lists several possibilities and has sub-pages that tell how to configure clean URLs on other systems:
http://drupal.org/getting-started/clean-urls
Comment #3
jhodgdonOh. And while we're at it, the file needs to be line-wrapped at 80-characters. Currently several lines go over that width.
Comment #4
claar CreditAttribution: claar commentedFrom INSTALL.txt:
The Beta 2 release announcement says Drupal 7 requires PHP 5.2.4 now.
Looks like INSTALL.txt could use some general TLC.
Comment #5
jhodgdonI started editing this... claar has volunteered to make the next update. This is NOT DONE. I got through step #2 (and had added a step, so the numbering is not correct any more). Just attaching it here so claar can take over (THANKS!!!)
Comment #6
claar CreditAttribution: claar commentedI did *not* address comment 2's point #3 (link to handbook instead of $base_url fix). Out of time for today.
Comment #7
jhodgdonI don't think we want all UPPERCASE headings for the INDIVIDUAL STEPS OF THE PROCESS. Uppercase is too much like shouting. The other changes sound good. I'll review this tomorrow (can't face it today sorry).
Comment #8
EvanDonovan CreditAttribution: EvanDonovan commentedSubscribing.
Comment #9
claar CreditAttribution: claar commentedWell, upper-casing or not, I'd vote for leaving in a concise "step title heading" if possible, rather than each step simply being a wall-of-text.
Comment #10
Sahin CreditAttribution: Sahin commented@claar, the text seems better now, thanks.
But all 3 points mentioned in the definition of this issue are still relevant.
Comment #11
claar CreditAttribution: claar commentedYes, Sahin -- sorry, the general cleanup kind of sidestepped the issue a bit. I left your points #2 and #3 to jhodgdon, who is better suited to address them. Your point #1, however:
is addressed in comment #2 above in "RE #1" -- the current plan is to reword INSTALL.txt to use more objective-oriented language, but leave the Unix command examples in, such as:
Since including commands specific to multiple operating systems would make the instructions hard to follow, I agree with jhodgen's recommendation there. I began that work, but I believe there's more to be done.
Heine said on IRC that installation on IIS should be very similar to an Apache config, so hopefully rewording like above will be sufficient for IIS users to follow along. However, if there were a good "Configuring Drupal on IIS" or "Configuring Drupal on non-Apache web servers" document on drupal.org, I'd make a case for linking to it from INSTALL.txt.
Comment #12
EvanDonovan CreditAttribution: EvanDonovan commentedI agree that we should not provide the steps for all possible systems - Unix/Linux is a good default. This is how it's handled in UPGRADE.txt also.
Will review more later.
Comment #13
jhodgdonI'll take a stab at updating this to a viable patch.
Comment #14
jhodgdonHere's a patch.
Regarding the original issue report:
1) Hopefully, everything that is Unix/Linux command line has been labeled as an "example command" to achieve some goal, rather than as the only way to do things.
2) The stuff about the private file directory has been removed, since this hasn't been created in the current Drupal 7 installer.
3) That separate issue will need to address what to do to fix those errors, and hopefully document them in the Handbook in an appropriate location, or better yet fix Drupal so they don't occur (or make another INSTALL.txt patch). Right now, I don't think the solution is determined? Anyway, I took that bit about $base_url out of the INSTALL.txt file and put it in the Handbook instead, since that is far from the only problem people might encounter.
Comment #15
jhodgdonI'm also upping this to major, since this file had numerous innacurracies in it, and I sincerely hope we don't ship Drupal 7's release with the file as it currently is.
Comment #16
claar CreditAttribution: claar commentedReally, really nice work, jhodgdon. Several things you fixed definitely bump this to major. My review:
A bit awkward. Perhaps change to:
Thoughts?
This paragraph is gold -- nice addition.
I like having this as its own step -- nice. "writeable" misspelled, though!
multi-site -> multisite please :)
This confused me at first. Perhaps change to "After installation, you may modify the file system path to store uploaded files in a different location." ?
I suggest combining these into:
Thoughts?
Suggest "8. Revoke documentation file permissions."
"visitors visiting" -- a bit of a funny phrase, but can't come up with better. Perhaps "those visiting" ?
should be cron_key=YOURKEY
Powered by Dreditor.
Comment #17
claar CreditAttribution: claar commentedForgot to set status back to needs work. Setting to "bug report" due to the major inaccuracies in the old version.
Comment #18
jhodgdonThanks! I think all the comments in #16 are on target. I'll make another patch sometime soon, unless someone beats me to it.
And by the way, that "Take note of the username..." paragraph was in the document before I edited it. I think I just moved it to its own paragraph, and perhaps reworded it. :)
Comment #19
jhodgdonOK, here's a new version addressing #16.
Comment #20
jhodgdonComment #21
EvanDonovan CreditAttribution: EvanDonovan commentedThis looks like a great improvement. Here's a few things I noted.
1)
These aren't tasks, or at least aren't worded as tasks. They're actually module-specific requirements, but I'm not sure of a good wording for that.
2) The Blogger API, etc. are not used very often, especially when compared to the others. They should probably be last in the list.
3) The description of the modules that require outbound network connections is a bit technical and quite verbose. At the least, it should be broken into two paragraphs.
4) In step 1 of the installation process, this sentence doesn't state that it's a sample command. I don't know if that's necessary.
5) In step 3, the first sentence in the second paragraph has "site" and "all" reversed:
6) The second part of that paragraph should probably be a second paragraph, and could read as follows:
Drupal ships with the following files describing this process for your database engine: INSTALL.mysql.txt, INSTALL.pgsql.txt, and INSTALL.sqlite.txt. For further assistance, contact your web hosting provider.
I think the file names make it sufficiently clear which is for which.
7) In step 4, the following should be corrected:
(see the Multisite section of this file below).
to
(see the "Multisite Configuration" section, below).
8) In step 9, do we really want to recommend automatic cron to most Drupal users? If people have feed aggregator modules enabled, or XML sitemap, etc., it will be quite slow. I think we should still recommend setting up cron jobs as a best practice.
9) In the themes section, the following is incorrect:
a theme changes the front-end look and behavior of your site.
The word "front-end" should probably be removed, or else "or administration" be added, since some themes, like Seven, are not for front-end use.
Also, it doesn't seem precisely accurate to say that themes changes behavior, although some that include a great deal of Javascript might. I think that could be kept, though.
10) Change
to
11) The mention of multisite in the modules and themes section could be improved.
Something like:
In a multisite install, however, modules and themes can be added for use on one site only. See the "Multisite Configuration" section, below.
12) I would change
to
The current wording only mentions how to do it the right way for modules, is more verbose, and uses shouting.
13) The following sentence has an extra space:
14) If we are to be consistent with the changes just committed to UPGRADE.txt, then periods should be removed from all the URLs that end sentences. This needs to be done in the "More Information" section.
Comment #22
jhodgdonGood comments, thanks Evan... Here's a new patch, with more cleanup.
One note on the end-of-sentence URLs -- it only makes sense to me (and only barely!) to omit the period if the URL is at the end of a paragraph. In the middle of a paragraph, without the period it's hard to tell it's the end of a sentence.
Comment #23
Sahin CreditAttribution: Sahin commentedThere should exist a rule to synchronize update.txt, comments in settings.php, related book document (Installation guide in our case) and this install.txt. They might conflict each other after the updates and they usually do.
Comment #24
EvanDonovan CreditAttribution: EvanDonovan commented@Sahin: Once update.txt & install.txt are updated, then we can create a task to update the documentation (create it in the "Documentation" issue queue).
@jhodgdon: I don't necessarily agree with the no-dot rule either, but it was followed in the other document. I definitely don't think it should be done in middle of paragraphs.
I'll look over this in more detail later.
Comment #25
jhodgdonRE #23 - normally, the instructions for updating and installing should not change much, during the cycle of minor releases for 7.1, 7.2, etc. The idea is that these releases are mostly for bug fixes, etc. It's just in this case that the update/install files had not been updated for a while, while Drupal 7 had been updated quite a bit (UI changes, etc.).
Comment #26
EvanDonovan CreditAttribution: EvanDonovan commentedI think Sahin meant more that we need to make sure that the documentation in the Drupal.org handbook is up to date with these documents. I think that the way they're written in here is probably better than the online version now, because of the extensive review.
I'll open up an issue in "Drupal documentation" project when I have the time, unless you want to.
Comment #27
jhodgdonThere's already an issue about updating the install guide for D7 I believe... let's see.
#538054: Review and update the Installation guide
Comment #28
EvanDonovan CreditAttribution: EvanDonovan commentedThanks, jhodgdon! Wow, that is an old issue now...almost 500,000 nodes ago :) I'll see if I have the time to do review on those docs after this gets committed.
Sorry to come up with more points of possible improvement:
1) I think that it might more clear to novices to call MariaDB "a fully compatible alternative to MySQL" than "a fully compatible drop-in replacement for MySQL", but that is minor.
2) As in my #2, above, could the clean URLs bullet point be moved to the top of the list, since it is the most important? (Also minor, though, and debatable.)
3) Is it true that "RSS syndication" requires the XML extension? I think actually "RSS aggregation", as in the last server requirements bullet point, would be more accurate.
4)
"consult" should be followed by "with"
5)
This sentence is awkwardly worded. Is there a way to dis-entangle it? :)
6)
Should the word "user" be added, possibly, after "web server" to avoid confusion?
7) Steps 6-9 are not actually complete sentences that contain a verb. Could they be in a separate section, called something like "After You Install", and given names like the following:
1. Test your configuration.
2. Check file system settings.
3. Set up cron jobs (automated maintenance tasks).
4. (Optional) Hide documentation files from web access.
8)
I think this is a legacy from D6: from what I remember, the D7 starting page doesn't say "Welcome"; it just says something like "Create a new page".
9)
Maybe this is just a case that has never happened for me, but I don't usually hear the word "front-ends" used in this context. I'm not sure what a better wording would be though.
10)
"triggerd" needs an "e".
Thanks for changing the file to recommend cron job configuration, btw! I think the built-in cron feature should really just be considered a fail-safe for people who forget or don't have the ability to do so.
11)
These URLs don't need capitalization. They aren't capitalized on http://drupal.org/project.
This section looks a lot better now!
12)
Is there a specific URL to which we could point people?
SUMMARY: Most of the issues I've noted above are minor, but would be nice to get fixed. Some are debatable, too. I think one more round of review, and this can be RTBC, although it would be nice if some other people could do a review also.
One thing I noticed in re-reading the file is that it is incredibly comprehensive, which is good, but could be intimidating for people new to Drupal. A broader question is whether we should add something like a QUICKSTART.txt or README.txt with the basic version of the install instructions, so people know that it doesn't have to be a complicated process.
That might be unnecessary, though, since most of the people who are new to Drupal will probably read whatever is linked on Drupal.org, which puts the burden on us in #538054: Review and update the Installation guide to make the basic docs both simple and clear.
Thanks again, jhodgdon, for your work on this.
Comment #29
jhodgdonThe language for (1) was thoroughly negotiated in another issue. Not going to revisit that now. :)
Not sure about the other points... I'll take a look. And please don't apologize for reviews. :)
Comment #30
EvanDonovan CreditAttribution: EvanDonovan commentedThanks, jhodgdon; no worries about the first point then - I didn't know its history. I'll probably have time to review another patch as of tomorrow.
Comment #31
jhodgdonAddressing the comments in #28 above... Took them into account (I hope) with the following exceptions and notes:
1) See comment #29.
3) You are correct. The RSS output from the node RSS feed does not use the PHP XML library.
4) I don't agree, left as-is.
6) I think on some systems (such as Windows/IIS), the web server "user" isn't ever called a "user", so it might be confusing. I think it's clear enough as it is?
8) It does say "Welcome", but it doesn't have configuration steps.
12) Added link to http://drupal.org/getting-started/6/install/multi-site -- but we need to get the /6/ out of that URL...
Note about simplifying: I can't think of any steps we could really leave out. Can you? I mean, someone can just run the install.php script and see what happens, but if they decide to read the readme files, they should get full information IMO.
Anyway, here's a new file/patch.
Comment #32
jhodgdonI've filed an issue
#959294: Add new alias
to ask for a non-6 alias to be created for that page.
In the meantime, we could also link to it in INSTALL.txt as
http://drupal.org/node/251040
I guess... probably best not to ship INSTALL.txt with a link to http://drupal.org/getting-started/6/install/multi-site ?
Comment #33
EvanDonovan CreditAttribution: EvanDonovan commentedI think this is great, setting to RTBC. The new alias issue could be handled in a follow-up after #959294: Add new alias, or you could demote your own issue and repost with a patch that has the unaliased URL.
Comment #34
mokko CreditAttribution: mokko commentedCan requirements be optional? Just a thought, Maybe just "Optional Server Settings".
The file is now a bit on more verbose than necessary for my taste. Stuff like "-- the files
are in .tar.gz format and can be extracted using most compression tools" is not really necessary, I think. Still ok, though.
Comment #35
jhodgdonI was not sure if "optional requirements" made sense either, but they aren't really settings either (they are modules you need to have compiled into your PHP or Apache programs). If someone can think of another word, I'd be happy to use it.
The wording on the files is the same as what is currently in the UPGRADE.txt file, which recently went through revision.
If you have other specific suggestions for wording changes to make things more concise, please make them! We'd like the file to be as snappy as possible, without leaving out essential information.
Also, the idea is that while we give linux command line examples, we give people using other methods/OS the information they need about what is being done (i.e., we tell them what they need to accomplish, then give a linux command line example of how it could be done, rather than just saying "do this at your linux command line", since not everyone installing Drupal has access to that).
Comment #36
mokko CreditAttribution: mokko commentedI looked at UPGRADE.txt again. It is good now. It does not contain the phrase "OPTIONAL SERVER REQUIREMENTS". I am not exactly sure to what exactly you are referring when you say the wording is the same. Probably not that important.
If "Server Settings" is not good, then maybe
I am not sure that the section on possible installation problems helps more than it distracts. Adds a lot of text that is only necessary if you don't follow the instructions. More specifically, I refer to this section.
I do like this section, but it not always necessary, so wouldn't it be better to put it in the handbook and link to it from the INSTALL.txt. I checked with Drupal 6 INSTALL.txt and it doesn't have these "in case something goes wrong remarks". It makes it look a little bit like we expect it to go wrong.
I quess "- You want to restrict access to uploaded files." needs some explanantion. (It is new comparative D6 INSTALL.txt.)
I am not quite through with reading the new INSTALL.txt carefully again, but that's all the time I have this morning. Sorry.
Comment #37
mokko CreditAttribution: mokko commentedmarked http://drupal.org/node/683892 as duplicate of this one, athough the other one is older.
Comment #38
EvanDonovan CreditAttribution: EvanDonovan commented@mokko: If you see up above, we changed it from "Optional Tasks", since they are clearly not tasks. I said that the most accurate phrase would be "Module-Specific Requirements" or "Server Requirements for Specific Modules", but neither of those exactly rolls off the tongue. I think it's fine to leave as is.
I see what you are saying about the troubleshooting section, but I think most people don't actually read the INSTALL.txt unless they are having problems. As such, it's probably good for it to be comprehensive.
Comment #39
EvanDonovan CreditAttribution: EvanDonovan commented@jhodgdon: Are there any parts of your patch from the earlier issue #683892: Text clean-up of INSTALL.txt that didn't make it in here?
Comment #40
jhodgdonAh, I'd forgotten about that other issue. I think it's all in here...
Comment #41
mokko CreditAttribution: mokko commentedthe rest is good. A bit on the verbose side, but good.
Comment #42
jhodgdonJust a note... after this (hopefully) gets committed, we should revisit this old issue I just found deep in the depths of the issue queue:
#285053: Improve INSTALL.txt description of multi-site installation with subdirectories
Comment #43
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks.