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

Embed "dmd2" folder, not its parent #103

Merged
merged 1 commit into from Jul 27, 2014

Conversation

brad-anderson
Copy link
Member

The beta installer was putting everything in C:\D\dmd.v2.066.0-b6.windows rather than C:\D.

@brad-anderson
Copy link
Member Author

This will need to be cherry picked into the 2.066 branch prior to 2.066 release. (ping @AndrewEdwards).

Also, please forgive me for accidentally making d2c01d7 and its revert 8ed9a9b. Github's online editor doesn't make a pull request when you are a committer apparently.

AndrewEdwards added a commit that referenced this pull request Jul 27, 2014
Embed "dmd2" folder, not its parent
@AndrewEdwards AndrewEdwards merged commit 23c4c29 into dlang:master Jul 27, 2014
@MartinNowak
Copy link
Member

Thanks, great to have the files embedded finally.
Is it really worth to use LZMA compression? Because that compression is terribly slow.

AndrewEdwards added a commit that referenced this pull request Jul 28, 2014
Embed "dmd2" folder, not its parent
@brad-anderson
Copy link
Member Author

Yeah, it's pretty slow. Now that we aren't including the files for every platform it's actually rather small already. I'll try out removing /SOLID, BZIP2, and ZLIB and try to find the sweet spot.

@MartinNowak
Copy link
Member

I'll try out removing /SOLID, BZIP2, and ZLIB and try to find the sweet spot.

Thanks, it's not of utmost importance though.

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