Remove 7zip requirement (and bundling)

drud / quicksprint

A basic toolkit to get people started with ddev and a Drupal codebase on macOS, Linux, or WIndows

Other

24 stars 16 forks source link

Remove 7zip requirement (and bundling) #86

Closed rfay closed 6 years ago

rfay commented 6 years ago

Since git-bash now bundles xz, we don't have to ship 7zip any more and the code is simpler.

This removes 7z/7zip/p7zip.

rfay commented 6 years ago

I worked through this with a full test on Windows 10 Pro.

Downloaded the package from here (tar)
Uninstalled docker, ddev, and git.
Checked for extra installs of 7zip or xz to remove.
Did the required installs of git, docker, ddev
Shared the C drive in docker.
Ran install.sh
Ran start_sprint.sh
Ran start_clean.sh

Looks like it's all working on that flow.

rfay commented 6 years ago

Except for one gotcha, a WIndows thing that everybody should know about when doing support: If the number of hosts assigned to 127.0.0.1 (or any IP) goes over 10, then windows won't resolve any more. So ddev now has ddev hostname --remove-inactive to clean up all that extra stuff. In this case, this was the 11th project that I'd spun up.

rfay commented 6 years ago

Reworked the install instructions to put git for windows install first. Good idea.

mradcliffe commented 6 years ago

Should this get another test run?

Edit: I think this is good otherwise.

rfay commented 6 years ago

I gave it a full run, and it did ok on everything but circleci. If we can't get circleci to behave we'll try to deal with that otherwise. Thanks!

I do hope we can get a couple of people besides us to try the package before Drupal Europe.