-
Notifications
You must be signed in to change notification settings - Fork 123
shell recorder: safe working directory #1982
Comments
This way we would also get rid of the leftovers files "stderr" and stdout" which are left in the source repo from time to time. |
@sanssecours wrote:
Could you please elaborate? TESTDIR points to a file? |
The variable saves the directory of the file under test. For example, if your file under test is |
I mark this issue stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping the issue by writing a message here or create a new issue with the remainder of this issue. |
I closed this issue now because it has been inactive for more than one year. If I closed it by mistake, please do not hesitate to reopen it or create a new issue with the remainder of this issue. |
Tests should definitely be more isolated (e.g. like proposed here somewhere under TMPDIR) and never modify other existing (configuration) files. |
I mark this issue stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping the issue by writing a message here or create a new issue with the remainder of this issue. |
I mark this stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping by writing a message here or create a new issue with the remainder of this issue. |
I closed this now because it has been inactive for more than one year. If I closed it by mistake, please do not hesitate to reopen it or create a new issue with the remainder of this issue. |
It might be a good idea to create a temporary directory below TMPDIR and set HOME and PWD to this path. Potential leftovers and programming errors would only affect this directory.
This is already done for the cframework test cases and works there quite well.
EDIT:
Furthermore, we need variables, that:
see also #1955 and #1981 for rationale
The text was updated successfully, but these errors were encountered: