RE: RFC: convert tester tools to use ELL

[bluez.test.bot@gmail.com]

[-- Attachment #1: Type: text/plain, Size: 4330 bytes --]

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

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=477641

---Test result---

Test Summary:
CheckPatch                    FAIL      6.19 seconds
GitLint                       FAIL      1.41 seconds
Prep - Setup ELL              PASS      50.12 seconds
Build - Prep                  PASS      0.14 seconds
Build - Configure             PASS      8.54 seconds
Build - Make                  PASS      208.48 seconds
Make Check                    PASS      9.62 seconds
Make Dist                     PASS      12.47 seconds
Make Dist - Configure         PASS      5.29 seconds
Make Dist - Make              PASS      86.32 seconds
Build w/ext ELL - Configure   PASS      8.61 seconds
Build w/ext ELL - Make        FAIL      68.12 seconds

Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script with rule in .checkpatch.conf
Output:
shared/bttester: tester framework wrapper to use ELL
WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __packed over __attribute__((packed))
#78: FILE: src/shared/bttester.c:51:
+} __attribute__((packed));

WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __packed over __attribute__((packed))
#83: FILE: src/shared/bttester.c:56:
+} __attribute__((packed));

WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __printf(1, 2) over __attribute__((format(printf, 1, 2)))
#335: FILE: src/shared/bttester.h:22:
+				__attribute__((format(printf, 1, 2)));

WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __printf(1, 2) over __attribute__((format(printf, 1, 2)))
#337: FILE: src/shared/bttester.h:24:
+				__attribute__((format(printf, 1, 2)));

WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __printf(1, 2) over __attribute__((format(printf, 1, 2)))
#339: FILE: src/shared/bttester.h:26:
+				__attribute__((format(printf, 1, 2)));

- total: 0 errors, 5 warnings, 318 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

"[PATCH] shared/bttester: tester framework wrapper to use ELL" has style problems, please review.

NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.


##############################
Test: GitLint - FAIL
Desc: Run gitlint with rule in .gitlint
Output:
shared/bttester: tester framework wrapper to use ELL
3: B6 Body message is missing


##############################
Test: Prep - Setup ELL - PASS
Desc: Clone, build, and install ELL

##############################
Test: Build - Prep - PASS
Desc: Prepare environment for build

##############################
Test: Build - Configure - PASS
Desc: Configure the BlueZ source tree

##############################
Test: Build - Make - PASS
Desc: Build the BlueZ source tree

##############################
Test: Make Check - PASS
Desc: Run 'make check'

##############################
Test: Make Dist - PASS
Desc: Run 'make dist' and build the distribution tarball

##############################
Test: Make Dist - Configure - PASS
Desc: Configure the source from distribution tarball

##############################
Test: Make Dist - Make - PASS
Desc: Build the source from distribution tarball

##############################
Test: Build w/ext ELL - Configure - PASS
Desc: Configure BlueZ source with '--enable-external-ell' configuration

##############################
Test: Build w/ext ELL - Make - FAIL
Desc: Build BlueZ source with '--enable-external-ell' configuration
Output:
/usr/bin/ld: src/.libs/libshared-ell.a(bttester.o): in function `parse_options':
/github/workspace/src2/src/shared/bttester.c:206: undefined reference to `__stop___ell_debug'
/usr/bin/ld: /github/workspace/src2/src/shared/bttester.c:206: undefined reference to `__start___ell_debug'
collect2: error: ld returned 1 exit status
make[1]: *** [Makefile:5959: tools/mgmt-tester] Error 1
make: *** [Makefile:4082: all] Error 2




---
Regards,
Linux Bluetooth