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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).