2020-04-24 23:14:42

by Tedd Ho-Jeong An

[permalink] [raw]
Subject: [PATCH] Test patch: build fail

DO NOT MERGE
---
tools/btpclient.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/btpclient.c b/tools/btpclient.c
index f9c693056..04c83f6ae 100644
--- a/tools/btpclient.c
+++ b/tools/btpclient.c
@@ -3215,7 +3215,7 @@ int main(int argc, char *argv[])

l_free(socket_path);

- l_main_exit();
+ l_main_exit()

return EXIT_SUCCESS;
}
--
2.25.3


2020-04-25 02:37:27

by bluez.test.bot

[permalink] [raw]
Subject: RE: Test patch: build fail


This is automated email and please do not replay to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
While we are preparing for reviewing the patches, we found the following
issue/warning.


Test Result:
Checkbuild Failed

Patch Series:
Test patch: build fail


Outputs:
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
tools/btpclient.c: In function ‘main’:
tools/btpclient.c:3218:15: error: expected ‘;’ before ‘return’
3218 | l_main_exit()
| ^
| ;
3219 |
3220 | return EXIT_SUCCESS;
| ~~~~~~
make[1]: *** [Makefile:6791: tools/btpclient.o] Error 1
make: *** [Makefile:4010: all] Error 2



---
Regards,
Linux Bluetooth

2020-04-27 16:31:20

by Luiz Augusto von Dentz

[permalink] [raw]
Subject: Re: Test patch: build fail

Hi Tedd,

On Fri, Apr 24, 2020 at 7:38 PM <[email protected]> wrote:
>
>
> This is automated email and please do not replay to this email!
>
> Dear submitter,
>
> Thank you for submitting the patches to the linux bluetooth mailing list.
> While we are preparing for reviewing the patches, we found the following
> issue/warning.
>
>
> Test Result:
> Checkbuild Failed
>
> Patch Series:
> Test patch: build fail
>
>
> Outputs:
> ar: `u' modifier ignored since `D' is the default (see `U')
> ar: `u' modifier ignored since `D' is the default (see `U')
> ar: `u' modifier ignored since `D' is the default (see `U')
> ar: `u' modifier ignored since `D' is the default (see `U')
> ar: `u' modifier ignored since `D' is the default (see `U')
> tools/btpclient.c: In function ‘main’:
> tools/btpclient.c:3218:15: error: expected ‘;’ before ‘return’
> 3218 | l_main_exit()
> | ^
> | ;
> 3219 |
> 3220 | return EXIT_SUCCESS;
> | ~~~~~~
> make[1]: *** [Makefile:6791: tools/btpclient.o] Error 1
> make: *** [Makefile:4010: all] Error 2

Nice, really cool that we finally have a CI working.

--
Luiz Augusto von Dentz