[LTP] [PATCH] README: Add missing configure in openposix step
xuyang2018.jy@fujitsu.com
xuyang2018.jy@fujitsu.com
Mon Sep 19 08:06:04 CEST 2022
Hi Joerg
> Hi,
>
> sorry for the late reply, I was on vacation.
>
> Am 9/2/2022 um 9:31 AM schrieb xuyang2018.jy@fujitsu.com:
>> Hi Petr
>>
>>> Hi Xu,
>>>
>>> [ Cc Joerg ]
>>>
>>>> Since 8071ba7("openposix: Setup autoconf and fix installation layout")
>>>> , we need to use configure firstly.
>>>
>>> nit: there should be Fixes: below:
>>> Fixes: 8071ba7 ("openposix: Setup autoconf and fix installation layout")
>>>
>>> Then you can use just 8071ba7 in the text.
>>>
>>>> Signed-off-by: Yang Xu <xuyang2018.jy@fujitsu.com>
>>>> ---
>>>> README.md | 6 ++++--
>>>> 1 file changed, 4 insertions(+), 2 deletions(-)
>>>
>>>> diff --git a/README.md b/README.md
>>>> index d45d1ee44..d0ca5f4d4 100644
>>>> --- a/README.md
>>>> +++ b/README.md
>>>> @@ -79,11 +79,13 @@ $ cd ../commands/foo
>>>> $ PATH=$PATH:$PWD:$PWD/../../lib/ ./foo01.sh
>>>> ```
>>>
>>>> -Open Posix Testsuite has it's own build system which needs Makefiles to be
>>>> -generated first, then compilation should work in subdirectories as well.
>>>> +Open Posix Testsuite has it's own configure and build system which needs
>>>> +Makefiles to be generated first, then compilation should work in subdirectories
>>>> +as well.
>>>
>>>> ```
>>>> $ cd testcases/open_posix_testsuite/
>>>
>>> You miss:
>>> $ make autotools
>>> as this creates configure
>>>
>>>> +$ ./configure
>>> But if you run make autotools && ./configure in the top level, none of these two
>>> is needed to be run in open posix directory.
>>
>> I have called them on top directory, but configure still be needed in
>> open posix directory.
> Did you run configure with --with-open-posix-testsuite? Otherwise
> configure is not executed in the open posix directory.
Oh, yes, I don't use this option. I only see the documentation said we
can use ./configure and make instead of make all in the openposix
direcotry. But miss this --with-open-posix-testsuite option .
I think we should mention this option in README ie
./configure --with-open-posix-testsuite --with-realtime-testsuite
diff --git a/README.md b/README.md
index d45d1ee44..7da393e37 100644
--- a/README.md
+++ b/README.md
@@ -51,6 +51,13 @@ $ make autotools
$ ./configure
```
+If you want to include open posix testsuite or realtime testsuite, the
chances are
+the following will work:
+
+```
+$ ./configure --with-open-posix-testsuite --with-realtime-testsuite
+```
+
Now you can continue either with compiling and r
Best Regards
Yang Xu
>
> Joerg
More information about the ltp
mailing list