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

drush dl (--gitsubmodule) downloads extra pieces into /modules (and points there with .gitmodules) when symlinked parent dir #566

Merged
merged 1 commit into from
Nov 7, 2014

Conversation

nhatthm
Copy link
Contributor

@nhatthm nhatthm commented Apr 7, 2014

@weitzman
Copy link
Member

weitzman commented Apr 8, 2014

I'll point out that one late comment there asked for more tests.

@weitzman weitzman changed the title Fix symlink issue. drush dl (--gitsubmodule) downloads extra pieces into /modules (and points there with .gitmodules) when symlinked parent dir Apr 8, 2014
@nhatthm
Copy link
Contributor Author

nhatthm commented Apr 8, 2014

Well at least, it's working on my local

@nhatthm
Copy link
Contributor Author

nhatthm commented Jun 25, 2014

Hi @weitzman, could you please take a look on this? What needs to be done in order to fix the issue?

@weitzman
Copy link
Member

The PM component is managed by @jonhattan.

@nhatthm
Copy link
Contributor Author

nhatthm commented Jul 29, 2014

@jonhattan Could you please tell me what needs to be done to release my fix? Thanks.

@luksak
Copy link

luksak commented Nov 6, 2014

@jonhattan What is missing here? I am using this patch since more than two years without any issues.

@jonhattan
Copy link
Member

Well I have been negligent to add tests for scenarios with symlinks. Since this change only affects git_drupalorg engine, we can go ahead and merge it. Sorry for the delay.

jonhattan added a commit that referenced this pull request Nov 7, 2014
drush dl (--gitsubmodule) downloads extra pieces into /modules (and points there with .gitmodules) when symlinked parent dir
@jonhattan jonhattan merged commit 972dd8a into drush-ops:master Nov 7, 2014
@luksak
Copy link

luksak commented Nov 7, 2014

Wow, this was fast :)

@fearlsgroove
Copy link

Submodules are broken on 6.x too whenever full absolute paths aren't already being used -- the same patch/fix applies. Can you cherry-pick for 6 please?

@jonhattan
Copy link
Member

Cherry picked from 6.x and 5.x

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.

5 participants