darcs

Message4396

Author tux_rocker
Recipients beschmi, dagit, droundy, jaredj, kowey, tommy, tux_rocker, twb
Date 2008-04-30.10:06:01
Issue Issue538 wish: trackdown --set-scripts-executable
Content
> 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?
History
Date User Action Args
2008-04-30 10:06:01tux_rockercreate
2008-04-30 10:06:03tux_rockerlinkissue538 messages
2008-04-30 10:06:03tux_rockersetrecipients: + tux_rocker, droundy, tommy, beschmi, kowey, dagit, trentbuck, twb, jaredj
2008-04-30 10:06:03tux_rockersetmessageid: <1209549963.21.0.866959227856.issue538@darcs.net>
2008-09-25 06:25:22adminsetrecipients: - trentbuck