-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
workingDir should not be an @InputDirectory #61
Comments
I just ran into seemingly exactly this issue while integrating All the detailed information can be found in the 2 issues referenced above. @psxpaul is there any intention on changing this? If so any preferred way? I would be willing to create a PR for it. |
Wanted to give my +1 since this is causing the build to require a large number of explicit dependencies to be set for the task when in reality it is just starting Xvfb and doesn't depend on any previous tasks.
|
I question the wisdom of the annotation in AbstractExecTask to declare
workingDir
as an input directory. This should be left to the consumers that use the tasks to declare additional file/directory inputs. TheworkingDir
is likely to be undesirable and most often too unstable to be an incremental execution up-to-date criterion.Can we remove the @InputDirectory from the code and let the users choose what might constitute a valid input file or directory?
The text was updated successfully, but these errors were encountered: