Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow downloading directly to a profile directory #94

Closed
wants to merge 3 commits into from
Closed

Allow downloading directly to a profile directory #94

wants to merge 3 commits into from

Conversation

BrianGilbert
Copy link

@BrianGilbert BrianGilbert commented Sep 12, 2013

If you use profiles for your site builds drush has no way to download to the profile directory without adding a custom destination path on every drush call, this patch allows you to download directly to the profile when you are inside it's directory.

If you are in the /profiles directory downloads fall back to default behaviour.

If you use profiles for your site builds drush has no way to download to the profile directory without adding a custom destination path on every drush call, this patch allows you to download directly to the profile when you are inside it's directory.

If you are in the /profiles directory downloads fall back to default behaviour.
@weitzman
Copy link
Member

weitzman commented Sep 19, 2013

I see this to be of limited use. Basically only profile developers would use it. They can use --destination IMO

@weitzman weitzman closed this Sep 19, 2013
@BrianGilbert
Copy link
Author

BrianGilbert commented Sep 19, 2013

This seems like a very odd response, even if it's of limited use, that doesn't mean it shouldn't be included if it doesn't break anything does it?

@BrianGilbert
Copy link
Author

BrianGilbert commented Mar 18, 2014

Could you please reconsider this?

using --destination doesn't respect the directory structure within the profile, eg if I download a theme it will get put in the directory set as the destination instead of the themes subdirectory of the profile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants