Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
As discussed on development list, this patch removes 4.7 - 5.x updates. Untested.
Comment | File | Size | Author |
---|---|---|---|
#25 | 47updates_5.patch | 23.88 KB | hswong3i |
#15 | 47updates_4.patch | 23.88 KB | scor |
#14 | 47updates.patch | 23.88 KB | hswong3i |
#13 | 47updates.patch | 23.86 KB | hswong3i |
#10 | 47updates.patch | 30.37 KB | hswong3i |
Comments
Comment #1
webernet CreditAttribution: webernet commented- Needs to keep updates-5.x-extra (1022)
- Needs to update the relevant line in the changelog.
Comment #2
catchYes, and yes.
Comment #3
Gábor HojtsyI tried to direct people to here from the mailing list. Let's see what they'd like to add.
Comment #4
Crell CreditAttribution: Crell commentedAs stated on the mailing list, I'd prefer that we keep a "full supported" set of upgrade functions. That is, Drupal 6 has upgrade functions for 4.7->5 and 5->6, D7 has upgrade functions from 5->6 and 6->7, etc. There will be people trying to make a 2 version jump and for non-complex sites (those most likely to not be kept up to date anyway) that's probably easier than doing it in stages, since you only have to upgrade custom code or custom themes once.
Older than that, I agree, no sense maintaining it in core.
Comment #5
catchUnless I'm missing something, the only extra work required is:
wget and unpack drupal-5.x tarball
copy/edit settings.php
run update.php
remove the folder ready for D6 tarball
As KarenS pointed out on the development list, it will be necessary to install and upgrade cck 5.x as well if present in 4.7 - this adds one or two steps to the above in those cases. There's no reason to upgrade custom code and themes as far as I can see, at all, custom modules and themes (and php blocks) should be disabled when running the updates anyway.
Comment #6
Gábor HojtsyI looked up what did we suggest earlier:
http://drupal.org/drupal-5.0 :
For the most trouble free transition, it is recommended that you first upgrade existing installations to Drupal 4.7.
http://drupal.org/drupal-4.7.0 :
For the most trouble free transition, it is recommended that you first upgrade existing installations to Drupal 4.6.
http://drupal.org/drupal-4.6.0 :
For the most trouble free transition, it is recommended that you first upgrade existing installations to Drupal 4.5.
http://drupal.org/drupal-4.5.0 :
For the most trouble free transition, it is recommended that you first upgrade existing installations to Drupal 4.4 and only then to Drupal 4.5.
http://drupal.org/drupal-4.4.0 :
First, for the most trouble free transition it is recommended that you first upgrade existing installations to Drupal 4.3.0, Drupal 4.3.1 or Drupal 4.3.2.
All in all, it looks like this "recommendation" is in place since Drupal 4.4.0 although we did not make this a requirement ever.
Warning, tired procrastination fun follows:
http://drupal.org/drupal-4.3.0 :
no info on upgrades, but "With this installment, specific focus has gone into improving user-friendliness, both to end-users and administrators." woot-woot :)
http://drupal.org/drupal-4.2.0 :
no info on upgrade, but "Support for Microsoft SQL server." woot-woot
http://drupal.org/drupal-4.1.0 :
no info on upgrade, but look at http://drupal.org/image/tid/7?page=0 :)
http://drupal.org/drupal-4.0.0 (this is unpublished for some reason):
no info on upgrades, but thats a press release! a press release!
no node found for Drupal 3.0.0, actually, the first nodes on drupal.org are later then the 3.0 release.
BTW I actually set the aliases for these nodes as I went down from 4.4.0 which was long ago enough, that it did not have an alias, and none of the earlier nodes had.
Comment #7
Gábor Hojtsycatch: you would actually need to update most of your contrib modules in two stages, as it is quite likely that more contrib modules follow suit if we enforce what we did recommend for a few years now.
Comment #8
catchJust found these three functions in update.php that ought to also be removed by this patch:
function update_fix_schema_version()
function update_fix_sessions()
function update_fix_watchdog_115()
Comment #9
hswong3i CreditAttribution: hswong3i commentedSubscribe
Comment #10
hswong3i CreditAttribution: hswong3i commentedI found that some more functions should also remove, beside listed in #8:
function update_fix_watchdog()
function update_fix_system_table()
function update_fix_access_table()
function update_convert_table_utf8($table)
function update_create_cache_tables()
Comment #11
hswong3i CreditAttribution: hswong3i commentedComment #12
Gábor HojtsyPatch needs to be updated, as #200271 was committed.
Comment #13
hswong3i CreditAttribution: hswong3i commentedPatch via latest CVS HEAD.
Comment #14
hswong3i CreditAttribution: hswong3i commentedPatch via latest CVS HEAD.
Comment #15
scor CreditAttribution: scor commentedneeded a reroll.
Comment #16
hswong3i CreditAttribution: hswong3i commentedI diff #14 and #15, but seems no different?
Comment #17
scor CreditAttribution: scor commentedwith #14 I had Hunk #1 FAILED at 1039 for modules/system/system.install
Comment #18
hswong3i CreditAttribution: hswong3i commented@scor: Oh you are right. I test #15 with latest CVS HEAD, all pass :-)
Comment #19
catch#15 still applies cleanly, tested by hswong3i, so back to rtbc.
I will note this saves 500kb of memory during the upgrade from 5.x - 6.x (and possibly the same at install time) according to scor on another issue, which is currently occupying two release blockers.
Comment #20
scor CreditAttribution: scor commentedThe estimation I gave earlier was approximate, and after retesting this on install and update on 2000 node site, I notice that we only save about 100KB, not more. is it worth removing the older updates then?
Comment #21
catchWe're removing the updates not to save memory, but to clean up the code base (and to prevent unsupported upgrades and associated support requests). It's a shame there's not much additional bonus, but that was never the original intent of this issue. Whether those updates are removed or not is up to the core committers, but the patch does the job.
Comment #22
hswong3i CreditAttribution: hswong3i commented@catch: +1. I guess the most safe upgrade procedure is follow our tradition as state in #6: as a normal user, this instruction should be enough clear; from a point of view of developer, we can keep our main focus in the correctness of upgrade from previous version ONLY, so simpler and less buggy.
P.S. sorry for changing issue title, as I am now trying to collect similar issues for a simpler tracing.
Comment #23
chx CreditAttribution: chx commentedAs Siren is not a project on Drupal.org but an unofficial fork, I am removing it from issue titles to avoid confusion.
Comment #24
Dries CreditAttribution: Dries commentedSorry, the patch no longer applies and needs a quick reroll.
Comment #25
hswong3i CreditAttribution: hswong3i commentedHere it is.
Comment #26
Gábor HojtsyCommitted, thanks.
Comment #27
(not verified) CreditAttribution: commentedAutomatically closed -- issue fixed for two weeks with no activity.