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

[adsp] Enable more platforms #23

Merged
merged 1 commit into from Mar 4, 2016

Conversation

Projects
None yet
2 participants
@AchimTuran
Copy link
Member

commented Feb 20, 2016

Still missing is Android for adsp.biquad.filters.

Furthermore I will create new branches or tags if they get stable enough.

Afterwards we can merge this PR.

Edit:
The WIP label is missing here and I'm not able to create it. That's the reason why I added it to the header of this PR.
From my side this commit is ready to merge.

@AchimTuran AchimTuran self-assigned this Feb 20, 2016

@AchimTuran AchimTuran changed the title [adsp] Enable more platforms [WIP] [adsp] Enable more platforms Feb 20, 2016

@AchimTuran AchimTuran force-pushed the AchimTuran:enable-more-adsp-platforms branch from 48a7f70 to b05656b Mar 2, 2016

@AchimTuran AchimTuran changed the title [WIP] [adsp] Enable more platforms [adsp] Enable more platforms Mar 2, 2016

@AchimTuran

This comment has been minimized.

Copy link
Member Author

commented Mar 2, 2016

@MartijnKaijser
If you want you can merge this commit to enable the build of the adsp on more platforms. adsp.biquad.filters is still missing because of some compiler flags, which is the reason why it is disabled on Android. If I fixed the issue I will create a new commit.

Here is the latest Jenkins Log.

MartijnKaijser added a commit that referenced this pull request Mar 4, 2016

@MartijnKaijser MartijnKaijser merged commit 33c6301 into xbmc:master Mar 4, 2016

@MartijnKaijser

This comment has been minimized.

Copy link
Member

commented Mar 6, 2016

adsp.biquad.filters fails to build on multiple platforms. Please fix or revert building on more platforms

@AchimTuran

This comment has been minimized.

Copy link
Member Author

commented Mar 6, 2016

@MartijnKaijser
Do you have a build log for me?

@MartijnKaijser

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.