Skip to content

feat(packaging): Provide standard Linux package via CMake - #752

Draft
gvhwurie wants to merge 8 commits into
developfrom
feature/provide-proper-packaging
Draft

feat(packaging): Provide standard Linux package via CMake#752
gvhwurie wants to merge 8 commits into
developfrom
feature/provide-proper-packaging

Conversation

@gvhwurie

Copy link
Copy Markdown

Fixes #666

@gvhwurie gvhwurie self-assigned this Jul 24, 2026
@github-project-automation github-project-automation Bot moved this to Architecture Review Required in halif_aidl Jul 24, 2026
@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


1 out of 2 committers have signed the CLA.
gvhwurie
manstrupt
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@manstrupt
manstrupt force-pushed the feature/provide-proper-packaging branch from 760b09a to 275f680 Compare August 11, 2026 08:04

@gvhwurie gvhwurie left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Todos:

  • Remove all purely transitive dependencies from the CMake files
  • Maybe generate project version from last Git tag

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing newline at end of file

Comment thread audiomixer/0.3.0.0/CMakeLists.txt Outdated
Comment thread avclock/current/CMakeLists.txt Outdated
Comment thread bootreason/0.1.0.0/CMakeLists.txt Outdated
find_path(
Binder_INCLUDE_DIR
NAMES binder/Binder.h
HINTS ${PC_Binder_INCLUDE_DIRS} "${CMAKE_CURRENT_SOURCE_DIR}/out/build/include/binder_sdk"

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shall we keep those remains to support locally build SDKs? Do we have to include standard paths just like in find_executable when searching for aidl? Would it have any negative consequences if we did so?

Comment thread CMakeModules/HandleModules.cmake Outdated
Comment thread CMakeModules/HandleModules.cmake Outdated
Comment thread CMakeModules/HandleModules.cmake Outdated
Comment thread common/0.2.0.0/CMakeLists.txt Outdated
Comment thread common/current/CMakeLists.txt Outdated
@gvhwurie

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

@gvhwurie

Copy link
Copy Markdown
Author

recheck

@manstrupt
manstrupt force-pushed the feature/provide-proper-packaging branch from b8abd55 to 5e68015 Compare August 19, 2026 10:42
gvhwurie and others added 6 commits August 19, 2026 14:05
…h for cmake packages Add full list of source files to cmake packages
If a module has no depends, the string is empty and
string replace will fail
Source was empty after using source and header variables as parameters
fix foreach to use ITEMS instead of LISTS

Fixed wrong naming in FindBinderUtils,add FindBinderUtils to install rule
@manstrupt
manstrupt force-pushed the feature/provide-proper-packaging branch from 5e68015 to 865c374 Compare August 19, 2026 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Architecture Review Required

Development

Successfully merging this pull request may close these issues.

Feature: Provide proper packaging

2 participants