tdtds / kindlegen

Installing kindlegen command via gem
GNU General Public License v3.0
47 stars 18 forks source link

Pass --no-same-owner to tar #15

Closed gabebw closed 9 years ago

gabebw commented 9 years ago

When installing Kindlegen on a Docker container on CI, there are a lot of "Cannot change ownership" errors like this:

tar zxf kindlegen_linux_2.6_i386_v2_9.tar.gz
tar: docs/german/Bittelesen.txt: Cannot change ownership to uid 311227, gid 100: Invalid argument

tar has a --no-same-owner option that does not preserve the original owner of the tar files. This means that those errors go away, since it just uses whoever the current user is as the owner.

gabebw commented 9 years ago

Thanks for packaging Kindlegen as a gem - it's super helpful for us.

tdtds commented 9 years ago

I think that option may only for GNU tar? But almost Linux distributions are using GNU tar, there will be no problem maybe. ... OK, I'll merge it. Thank you!

tdtds commented 9 years ago

and released new gem!

gabebw commented 9 years ago

Thank you!