From: Michal Hocko <mstsxfx@gm...> - 2009-04-07 07:56:51
[Answering to devel mailing list]
On Tue, Apr 07, 2009 at 01:13:17AM +0200, Martin Petricek wrote:
>> Finally this patch unifies the way on how we specify script interpreter when
>> the licence header is present. The best way how to do it is to add
>> #!PATH_TO_INTERPRETER right after the header because we will usually change
> This won't work, #!PATH_TO_INTERPRETER _MUST_ be on the first line, or
> it won't be recognized.
> (for example trans-qs.pl stopped working, because bash is used on that
> instead of perl)
You are right. It has to be at the very beginning of the file. Thanks
for spotting. I have tried to run some scripts but they were all shell
scripts so I haven't noticed the breakage.
> If mass patch does not cope with this, we have to use another tool.
mass_patch is only simply wrapper for patch, but I have just tried and
it copes well with different first line(s). It just gets to fuzzy 2 mode
> I've fixed all shell files to have the path to interpreter on first line.
> I've committed it along with the flattener GUI.
Thanks for doing this!
> Martin Petricek