Issue #12: Provide full support for multisites.

[ghost]

Fixes #12

[indigoxela]

Nice work! But I do have one concern.

Currently b dl forces downloading to the main directory (modules, themes...) You now enforce downloading to the sites subdirs if it's a multisite install.

Both doesn't seem perfect to me.

If I use multisite, I always share at least the majority of contrib modules (hence main dir is fine). Only single modules and likely the theme or some layouts are site specific.
So your approach actually makes it more complicated for me to use "b" with multisite.

[ghost]

@indigoxela Thanks for the feedback! It's good to hear how other people use Backdrop/multisite.

So with this PR if you run b dl from within your site directory, it'll download to there (e.g. /sites/[SITE]/modules). Whereas if you run the same command from outside of a site directory (e.g. /backdrop), it'll download to there instead (e.g. /backdrop/modules). So basically it should download to wherever you run this from.

I believe this is how Drush for Drupal 7 works. Drush for Backdrop doesn't fully support multisites yet, which is why I started using b instead as it's much easier to fix 😃

How would you prefer it to work? E.g. where do you prefer to run commands from, and where do you want them to download to?

[ghost]

BTW @indigoxela, are you on Gitter? https://gitter.im/backdrop/backdrop-issues If so, we can chat there if it's easier.

[indigoxela]

it should download to wherever you run this from.

Ah, I didn't test that from the backdrop root, as "b" says "BackdropCMS is not installed yet." - but downloads it then to the main dirs. Cool, very helpful!

The warning message was misleading me.

BTW: Drush with D7 allows downloading projects to anywhere, regardless of an install.

[ghost]

The warning message was misleading me.

Yeah, that message is displayed if b can't find a settings.php file and therefore there's technically no Backdrop site available. But in that case it can (and should IMO) still download things to the root directory. Maybe a separate issue can address the wording/display of that message...