On Fri, Apr 30, 2010 at 12:27 PM, Henry Yei <[email protected]> wrote:
> Speaking of the utimesat test and prerequisites, our test systems
> don't have a sudoers file by default, so we have an internal patch to
> utimesat to create a default one if non exist and remove it after the
> test is done. Would that be of interest to LTP?
>
> On Tue, Apr 27, 2010 at 11:27 PM, Caspar Zhang <[email protected]> wrote:
>> On Wed, Apr 28, 2010 at 2:04 PM, Garrett Cooper <[email protected]> wrote:
>>> The first item is different from the other two. The first one ignores the
>>> signal (SIG_IGN), whereas the latter two cases reset the handler to the
>>> default one (SIG_DFL). I prefer the former format, because otherwise the
>>> signal handlers become reentrable on accident.
>>
>> I see. Thank you.
>>
>> The final patch ;-)

If it:

1. Works in all cases, i.e. doesn't use version specific constructs
for sudo (which I haven't seen thus far, but just to be safe).
2. Is properly reverted when the test is done (which includes the
following scenarios):
    a. File already exists. Backup the old file, revert it when the
test is completed (regardless of whether or not the test passed or the
test failed properly [*]).
    b. File doesn't exist. Nuke the file after the test is done.

    ... sure.
Thanks,
-Garrett

[*] SIGKILL or SIGSTOP can't be avoided, so technically it's a best effort.

------------------------------------------------------------------------------
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to