From: Jonathan Nieder <jrnieder@gmail.com>
To: git@vger.kernel.org
Cc: Paul Mackerras <paulus@samba.org>,
Pat Thoyts <patthoyts@users.sourceforge.net>,
Clemens Buchacher <drizzd@aon.at>,
Marc Branchaud <marcnarc@xiplink.com>
Subject: [PATCH 0/9] War on #! lines in shell libraries
Date: Mon, 25 Nov 2013 12:51:19 -0800 [thread overview]
Message-ID: <20131125205119.GQ4212@google.com> (raw)
Hi,
This is an old series that I just wanted to flush out of my working
directory. It does two things:
On one hand, it replaces the line
#!/bin/sh
at the start of shell libraries with a simple comment
# Shell library for <etc, explaining how it is used>
to avoid confusing readers into thinking the shell library is going to
be run directly with /bin/sh. And on the other hand, it sets the
executable bit on actual scripts that are run directly with /bin/sh.
(Likewise for some perl scripts, too.)
The heart of the series has been well tested in the context of Debian
git for a couple of years. Thanks to list denizens for the
improvements last time I brought it up[1]. Hopefully this version is
more sensible.
Anyway, hopefully it can provide some amusement.
Thanks,
Jonathan Nieder (9):
mark Windows build scripts executable
mark perl test scripts executable
mark contributed hooks executable
contrib: remove git-p4import
gitk: chmod +x po2msg
git-gui: chmod +x po2msg, windows/git-gui.sh
test: make FILEMODE a lazy prereq
test: replace shebangs with descriptions in shell libraries
remove #!interpreter line from shell libraries
[1] http://thread.gmane.org/gmane.comp.version-control.git/192582
compat/vcbuild/scripts/clink.pl | 0
compat/vcbuild/scripts/lib.pl | 0
contrib/buildsystems/engine.pl | 0
contrib/buildsystems/generate | 0
contrib/buildsystems/parse.pl | 0
contrib/completion/git-completion.bash | 2 -
contrib/completion/git-completion.tcsh | 2 -
contrib/hooks/post-receive-email | 1 -
contrib/hooks/pre-auto-gc-battery | 1 -
contrib/hooks/setgitperms.perl | 0
contrib/hooks/update-paranoid | 0
contrib/p4import/README | 1 -
contrib/p4import/git-p4import.py | 365 ---------------------------------
contrib/p4import/git-p4import.txt | 167 ---------------
git-gui/po/po2msg.sh | 0
git-gui/windows/git-gui.sh | 0
git-mergetool--lib.sh | 3 +-
git-parse-remote.sh | 4 +-
git-rebase--am.sh | 3 +-
git-rebase--interactive.sh | 9 +-
git-rebase--merge.sh | 4 +-
git-sh-i18n.sh | 5 +-
git-sh-setup.sh | 9 +-
gitk-git/po/po2msg.sh | 0
t/Git-SVN/00compile.t | 0
t/Git-SVN/Utils/add_path_to_url.t | 0
t/Git-SVN/Utils/can_compress.t | 0
t/Git-SVN/Utils/canonicalize_url.t | 0
t/Git-SVN/Utils/collapse_dotdot.t | 0
t/Git-SVN/Utils/fatal.t | 0
t/Git-SVN/Utils/join_paths.t | 0
t/gitweb-lib.sh | 3 +-
t/lib-bash.sh | 7 +-
t/lib-cvs.sh | 2 +-
t/lib-diff-alternative.sh | 3 +-
t/lib-gettext.sh | 3 +-
t/lib-git-daemon.sh | 18 +-
t/lib-httpd.sh | 29 ++-
t/lib-pack.sh | 2 -
t/lib-pager.sh | 2 +-
t/lib-prereq-FILEMODE.sh | 11 -
t/lib-read-tree.sh | 2 -
t/lib-rebase.sh | 2 +-
t/lib-terminal.sh | 2 +-
t/perf/perf-lib.sh | 4 +-
t/t0202/test.pl | 0
t/t3701-add-interactive.sh | 1 -
t/t4102-apply-rename.sh | 1 -
t/t4120-apply-popt.sh | 1 -
t/t4129-apply-samemode.sh | 1 -
t/t6031-merge-recursive.sh | 1 -
t/t9150/make-svk-dump | 0
t/t9151/make-svnmerge-dump | 0
t/t9200-git-cvsexportcommit.sh | 1 -
t/test-lib-functions.sh | 3 +-
t/test-lib.sh | 12 +-
56 files changed, 87 insertions(+), 600 deletions(-)
mode change 100644 => 100755 compat/vcbuild/scripts/clink.pl
mode change 100644 => 100755 compat/vcbuild/scripts/lib.pl
mode change 100644 => 100755 contrib/buildsystems/engine.pl
mode change 100644 => 100755 contrib/buildsystems/generate
mode change 100644 => 100755 contrib/buildsystems/parse.pl
mode change 100644 => 100755 contrib/hooks/pre-auto-gc-battery
mode change 100644 => 100755 contrib/hooks/setgitperms.perl
mode change 100644 => 100755 contrib/hooks/update-paranoid
delete mode 100644 contrib/p4import/README
delete mode 100644 contrib/p4import/git-p4import.py
delete mode 100644 contrib/p4import/git-p4import.txt
mode change 100644 => 100755 git-gui/po/po2msg.sh
mode change 100644 => 100755 git-gui/windows/git-gui.sh
mode change 100644 => 100755 gitk-git/po/po2msg.sh
mode change 100644 => 100755 t/Git-SVN/00compile.t
mode change 100644 => 100755 t/Git-SVN/Utils/add_path_to_url.t
mode change 100644 => 100755 t/Git-SVN/Utils/can_compress.t
mode change 100644 => 100755 t/Git-SVN/Utils/canonicalize_url.t
mode change 100644 => 100755 t/Git-SVN/Utils/collapse_dotdot.t
mode change 100644 => 100755 t/Git-SVN/Utils/fatal.t
mode change 100644 => 100755 t/Git-SVN/Utils/join_paths.t
delete mode 100644 t/lib-prereq-FILEMODE.sh
mode change 100644 => 100755 t/t0202/test.pl
mode change 100644 => 100755 t/t9150/make-svk-dump
mode change 100644 => 100755 t/t9151/make-svnmerge-dump
--
1.8.4.1
next reply other threads:[~2013-11-25 20:51 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-25 20:51 Jonathan Nieder [this message]
2013-11-25 20:52 ` [PATCH 1/9] mark Windows build scripts executable Jonathan Nieder
2013-11-25 20:53 ` [PATCH 2/9] mark perl test " Jonathan Nieder
2013-11-25 20:55 ` [PATCH 3/9] mark contributed hooks executable Jonathan Nieder
2013-11-25 20:58 ` [PATCH 4/9] contrib: remove git-p4import Jonathan Nieder
2013-11-26 12:31 ` Pete Wyckoff
2013-11-25 21:00 ` [PATCH 5/9 gitk] gitk: chmod +x po2msg Jonathan Nieder
2013-11-25 21:01 ` [PATCH 6/9 git-gui] git-gui: chmod +x po2msg, windows/git-gui.sh Jonathan Nieder
2013-11-25 21:02 ` [PATCH 7/9] test: make FILEMODE a lazy prereq Jonathan Nieder
2013-11-25 21:03 ` [PATCH 8/9] test: replace shebangs with descriptions in shell libraries Jonathan Nieder
2013-11-26 5:18 ` Eric Sunshine
2013-11-26 21:39 ` [PATCH 8/9 v2] " Jonathan Nieder
2013-11-25 21:03 ` [PATCH 9/9] remove #!interpreter line from " Jonathan Nieder
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131125205119.GQ4212@google.com \
--to=jrnieder@gmail.com \
--cc=drizzd@aon.at \
--cc=git@vger.kernel.org \
--cc=marcnarc@xiplink.com \
--cc=patthoyts@users.sourceforge.net \
--cc=paulus@samba.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.