Skip to content

Conversation

@tony-josi-aws
Copy link
Member

Description

The PR updates the path for the cmake project and and sample configuration.

Test Steps

Tested by building the demo on 64 bit Linux using GCC:

(base) [δ:0s T:13:47:11]ubuntu@ip-172-31-69-236:~/Projects/FreeRTOS-Kernel$ cd build
(base) [δ:0s T:13:47:37]ubuntu@ip-172-31-69-236:~/Projects/FreeRTOS-Kernel/build$ rm -rf *
(base) [δ:0s T:13:47:41]ubuntu@ip-172-31-69-236:~/Projects/FreeRTOS-Kernel/build$ cmake ../examples/cmake_example/
-- The C compiler identification is GNU 9.4.0
-- The CXX compiler identification is GNU 9.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
-- Generating done
-- Build files have been written to: /home/ubuntu/Projects/FreeRTOS-Kernel/build
(base) [δ:1s T:13:47:55]ubuntu@ip-172-31-69-236:~/Projects/FreeRTOS-Kernel/build$ make
Scanning dependencies of target freertos_kernel_port
[  7%] Building C object FreeRTOS-Kernel/portable/CMakeFiles/freertos_kernel_port.dir/template/port.c.o
[ 15%] Linking C static library libfreertos_kernel_port.a
[ 15%] Built target freertos_kernel_port
Scanning dependencies of target freertos_kernel
[ 23%] Building C object FreeRTOS-Kernel/CMakeFiles/freertos_kernel.dir/croutine.c.o
[ 30%] Building C object FreeRTOS-Kernel/CMakeFiles/freertos_kernel.dir/event_groups.c.o
[ 38%] Building C object FreeRTOS-Kernel/CMakeFiles/freertos_kernel.dir/list.c.o
[ 46%] Building C object FreeRTOS-Kernel/CMakeFiles/freertos_kernel.dir/queue.c.o
[ 53%] Building C object FreeRTOS-Kernel/CMakeFiles/freertos_kernel.dir/stream_buffer.c.o
[ 61%] Building C object FreeRTOS-Kernel/CMakeFiles/freertos_kernel.dir/tasks.c.o
[ 69%] Building C object FreeRTOS-Kernel/CMakeFiles/freertos_kernel.dir/timers.c.o
[ 76%] Building C object FreeRTOS-Kernel/CMakeFiles/freertos_kernel.dir/portable/MemMang/heap_4.c.o
[ 84%] Linking C static library libfreertos_kernel.a
[ 84%] Built target freertos_kernel
Scanning dependencies of target example
[ 92%] Building C object CMakeFiles/example.dir/main.c.o
[100%] Linking C executable example
[100%] Built target example
(base) [δ:1s T:13:47:58]ubuntu@ip-172-31-69-236:~/Projects/FreeRTOS-Kernel/build$ 

Checklist:

  • I have tested my changes. No regression in existing tests.
  • [ ] I have modified and/or added unit-tests to cover the code changes in this Pull Request. NA

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov
Copy link

codecov bot commented Oct 23, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (fc7aca7) 93.64% compared to head (c29b290) 93.64%.

❗ Current head c29b290 differs from pull request most recent head 6ac7ca4. Consider uploading reports for the commit 6ac7ca4 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #851   +/-   ##
=======================================
  Coverage   93.64%   93.64%           
=======================================
  Files           6        6           
  Lines        2549     2549           
  Branches      608      608           
=======================================
  Hits         2387     2387           
  Misses        107      107           
  Partials       55       55           
Flag Coverage Δ
unittests 93.64% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 5 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@tony-josi-aws tony-josi-aws merged commit a8650b9 into FreeRTOS:main Oct 23, 2023
n9wxu pushed a commit to n9wxu/FreeRTOS-Kernel that referenced this pull request Oct 26, 2023
* fix build on 64 bit platform

* moving sample cmake project to a separate root level dir

* moving sample cmake project to a separate root level dir

* updating paths for the sample cmake project

* rename example folder

* use configKERNEL_PROVIDED_STATIC_MEMORY

* update comments

* update comments

* rename folder to examples

* fix formatting
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.

3 participants