The sites/all/ directory, while an excellent idea, is ambiguous and somewhat confusing in its implementation. Improved documentation, as per the sites/all/README.txt clarification, is needed. However ambiguity can be further reduced by pre-installing empty “modules” and “themes” sub-directories in the sites/all/ directory.

Comments

Rainy Day’s picture

Rainy Day commented

Just a clarification: Pre-installing these empty sub-directories makes the intent of the sites/all/ directory clearer and its proper use more intuitive.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

This would be nice, but unfortunately you can't package empty directories w/ CVS. So both the sites/all/modules and sites/all/themes directories would *also* need README.txts, and I believe it was discussed already that this wouldn't be done. :\

Rainy Day’s picture

Rainy Day commented

That is really too bad as it is such a simple thing to do, would reduce confusion, and make Drupal more user-friendly and intuitive.

It is attention to little details like this which, collectively, are important to the ease of use of and polish of the overall package.

chx’s picture

chx commented
Title: sites/all/ sub-directories should be pre-installed » sites/all for Drupal project is almost useless
Project: Drupal core » Drupal.org site moderators
Version: 5.x-dev »
Component: base system » other
Category: feature » bug

This can be solved easily by the packaging script adding two empty dirs.

christefano’s picture

christefano commented
Title: sites/all for Drupal project is almost useless » have the packaging scripts add empty "modules" and "themes" directories to sites/all
Component: other » Other

Renaming the issue to make it more identifiable at a glance.

chriscf’s picture

chriscf commented
Project: Drupal.org site moderators » Drupal core
Version: » 7.x-dev
Component: Other » other

This is a Drupal issue, rather than an issue with the drupal.org site. I can't believe this is much more complicated than "mkdir -p sites/all/{modules,themes}" somewhere after the checkout. Tagging to D7, though would be better to have this done for future D6 releases.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Active » Fixed

This is fixed in D7.

chriscf’s picture

chriscf commented
Version: 7.x-dev » 6.12
Priority: Normal » Minor
Status: Fixed » Active

In that case, re-tagging to current D6 and reducing priority.

multiplextor’s picture

multiplextor commented
Status: Active » Closed (won't fix)

Closed. The reason: expired.