On 8/30/24 10:29, Dev Jain wrote:
On 8/27/24 17:16, Dev Jain wrote:
On 8/27/24 17:14, Shuah Khan wrote:
On 8/22/24 06:14, Dev Jain wrote:
Rename sigaltstack to generic signal directory, to allow adding more signal tests in the future.
Sorry - I think I mentioned I don't like this test renamed. Why are you sending this rename still included in the patch series?
I am not renaming the test, just the directory. The directory name is changed to signal, and I have retained the name of the test - sas.c.
Gentle ping: I guess there was a misunderstanding; in v5, I was also changing the name of the test, to which you objected, and I agreed. But, we need to change the name of the directory since the new test has no relation to the current directory name, "sigaltstack". The patch description explains that the directory should be generically named.
Right. You are no longer changing the test name. You are still changing the directory name. The problem I mentioned stays the same. Any fixes to the existing tests in this directory can no longer auto applied to stables releases.
Other than the desire to rename the directory to generic, what other value does this change bring?
thanks, -- Shuah