Skip to content
This repository has been archived by the owner on Nov 25, 2020. It is now read-only.

Add tar binary command support to the compression plugin #1196

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

snw35
Copy link

@snw35 snw35 commented Jul 20, 2016

This allows the compression plugin to use the tar binary (if found) for the creation and extraction of archive files, which improves performance. It also supports and detects the parallel compression utilities: pigz, lbzip2, and pbzip2. If any of these are installed, then they will be used to compress or extract archive files while utilizing all available CPU threads on the system, increasing performance even further. This will work natively in nearly all flavours of Linux, and Windows is also supported as long as a usable tar binary is present in the command path. It will fall back to using the existing Phar implementation if the tar command is not found.

@cdujeu cdujeu added this to the 7.1.0 milestone Sep 21, 2016
@cdujeu
Copy link
Member

cdujeu commented Sep 21, 2016

Can you update the PR to make it compatible again? Not urgent, will probably not be merged before 7.0, but after.

@cdujeu
Copy link
Member

cdujeu commented Oct 5, 2016

mm, seems like there still are some conflicts, + the PR comes with a merge of 3000+ files... Can you maybe checkout a clean version and submit a new PR from there? thx!

@snw35
Copy link
Author

snw35 commented Oct 5, 2016

Is that any better now, or would submitting a new PR from a fresh fork still be the best way to go?
Thanks

@snw35
Copy link
Author

snw35 commented Oct 6, 2016

Ok, I think that's cleaned it up. I reset my local copy, rebased my fork onto the latest develop, re-implemented the change in a new commit, and then force-pushed to my repo.

I hope this is OK to merge now - let me know if you'd still like me to create a new fork and start again though, whichever is easier for you to work with.

@cdujeu cdujeu modified the milestones: 7.0.2, 7.2.X Nov 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants