Drupal 6 is unsupported, let's remove it.

CommentFileSizeAuthor
#2 d6-remove-2676682-2.patch452 bytesklausi

Comments

klausi created an issue. See original summary.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Active » Needs review
StatusFileSize
new d6-remove-2676682-2.patch452 bytes

Patch.

chandeepkhosa’s picture

chandeepkhosa commented
Status: Needs review » Reviewed & tested by the community

Tested & confirmed as RTBC :)

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

You know, I don't really understand why we have a single MAINTAINERS that applies to all branches anyway. Why should the D8 MAINTAINERS have anything about D7 in it, much less older versions?

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented

@jhodgdon yes, it is a bit confusing, but probably off-topic for this issue. Care to open a separate one to discuss branches and what should be in MAINTAINERS.txt?

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

Not really... but I don't know why we should particularly remove D6 at this time.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

I think it does make sense if you need to look up who to talk to for backports, etc. You don't need to open the other branch docs. Also for credit. We also keep the changelog for the past versions for all the branches. Ie. D7 changelog has all the D6 releases, etc.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented

@jhodgon: we remove d6 now because it has been marked unsupported a couple of days ago.

catch’s picture

catch
he/him
Primary language English
commented
Version: 8.1.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

I think it does make sense if you need to look up who to talk to for backports, etc. You don't need to open the other branch docs.

Yes I think it's potentially useful for people using an 8.x tarball from scratch, it shows what the structure of the previous releases is.

Gabor definitely deserves to be off the hook in the docs. Comitted/pushed to 8.1.x, and cherry-picked to 8.0.x.

Should also backport to 7.x.

  • catch committed 418196b on 8.1.x 
    Issue #2676682 by klausi: Remove Drupal 6 from MAINTAINERS.txt

  • catch committed c701bc4 on 8.0.x 
    Issue #2676682 by klausi: Remove Drupal 6 from MAINTAINERS.txt

(cherry...
therealssj’s picture

therealssj commented
Status: Patch (to be ported) » Closed (fixed)

This has been fixed in 7.x.
Closing the issue.