Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issues with GCC 12.1 #60

Closed
FireBurn opened this issue May 14, 2022 · 3 comments · Fixed by #61
Closed

Issues with GCC 12.1 #60

FireBurn opened this issue May 14, 2022 · 3 comments · Fixed by #61

Comments

@FireBurn
Copy link

Hi

I need the following patch to compile with GCC 12.1

From 752fc3bd9a971858dbb7b51a0ef0289232e67e58 Mon Sep 17 00:00:00 2001
From: Mike Lothian <[email protected]>
Date: Sat, 14 May 2022 02:01:18 +0100
Subject: [PATCH] Add functional include

---
tls/include/adb/tls/tls_connection.h | 1 +
1 file changed, 1 insertion(+)

diff --git a/tls/include/adb/tls/tls_connection.h
b/tls/include/adb/tls/tls_connection.h
index bc5b98ab..a112756c 100644
--- a/tls/include/adb/tls/tls_connection.h
+++ b/tls/include/adb/tls/tls_connection.h
@@ -19,6 +19,7 @@
#include <stddef.h>
#include <stdint.h>

+#include <functional>
#include <string_view>
#include <vector>

It also warns about:

 * QA Notice: Package triggers severe warnings which indicate that it
 *            may exhibit random runtime failures.
 * /var/tmp/portage/dev-util/android-tools-31.0.3_p1/work/android-tools-31.0.3p1/vendor/selinux/libselinux/src/label_file.h:289:25: warning: ‘free’ called on pointer ‘mmap_area_89->next_addr’ with nonzero offset 4 [-Wfree-nonheap-object]

Which is fixed with SELinuxProject/selinux@6e5d16a

@nmeum
Copy link
Owner

nmeum commented May 18, 2022

Could create a pull request for the patch? (:

@FireBurn
Copy link
Author

It's in a subproject, I'm not entirely sure how to do a PR on Google code, but I've emailed the last person that touched the code

nmeum added a commit that referenced this issue May 18, 2022
@nmeum
Copy link
Owner

nmeum commented May 18, 2022

I was referring to this repository. Can you test #61?

@nmeum nmeum closed this as completed in #61 Jul 14, 2022
nmeum added a commit that referenced this issue Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants