[LTP] [PATCH] README: Add missing configure in openposix step
Petr Vorel
pvorel@suse.cz
Wed Sep 21 13:26: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
I guess all people who has been using open posix by now knows
--with-open-posix-testsuite is needed. I mean, it does not harm to document it,
but there are other options. I suppose people run ./configure --help on project
they don't know.
Kind regards,
Petr
More information about the ltp
mailing list