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

[PATCH v1] Several build system fixes #440

Closed
wants to merge 6 commits into from

Conversation

lumag
Copy link

@lumag lumag commented Jan 31, 2018

Few updates to make implementation life easier.

Dmitry Eremin-Solenikov added 6 commits January 31, 2018 15:49
platform m4/configure.m4 file can make use of ODP_ABI_COMPAT. So let's
set it before calling into platform file.

Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>
Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>
platform m4/configure.m4 file might want to change compiler flags. So let's
set them before calling into platform file.

Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>
Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>
Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>
${with_platform} in doc/platform-api-guide/Makefile.am will add to
distro platform docs only for currently selected platform. Move this to
platform/Makefile.inc, so that every platform doc file gets included
into distro.

Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>
@muvarov muvarov changed the title Several build system fixes [PATCH v1] Several build system fixes Jan 31, 2018
@codecov
Copy link

codecov bot commented Jan 31, 2018

Codecov Report

Merging #440 into master will decrease coverage by 0.008%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##            master      #440       +/-   ##
=============================================
- Coverage   77.774%   77.766%   -0.009%     
=============================================
  Files          199       199               
  Lines        35766     35766               
=============================================
- Hits         27817     27814        -3     
- Misses        7949      7952        +3
Impacted Files Coverage Δ
test/validation/api/timer/timer.c 76.656% <0%> (-2.209%) ⬇️
test/validation/api/time/time.c 95.278% <0%> (-0.859%) ⬇️
platform/linux-generic/odp_schedule.c 92.292% <0%> (+0.385%) ⬆️
test/performance/odp_pktio_ordered.c 71.747% <0%> (+0.406%) ⬆️
platform/linux-generic/pktio/pcap.c 82.722% <0%> (+1.047%) ⬆️

Copy link
Contributor

@Bill-Fischofer-Linaro Bill-Fischofer-Linaro left a comment

Choose a reason for hiding this comment

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

Reviewed-by: Bill Fischofer bill.fischofer@linaro.org

@muvarov
Copy link
Contributor

muvarov commented Feb 2, 2018

Merged.

@muvarov muvarov closed this Feb 2, 2018
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

3 participants