search

fabacab / git-archive-all.sh

A bash shell script wrapper for git-archive that archives a git superproject and its submodules, if it has any.
214 stars 86 forks source link

Set $IFS using printf #33

Closed cheungpat closed 8 years ago

cheungpat commented 8 years ago

The existing line of code makes bash to treat n and t as separator, which cause echo $TARCMD to print /usr/bi / tar.

jrfoell commented 8 years ago

Just want to confirm that this pull request fixes the IFS set (at least on OSX). It otherwise treats n and t as separators, doing unexpected things to any paths/commands that contain those characters. Running git-archive-all.sh without this patch on OSX by itself will yield: ./git-archive-all.sh: line 203: /usr/local/bi: No such file or directory Note this missing n as it's turning a$TARCMD of /usr/local/bin/tar into /usr/local/bi / ar