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

add support for .tar.xz files #71

Merged
merged 1 commit into from
Sep 12, 2013
Merged

add support for .tar.xz files #71

merged 1 commit into from
Sep 12, 2013

Conversation

jf647
Copy link

@jf647 jf647 commented Sep 6, 2013

This adds support for .tar.xz and .txz files, because some authors feel the need to make my life miserable for the sake of a few bytes.

@seth
Copy link

seth commented Sep 7, 2013

This patch looks good. We have a contribution process that we follow for all Opscode repositories. Details below. I hope you'll be up for the few minutes inconvenience so that we can get this merged.

Thank you for taking the time to help make Chef better! We require a Contributor License Agreement (CLA) from contributors to help protect all Chef users. It is easy to complete one online. Instructions to do so are on our wiki, which is linked below. Once your account is updated, you will have access to 'resolve' tickets that indicates to the code review team that your contribution is ready for review.

http://wiki.opscode.com/display/chef/How+to+Contribute
http://wiki.opscode.com/display/chef/Code+Review

@lamont-granquist
Copy link
Contributor

:shipit:

@jf647
Copy link
Author

jf647 commented Sep 11, 2013

I've filed CHEF-4537 for the fix. I assume omnibus tickets go under the Chef project, as there didn't seem to be a better match.

I have filed a CLA (on Sept 6/13) and received the paperwork from EchoSign, but I haven't heard back from Opscode legal, nor do I appear to have permission to resolve the ticket, so once that comes through I'll action it.

@seth
Copy link

seth commented Sep 11, 2013

Awesome! Thanks for taking the time to do the paperwork. Filing in the CHEF project was the right thing. I'll ask about CLA status today.

@seth seth merged commit 62ec360 into chef:master Sep 12, 2013
@seth
Copy link

seth commented Sep 12, 2013

The specs pass for me and the change looks straight forward.

@seth
Copy link

seth commented Sep 12, 2013

Thanks again!

@jf647 jf647 deleted the add-xz-support branch September 12, 2013 21:33
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.

3 participants