> Did you mean 'what the semantics of --set-scripts-executable for
> /trackdown/
> can and should be'?
Possibly. But the way I made it work for unhashed repositories, I enabled
--set-scripts-executable for record. That way, scripts get stored
executably in the pristine and will end up as executables in the test
working directory. This is, I presume, why the test works if you get
--set-scripts-executable.
And of course, if you want the test to work for trackdown, you also want
it to work for record.
> If so, perhaps we can discuss this. I get the impression that the
> expected
> behaviour would be for all scripts in the temporary working dir created by
> trackdown to be +x at all times. For instance, if the trackdown decides
> it
> needs to unapply a patch, it should ensure that any scripts created in the
> process are executable.
The only case I can see, in which a file needs to be set executable during
the trackdown, is when a patch that removes a script is unapplied. In that
case, doesn't darcs eventually call the apply function for a hunk patch
that sets the file executable if it starts with '#!' and
--set-scripts-executable is given? |