| Message ID | 20230218004555.415122-1-hj.tedd.an@gmail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [BlueZ] configure: Check ell path | expand |
| Context | Check | Description |
|---|---|---|
| tedd_an/pre-ci_am | success | Success |
| tedd_an/CheckPatch | success | CheckPatch PASS |
| tedd_an/GitLint | success | Gitlint PASS |
| tedd_an/BuildEll | success | Build ELL PASS |
| tedd_an/BluezMake | success | Bluez Make PASS |
| tedd_an/MakeCheck | success | Bluez Make Check PASS |
| tedd_an/MakeDistcheck | fail | Make Distcheck FAIL: configure: error: ELL source is required at ../ell make: *** [Makefile:11618: distcheck] Error 1 |
| tedd_an/CheckValgrind | success | Check Valgrind PASS |
| tedd_an/CheckSmatch | success | CheckSparse PASS |
| tedd_an/bluezmakeextell | success | Make External ELL PASS |
| tedd_an/IncrementalBuild | success | Incremental Build PASS |
| tedd_an/ScanBuild | success | Scan Build PASS |
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=723140 ---Test result--- Test Summary: CheckPatch PASS 0.36 seconds GitLint PASS 0.22 seconds BuildEll PASS 32.23 seconds BluezMake PASS 1068.41 seconds MakeCheck PASS 12.67 seconds MakeDistcheck FAIL 30.47 seconds CheckValgrind PASS 292.83 seconds CheckSmatch PASS 400.75 seconds bluezmakeextell PASS 118.62 seconds IncrementalBuild PASS 897.90 seconds ScanBuild PASS 1283.01 seconds Details ############################## Test: MakeDistcheck - FAIL Desc: Run Bluez Make Distcheck Output: configure: error: ELL source is required at ../ell make: *** [Makefile:11618: distcheck] Error 1 --- Regards, Linux Bluetooth
diff --git a/configure.ac b/configure.ac index f9f0faf57..6e76133c2 100644 --- a/configure.ac +++ b/configure.ac @@ -289,6 +289,10 @@ if (test "${enable_external_ell}" = "yes"); then AC_SUBST(ELL_CFLAGS) AC_SUBST(ELL_LIBS) fi +if (test "${enable_external_ell}" != "yes"); then + AC_CHECK_FILE(../ell, dummy=yes, + AC_MSG_ERROR(ELL source is required at ../ell)) +fi AM_CONDITIONAL(EXTERNAL_ELL, test "${enable_external_ell}" = "yes" || (test "${enable_btpclient}" != "yes" && test "${enable_mesh}" != "yes"))
From: Tedd Ho-Jeong An <tedd.an@intel.com> If the 'enable-external-ell' is not specified in the configure parameter, the build system assumes that the ELL source is located same level where the bluez source is cloned. But the configure doens't check the folder and user will get the build error while building the source. This patch checks if the ELL source path if the 'enable-external-ell' flag is not set and throws an error if the ELL doesn't exist. --- configure.ac | 4 ++++ 1 file changed, 4 insertions(+)