[PATCH] doc: correct the build steps for open_posix_testsuite

[PATCH] doc: correct the build steps for open_posix_testsuite

[xiubli]

From: Xiubo Li <xiubli@redhat.com>

'./configure' is needed just before generating the Makefiles.

Signed-off-by: Xiubo Li <xiubli@redhat.com>
---
 doc/users/quick_start.rst | 1 +
 1 file changed, 1 insertion(+)

diff --git a/doc/users/quick_start.rst b/doc/users/quick_start.rst
index 1581b1f0c..e80c1c244 100644
--- a/doc/users/quick_start.rst
+++ b/doc/users/quick_start.rst
@@ -54,6 +54,7 @@ generated first:
 .. code-block:: console
 
    $ cd testcases/open_posix_testsuite/
+   $ ./configure
    $ make generate-makefiles
    $ cd conformance/interfaces/foo
    $ make
-- 
2.46.0

Re: [LTP] [PATCH] doc: correct the build steps for open_posix_testsuite

[Cyril Hrubis]

Hi!
> './configure' is needed just before generating the Makefiles.
> 
> Signed-off-by: Xiubo Li <xiubli@redhat.com>
> ---
>  doc/users/quick_start.rst | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/doc/users/quick_start.rst b/doc/users/quick_start.rst
> index 1581b1f0c..e80c1c244 100644
> --- a/doc/users/quick_start.rst
> +++ b/doc/users/quick_start.rst
> @@ -54,6 +54,7 @@ generated first:
>  .. code-block:: console
>  
>     $ cd testcases/open_posix_testsuite/
> +   $ ./configure

That should be before the 'cd testcase/...' otherwise good catch, this
is clearly missing there.

>     $ make generate-makefiles
>     $ cd conformance/interfaces/foo
>     $ make
> -- 
> 2.46.0
> 
> 
> -- 
> Mailing list info: https://lists.linux.it/listinfo/ltp

-- 
Cyril Hrubis
chrubis@suse.cz

Re: [LTP] [PATCH] doc: correct the build steps for open_posix_testsuite

[Xiubo Li]

On 11/29/24 18:27, Cyril Hrubis wrote:
> Hi!
>> './configure' is needed just before generating the Makefiles.
>>
>> Signed-off-by: Xiubo Li <xiubli@redhat.com>
>> ---
>>   doc/users/quick_start.rst | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/doc/users/quick_start.rst b/doc/users/quick_start.rst
>> index 1581b1f0c..e80c1c244 100644
>> --- a/doc/users/quick_start.rst
>> +++ b/doc/users/quick_start.rst
>> @@ -54,6 +54,7 @@ generated first:
>>   .. code-block:: console
>>   
>>      $ cd testcases/open_posix_testsuite/
>> +   $ ./configure
> That should be before the 'cd testcase/...' otherwise good catch, this
> is clearly missing there.

Good catch.

Fixed it.

Thanks!


>>      $ make generate-makefiles
>>      $ cd conformance/interfaces/foo
>>      $ make
>> -- 
>> 2.46.0
>>
>>
>> -- 
>> Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH] doc: correct the build steps for open_posix_testsuite

[Cyril Hrubis]

Hi!
> Wait.
> 
> I check it again. I think we should do it under 
> 'testcases/open_posix_testsuite/' too ? For me it doesn't work only do 
> this just before 'cd testcase/...'

Ah it's a bit more complicated that I remmebered. There are two ways how
to compile the open posix testsuite.

The top level configure script configures the open posix testsuite, but
you have to pass the --with-open-posix-testsuite flag to it.

Or you can run the open posix configure from the open posix directory.

-- 
Cyril Hrubis
chrubis@suse.cz

Re: [LTP] [PATCH] doc: correct the build steps for open_posix_testsuite

[Xiubo Li]

On 12/2/24 18:43, Cyril Hrubis wrote:
> Hi!
>> Wait.
>>
>> I check it again. I think we should do it under
>> 'testcases/open_posix_testsuite/' too ? For me it doesn't work only do
>> this just before 'cd testcase/...'
> Ah it's a bit more complicated that I remmebered. There are two ways how
> to compile the open posix testsuite.
>
> The top level configure script configures the open posix testsuite, but
> you have to pass the --with-open-posix-testsuite flag to it.
>
> Or you can run the open posix configure from the open posix directory.
>
Yeah, this looks better.

Does it make sense to fix the doc about this ? If so I will send the V3.

Thanks

- Xiubo

Re: [LTP] [PATCH] doc: correct the build steps for open_posix_testsuite

[Cyril Hrubis]

Hi!
> > The top level configure script configures the open posix testsuite, but
> > you have to pass the --with-open-posix-testsuite flag to it.
> >
> > Or you can run the open posix configure from the open posix directory.
> >
> Yeah, this looks better.
> 
> Does it make sense to fix the doc about this ? If so I will send the V3.

Yes please (and sorry for the delayed response).

-- 
Cyril Hrubis
chrubis@suse.cz

Re: [LTP] [PATCH] doc: correct the build steps for open_posix_testsuite

[Xiubo Li]

On 12/10/24 20:15, Cyril Hrubis wrote:
> Hi!
>>> The top level configure script configures the open posix testsuite, but
>>> you have to pass the --with-open-posix-testsuite flag to it.
>>>
>>> Or you can run the open posix configure from the open posix directory.
>>>
>> Yeah, this looks better.
>>
>> Does it make sense to fix the doc about this ? If so I will send the V3.
> Yes please (and sorry for the delayed response).
>
Done. Thanks!