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

dhd: Omit droidmedia+audioflingerglue from libexec #213

Merged
merged 1 commit into from Mar 15, 2019
Merged

dhd: Omit droidmedia+audioflingerglue from libexec #213

merged 1 commit into from Mar 15, 2019

Conversation

ix5
Copy link
Contributor

@ix5 ix5 commented Mar 10, 2019

Remove droidmedia and audioflingerglue libs+bins from $RPM_BUILD_ROOT/%{_libexecdir} when building droid-hal-device, thus preventing clashing files with the localbuild variants.

This commit simply moves the logic around so that one can build all HABUILD_SDK tasks at once and then not have to worry about the order of execution for building droid-hal and droidmedia/audioflingerglue-localbuild.

Without this commit, one has to re-build the droidmedia/audioflingerglue libs in the HABUILD_SDK every time before packaging the localbuild variants.

Remove droidmedia and audioflingerglue libs+bins from
$RPM_BUILD_ROOT/%{_libexecdir} when building droid-hal-device, thus
preventing clashing files with the localbuild variants.

This commit simply moves the logic around so that one can build all
HABUILD_SDK tasks at once and then not have to worry about the order of
execution for building droid-hal and
droidmedia/audioflingerglue-localbuild.

Without this commit, one has to re-build the droidmedia/audioflingerglue
libs in the HABUILD_SDK every time before packaging the localbuild
variants.
@sledges sledges changed the title [WIP] dhd: Omit droidmedia+audioflingerglue from libexec dhd: Omit droidmedia+audioflingerglue from libexec Mar 15, 2019
Copy link
Contributor

@sledges sledges left a comment

Choose a reason for hiding this comment

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

Tested, rebased, pushed, all works! Thanks

@sledges sledges merged commit 33d4dc7 into mer-hybris:master Mar 15, 2019
@ix5 ix5 deleted the dhdinc-exclude-localbuilt branch January 12, 2021 21:08
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 this pull request may close these issues.

None yet

2 participants