Skip to content

Conversation

@s-ludwig
Copy link
Member

This changes the behavior of #712 to be backwards compatible with the old single-layer package structure. It also fixes searching within non-managed paths ("dub add-path") to not expect the new layout.

@s-ludwig s-ludwig changed the title Follow-up pull request for #712 Fetch packages into a folder with their name. Fixes #502. Dec 14, 2015
@s-ludwig s-ludwig force-pushed the issue502-nested-installed-packages branch from 27f6588 to 7e4f4ca Compare December 14, 2015 15:42
@skoppe
Copy link
Contributor

skoppe commented Dec 15, 2015

Looks good to me.

@s-ludwig
Copy link
Member Author

Thanks! Practical testing didn't show any issues so far either, will merge.

s-ludwig added a commit that referenced this pull request Dec 15, 2015
…nstalled-packages

Fetch packages into a folder with their name. Fixes #502.
@s-ludwig s-ludwig merged commit 1d8a310 into master Dec 15, 2015
@s-ludwig s-ludwig deleted the issue502-nested-installed-packages branch December 15, 2015 20:57
@MartinNowak
Copy link
Member

Great, that this is merged now. Would be nice if some of the git submodule people could test this and write a small how-to @CyberShadow, @Hackerpilot.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, didn't knew there was a system-wide installation path.
This is useful for caching common dependencies, why is it deprecated?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Huh, what gives you the idea it's deprecated?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants