[Wait for #2720][CI] Fix meson ubuntu ci build #2716
Conversation
DonghakPark
requested review from
myungjoo,
jijoongmoon,
again4you,
jaeyun-jung,
leemgs,
wooksong,
helloahn,
kparichay,
gichan-jang,
anyj0527,
zhoonit,
lhs8928,
songgot,
jihochu,
SeoHyungjun,
baek2sm,
skykongkong8,
djeong20,
EunjuYang and
a team
as code owners
|
📝 TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2716. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/. |
DonghakPark
force-pushed
the
fix_ubuntu_build
branch
5 times, most recently
from
bb67730
to
5b2ca0b
Compare
There was a problem hiding this comment.
@DonghakPark, 💯 All CI checkers are successfully verified. Thanks.
|
It is normal to fail because the CI has not been working properly so far. Additionally, for the current changes, instead of deleting directories, each name was changed differently, and the meson setup was removed. |
DonghakPark
force-pushed
the
fix_ubuntu_build
branch
from
5b2ca0b
to
51bab61
Compare
|
For clarity's sake, let me provide some comments. So far, the reason why Meson build & Ninja tests have not been working as intended is due to the redundant usage of both 'meson setup build' and 'meson build'. In addition to this, I initially thought that jobs in GitAction would run within the same container but with @myungjoo help, I realized it was unnecessary to separate the names of build directories, so I made adjustments accordingly. |
|
|
cibot:
DonghakPark
force-pushed
the
fix_ubuntu_build
branch
2 times, most recently
from
b890eda
to
64e2d0c
Compare
DonghakPark
force-pushed
the
fix_ubuntu_build
branch
3 times, most recently
from
9e2e91a
to
d56c8a7
Compare
There was a problem hiding this comment.
@DonghakPark, 💯 All CI checkers are successfully verified. Thanks.
DonghakPark
force-pushed
the
fix_ubuntu_build
branch
from
d56c8a7
to
592fceb
Compare
There was a problem hiding this comment.
@DonghakPark, 💯 All CI checkers are successfully verified. Thanks.
|
For gcc-13 in Ubuntu Jammy, use |
DonghakPark
force-pushed
the
fix_ubuntu_build
branch
2 times, most recently
from
5d79355
to
7cdfbfb
Compare
There was a problem hiding this comment.
@DonghakPark, 💯 All CI checkers are successfully verified. Thanks.
DonghakPark
changed the title
There was a problem hiding this comment.
@DonghakPark, 💯 All CI checkers are successfully verified. Thanks.
Fix build bug - Currently, there is a bug in the matrix used in CI where the first Meson build runs successfully but subsequent builds fail due to the presence of a 'build' folder. I would like to fix this issue. - Before running the Meson build, ensure that any existing folders named 'build' are deleted. - fix gcc version to 13 Resolves: - nnstreamer#2715 **Self evaluation:** 1. Build test: [X]Passed [ ]Failed [ ]Skipped 2. Run test: [X]Passed [ ]Failed [ ]Skipped Signed-off-by: Donghak PARK <[email protected]> Co-authored-by: hyeonseok <[email protected]> Co-authored-by: Donghyeon Jeong <[email protected]>
DonghakPark
force-pushed
the
fix_ubuntu_build
branch
from
03d9445
to
ea8ec32
Compare
- This commit adds some updates for Tizen7.0 backporting - Type mismatch bug is fixed. - Unused variable is removed. - Missing header files are added in spec file. - spec file is updated Self evaluation: Build test: [X]Passed [ ]Failed [ ]Skipped Run test: [X]Passed [ ]Failed [ ]Skipped Signed-off-by: Eunju Yang <[email protected]>
DonghakPark
force-pushed
the
fix_ubuntu_build
branch
from
ea8ec32
to
0d764b9
Compare
There was a problem hiding this comment.
@DonghakPark, 💯 All CI checkers are successfully verified. Thanks.
Fix build bug
Resolves:
Self evaluation:
Signed-off-by: Donghak PARK [email protected]
Co-authored-by: hyeonseok [email protected]
Co-authored-by: Donghyeon Jeong [email protected]