Re: [PATCH spice-gtk] build: Make usb-acl-helper test work under Meson

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 09/03/2018 01:34 PM, Frediano Ziglio wrote:
> Meson waits for pipes (stdout and stderr) to be not readable but
> our mocking helper does not close these handles causing Meson
> to give some problems.
> 

Can you elaborate on the "some problems"? Also this patch looks more
like a workaround from our side for a real bug in meson?

> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  tests/usb-acl-helper.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/tests/usb-acl-helper.c b/tests/usb-acl-helper.c
> index 06f8036..310bdaf 100644
> --- a/tests/usb-acl-helper.c
> +++ b/tests/usb-acl-helper.c
> @@ -15,6 +15,7 @@
>     License along with this library; if not, see <http://www.gnu.org/licenses/>.
>  */
>  
> +#include <stdio.h>
>  #include <glib.h>
>  #include "usb-acl-helper.h"
>  
> @@ -198,6 +199,13 @@ static void test_acl_helper_no_response(Fixture *fixture, gconstpointer user_dat
>  
>  int main(int argc, char* argv[])
>  {
> +    /* Meson wait for stderr to be not readable so make sure the
> +     * helper does not keep the pipe open (this happens with current
> +     * mock helper) */
> +    if (freopen("usb-acl-helper-error.log", "w", stderr) == NULL) {
> +        return 1;
> +    }
> +
>      g_test_init(&argc, &argv, NULL);
>  
>      g_test_add("/usb-acl-helper/success", Fixture, NULL,
> 


-- 
Eduardo de Barros Lima (Etrunko)
Software Engineer - RedHat
etrunko@xxxxxxxxxx
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]