I'm attaching a patch to the UPGRADE.txt file to add the following statement in the D6->D7 upgrade instructions:

"In addition to updating to the latest available version of Drupal 6.x core,
you must also upgrade all of your contributed modules for Drupal to their
latest Drupal 6.x versions."

The instructions currently explain the need to update to the latest version of Drupal 6.x but are not explicit about the need to update to the latest version for contrib modules specifically. It is especially important to point this out for D6->D7 upgrades because many module developers are not including the D6 hook_update_N() functions for their modules in their .install files. This is happening much more frequently than it was with the D5->D6 upgrades. (It's probably happening because D7 introduced a more radical change to the database API, so module developers would have to substantially rewrite their hook_update_N() functions to get them working in D7 and are (perhaps understandably) reluctant to do so.

Files: 
CommentFileSizeAuthor
#13 upgrade-1167996-13.patch612 bytesBWPanda
PASSED: [[SimpleTest]]: [MySQL] 39,154 pass(es).
[ View ]
#9 upgrade-1167996-9.patch4.51 KBrocket_nova
PASSED: [[SimpleTest]]: [MySQL] 37,041 pass(es).
[ View ]
#5 upgrade-1167996-5.patch4.54 KBrocket_nova
PASSED: [[SimpleTest]]: [MySQL] 33,996 pass(es).
[ View ]
#3 upgrade-1167996-3.patch4.54 KBrocket_nova
PASSED: [[SimpleTest]]: [MySQL] 33,999 pass(es).
[ View ]
#1 UPGRADE.txt_needs_additional_instruction-1167996-1.patch5.03 KBSheldon Rampton
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch UPGRADE.txt_needs_additional_instruction-1167996-1.patch. This may be a -p0 (old style) patch, which is no longer supported by the testbots.
[ View ]

Comments

StatusFileSize
new5.03 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch UPGRADE.txt_needs_additional_instruction-1167996-1.patch. This may be a -p0 (old style) patch, which is no longer supported by the testbots.
[ View ]

Here's the patch.

Version:7.x-dev» 8.x-dev
Status:Active» Needs work
Issue tags:+needs backport to D7

There are changes in your patch that are not related to this fix (which I think is a good idea). Please make a patch that doesn't make the other changes (such as removing the mention of the .zip format, which we would please like to keep in there). I think that's the only one...

I also think it would be helpful to mention how you go about updating the modules? Maybe not...

And when you add a patch, please set the status to "needs review" so that both the test bot and people know it's time to review it.

And finally, this technically needs to be patched against D8 and then applied to both d8 and d7. Hmmm. The D8 upgrade guide probably needs to have its numbers updated first. Sigh.

Thanks!

Assigned:Unassigned» rocket_nova
Status:Needs work» Needs review
StatusFileSize
new4.54 KB
PASSED: [[SimpleTest]]: [MySQL] 33,999 pass(es).
[ View ]

Re-roll patch from #1.

Status:Needs review» Needs work

Looks pretty good... One mention of 6.x crept in though:

+3. In addition to updating to the latest available version of Drupal 6.x core,

Status:Needs work» Needs review
StatusFileSize
new4.54 KB
PASSED: [[SimpleTest]]: [MySQL] 33,996 pass(es).
[ View ]

Good catch @jhodgdon. Fixed in the attached patch.

Status:Needs review» Reviewed & tested by the community

Looks good, thanks!

Of course the wording will have to be changed back to 6.x when the D8 patch is backported to D7.

Version:8.x-dev» 7.x-dev
Status:Reviewed & tested by the community» Needs work

Committed to 8.x. Moving to 7.x-dev for backporting.

Status:Needs work» Needs review
StatusFileSize
new4.51 KB
PASSED: [[SimpleTest]]: [MySQL] 37,041 pass(es).
[ View ]

Patch for 7.x.

Status:Needs review» Reviewed & tested by the community

Thanks, looks ready to me!

Status:Reviewed & tested by the community» Fixed

Looks great!

Committed and pushed to 7.x. Thanks!

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Assigned:rocket_nova» Unassigned
Status:Closed (fixed)» Needs review
StatusFileSize
new612 bytes
PASSED: [[SimpleTest]]: [MySQL] 39,154 pass(es).
[ View ]

While upgrading a site from 6.x to 7.x, I noticed that one of the 7.x references wasn't changed to 6.x in UPGRADE.txt. The attached patch fixes this.

Status:Needs review» Fixed

Thanks for finding and fixing this error! I've committed this change to 7.x.

Automatically closed -- issue fixed for 2 weeks with no activity.