updated the readme with useful information about lib folder paths

CommentFileSizeAuthor
#2 2592595-patch.diff852 bytesdakku
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dakku created an issue. See original summary.

dakku’s picture

Status: Active » Needs review
FileSize
852 bytes
rjacobs’s picture

Category: Bug report » Task

This 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.

tstoeckler’s picture

Version: 8.x-3.x-dev » 7.x-2.x-dev
Status: Needs review » Needs work

Because 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.

rjacobs’s picture

Status: Needs work » Closed (won't fix)

Because 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.

Let'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.