UmlautSoftwareDevelopmentAccount / AP-Functional-Cluster-Template

The template repository for Adaptive Platform Functional Cluster
MIT License
2 stars 10 forks source link

travis.yml: add the build_ci script #22

Closed phongt closed 4 years ago

phongt commented 4 years ago

reference applied PR from AP-CoreTypes https://github.com/UmlautSoftwareDevelopmentAccount/AP-CoreTypes/pull/49

phongt commented 4 years ago

2 build failure happen in Build #24

error: declaration of 'writer' shadows a global declaration [-Werror=shadow]

Track by https://github.com/UmlautSoftwareDevelopmentAccount/AP-Functional-Cluster-Template/issues/21

error: unknown warning option '-Wduplicated-cond' [-Werror,-Wunknown-warning-option] error: unknown warning option '-Wduplicated-branches' [-Werror,-Wunknown-warning-option] error: unknown warning option '-Wlogical-op'; did you mean '-Wlong-long'? [-Werror,-Wunknown-warning-option] error: unknown warning option '-Wuseless-cast' [-Werror,-Wunknown-warning-option]

Those compiler cpp flags don't recognize by clang.

phongt commented 4 years ago

error: unknown warning option '-Wduplicated-cond' [-Werror,-Wunknown-warning-option] error: unknown warning option '-Wduplicated-branches' [-Werror,-Wunknown-warning-option] error: unknown warning option '-Wlogical-op'; did you mean '-Wlong-long'? [-Werror,-Wunknown-warning-option] error: unknown warning option '-Wuseless-cast' [-Werror,-Wunknown-warning-option]

Those compiler cpp flags don't recognize by clang.

Fixed by 8f7b6d4

bburda commented 4 years ago

I think, that you can just remove -Wshadow for now, and we can merge this change.

phongt commented 4 years ago

I think, that you can just remove -Wshadow for now, and we can merge this change.

@bburda Ok. done in 6361d67

just for notice fmt were fixed by

https://github.com/fmtlib/fmt/commit/c3fa33314045cb4a132eef9a94524fb073a1e696 https://github.com/fmtlib/fmt/commit/d6cea50d01d7779ecb5ae4da0d785d9bba47a2f7

But the spdlog conan package have not used that version yet.