Skip to content

Commit 8aee90a

Browse files
committed
compiler warning fixes and cmake support
1 parent 1cfb398 commit 8aee90a

12 files changed

Lines changed: 69 additions & 11 deletions

File tree

.gitignore

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,4 +10,5 @@ mazda/installer/config/androidauto/data_persist/dev/bin/headunit
1010
*.creator.user
1111
bin/
1212
*/version\.h
13-
.vscode
13+
.vscode
14+
build

cmake/deps.cmake

Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,54 @@
1+
2+
find_package(Threads REQUIRED)
3+
find_package(OpenSSL REQUIRED)
4+
find_package(Protobuf REQUIRED)
5+
find_package(SDL2 REQUIRED)
6+
find_package(ALSA REQUIRED)
7+
find_package(X11 REQUIRED)
8+
9+
find_package(PkgConfig REQUIRED)
10+
if(PKG_CONFIG_EXECUTABLE)
11+
12+
pkg_check_modules(GST REQUIRED gstreamer-1.0
13+
gstreamer-video-1.0
14+
gstreamer-app-1.0
15+
gstreamer-audio-1.0
16+
gstreamer-codecparsers-1.0)
17+
18+
pkg_check_modules(USB REQUIRED libusb-1.0)
19+
pkg_check_modules(GTK3 REQUIRED gtk+-3.0)
20+
pkg_check_modules(UNWIND REQUIRED libunwind)
21+
pkg_check_modules(UDEV REQUIRED libudev)
22+
endif()
23+
24+
find_package(Git)
25+
if(GIT_EXECUTABLE)
26+
execute_process(
27+
WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"
28+
COMMAND "${GIT_EXECUTABLE}" describe --tags --always
29+
OUTPUT_VARIABLE GIT_HASH
30+
OUTPUT_STRIP_TRAILING_WHITESPACE
31+
)
32+
set(BUILD_HASH ${GIT_HASH})
33+
34+
if (NOT REPRODUCIBLE)
35+
execute_process(
36+
WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"
37+
COMMAND "whoami"
38+
OUTPUT_VARIABLE GIT_USER
39+
OUTPUT_STRIP_TRAILING_WHITESPACE
40+
)
41+
execute_process(
42+
WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"
43+
COMMAND "hostname"
44+
OUTPUT_VARIABLE GIT_HOST
45+
OUTPUT_STRIP_TRAILING_WHITESPACE
46+
)
47+
string(REGEX REPLACE "([^\\])[\\]([^\\])" "\\1\\\\\\\\\\2" GIT_USER ${GIT_USER})
48+
set(BUILD_HASH ${GIT_USER}@${GIT_HOST}-${GIT_HASH})
49+
endif()
50+
51+
message("Git commit hash: ${BUILD_HASH}")
52+
53+
configure_file(cmake/version.in.h ${CMAKE_CURRENT_BINARY_DIR}/version.h @ONLY)
54+
endif()

cmake/version.in.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
2+
#define HEADUNIT_VERSION "@BUILD_HASH@"

common/audio.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
#include <glib.h>
44
#include <stdio.h>
5-
#include <asoundlib.h>
5+
#include <alsa/asoundlib.h>
66
#include <thread>
77

88
#include "hu_uti.h"

hu/hu_aad.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -234,13 +234,13 @@ Channel specified for each service:
234234

235235
if (log_dmp) {
236236
//logd ("iaad_dmp_arry n: %d num: %d alen: %ld", n, num, alen); // Dump raw array
237-
char str_buf [256] = {0};
237+
char str_buf [255] = {0};
238238
int ctr = 0;
239239
for (ctr = 0; ctr < n - 1; ctr ++)
240240
strncat (str_buf, " ", sizeof(str_buf) - strlen(str_buf) - 1);
241241

242242
char str_buf2 [256] = {0};
243-
snprintf (str_buf2, sizeof (str_buf), "%s%1.1u", str_buf, num); // Dump raw array
243+
snprintf (str_buf2, sizeof (str_buf2), "%s%1.1u", str_buf, num); // Dump raw array
244244
hex_dump (str_buf2, 16, buf, alen);
245245
}
246246

hu/hu_aap.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1166,6 +1166,7 @@
11661166
delete ptr;
11671167
loge("hu_queue_command error %d", ret);
11681168
}
1169+
return ret;
11691170
}
11701171

11711172
int HUServer::hu_aap_shutdown()

hu/hu_ssl.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@
138138
else
139139
logd ("BIO_free(pkey_bio) ret: %d", ret);
140140

141-
hu_ssl_method = (SSL_METHOD *) TLSv1_2_client_method ();
141+
hu_ssl_method = (SSL_METHOD *)SSLv23_client_method();
142142
if (hu_ssl_method == NULL) {
143143
loge ("TLSv1_2_client_method() error");
144144
return (-1);

hu/hu_usb.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
#include <algorithm>
66

77

8-
#include <libusb.h>
8+
#include <libusb-1.0/libusb.h>
99

1010
#ifndef LIBUSB_LOG_LEVEL_NONE
1111
#define LIBUSB_LOG_LEVEL_NONE 0
@@ -355,7 +355,7 @@ int HUTransportStreamUSB::Start(bool waitForDevice) {
355355
return (-1);
356356
}
357357

358-
libusb_set_debug(iusb_ctx, LIBUSB_LOG_LEVEL_INFO);
358+
libusb_set_option(iusb_ctx, LIBUSB_OPTION_LOG_LEVEL, LIBUSB_LOG_LEVEL_INFO);
359359

360360
//See if there is a OAP device already
361361
while ((iusb_dev_hndl = find_oap_device()) == nullptr)

hu/hu_uti.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ int hu_log (int prio, const char * tag, const char * func, const char * fmt, ...
102102
//Time doesn't work on CMU anyway, always says 1970
103103
time_t timestamp;
104104
time(&timestamp);
105-
printf ("%d %s: %s: %s : %s\n", timestamp, prio_get (prio), tag, func, log_line);
105+
printf ("%ld %s: %s: %s : %s\n", timestamp, prio_get (prio), tag, func, log_line);
106106

107107
va_end(aq);
108108
#endif

mazda/callbacks.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
#include "command_server.h"
88
#include "audio.h"
99
#include <dbus-c++/dbus.h>
10-
#include <asoundlib.h>
10+
#include <alsa/asoundlib.h>
1111

1212
#include "dbus/generated_cmu.h"
1313
#include "version.h"

0 commit comments

Comments
 (0)