Added LZ4 compression #23

Closed
wants to merge 2 commits into
from

Projects

None yet

2 participants

@railsmechanic
Collaborator
railsmechanic commented Feb 15, 2017 edited

Added LZ4 compression...

railsmechanic added some commits Feb 15, 2017
@railsmechanic railsmechanic Added Tar.LZ4 compression
Added Tar.LZ4 compression
736a751
@railsmechanic railsmechanic Update README.md
1477215
@railsmechanic railsmechanic changed the title from Added Tar.LZ4 compression to Added LZ4 compression Feb 15, 2017
@mholt

Cool, thanks! In order to be able to access this with the CLI tool, please make the additions necessary in main.go.

+
+// isTarLz4 checks the file has the bzip2 compressed Tar format header by
+// reading its beginning block.
+func isTarLz4(tarlz4Path string) bool {
@mholt
mholt Feb 15, 2017 Owner

The comment says bzip2, but the code just checks the tar format?

+// can be those of regular files or directories. Regular
+// files are stored at the 'root' of the archive, and
+// directories are recursively added.
+func (tarLz4Format) Make(tarlz4Path string, filePaths []string) error {
@mholt
mholt Feb 15, 2017 Owner

Don't forget to update the godoc comments ;)

@railsmechanic railsmechanic reopened this Feb 15, 2017
@mholt
Owner
mholt commented Feb 15, 2017

Why did you close this one?

@railsmechanic
Collaborator

Sorry my fault... Plz ignore this one and head over to PR #24

Many thanks 👍

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