Closed (won't fix)
Project:
Drupal core
Version:
7.x-dev
Component:
documentation
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
29 Aug 2012 at 20:54 UTC
Updated:
10 Apr 2024 at 21:23 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
David_Rothstein commentedHere's the patch for Drupal 7. And somehow I posted this issue in the wrong component...
Comment #2
jhodgdonThat looks OK... Does it need to get put into the 8.x branch too? If so, by usual policy, we should start there and then patch 7.x.
Alternatively, I wonder if we need all the 4.x, 5.x, 6.x, etc. versions in the 7.x file at all?
Comment #3
David_Rothstein commentedI guess it's time to get this into Drupal 8.
Could be a simple patch, except for two questions (somewhat related to jhodgdon's comment above):
Comment #4
David_Rothstein commentedThinking about it more I'm actually leaning towards not. Everything added in those later releases should already be in the next version of Drupal anyway (unless it's not applicable). For example, Drupal 6.37 was for SA-CORE-2015-003, but so was Drupal 7.39 - so we'd just be repeating the same thing twice in the CHANGELOG.txt.
This also means we don't have to keep copying new CHANGELOG.txt entries forward ad infinitum (which probably wouldn't happen anyway).
So if we do it that way, we would just need to copy Drupal 6 entries through Drupal 6.20 and Drupal 7 entries through Drupal 7.41 to the Drupal 8 CHANGELOG.txt, and not add any others.
Comment #13
quietone commentedCHANGELOG.txt has been modified a few times since this issue was create. The most relevant is #2647824: [policy and patch] Determine what should go in CHANGELOG.txt for 8.x core releases where the was decided to include link to pages with relevant information.
This still applies to the Drupal 7 file, though.
Comment #14
mulambo commentedThere shouldn't be any harm done by adding patch notes from previous versions (6.x), like it was before 6.23. Since Drupal 8 doesn't include patch notes in changelog and Drupal 6 reached EOL this would be limited to one time only action in Drupal 7.
There are 2 aspects:
1) We include patch notes untill final release 6.38 (patch changelog-txt-update-1762708-14-1.patch).
2) We include patch notes from Drupal 6 LTS 6.52 (patch changelog-txt-update-1762708-14-2.patch)
Comment #15
bhumikavarshney commentedHi @Mulambo ,
Both the patch applies cleanly and i have tested this for Drupal 7.
Thanks
Comment #16
poker10 commentedThanks for reporting and working on this.
I do not think this is worth doing - D10 CHANGELOG.txt was changed and it does not contain such information anymore, see: https://api.drupal.org/api/drupal/core%21CHANGELOG.txt/10 (and #2647824: [policy and patch] Determine what should go in CHANGELOG.txt for 8.x core releases). The file never contained information about all D7 releases. Agree with David comment #4, that adding all D6 releases would duplicate fair amount of fixes, as most of them were backported from D7 to D6 in later phases.
We will continue with adding short changelog entries on each D7 release until the D7 EOL. But regarding the D6 entries, I think the best approach is to close this as Won't fix.