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.
updated the readme with useful information about lib folder paths
Comment | File | Size | Author |
---|---|---|---|
#2 | 2592595-patch.diff | 852 bytes | dakku |
|
Comments
Comment #2
dakku CreditAttribution: dakku as a volunteer commentedComment #3
rjacobs CreditAttribution: rjacobs commentedThis would probably be a good note to have in the readme, for both D8 and D7. I'm pretty sure that the supported library locations will be consistent for both D7 and D8 so likely this could apply to each. I suppose we should also confirm that the notes at https://www.drupal.org/node/1440066 are consistent with this list.
I just need to confirm that those 4 lib location options will remain unchanged in D8, and then these notes could probably just be inserted into the readme files.
Comment #4
tstoecklerBecause https://www.drupal.org/node/1440066 basically explains this - including the different directories - in more detail I think the sentence with the reference to it should remain the last sentence. Otherwise this looks fine to me. Just as a note: The patch contains trailing whitespace, that can be fixed on commit, though.
Not sure what we will end up doing in D8, so moving back to 7.x-2.x. I don't mind merging back and forth and D8 is far from finished, so I don't think it makes sense to follow the core "D8 then backport" policy here.
Comment #5
rjacobs CreditAttribution: rjacobs commentedLet's run with that decision. It's probably smart to continue managing these details within one central place within the docs, as edits are much easier there. I was also reminded that the install locations may still be in flux a bit for 8.x (e.g. add
sites/all/assets
).If others have strong feelings otherwise (for 7.x or 8.x) I'm happy to re-open and assign myself to this, but I think it might be best to close it for now.