🗑️ Code cleanup and picolibc bug fixes (#47)

* 👷 Use ci/4.x.y version

The 4.x.y branch scheme means that this repo will get the latest
changes to the ci that does not include breaking changes.

* 🗑️ Additional code cleanup

Author: Khalil Estell

.github/workflows/ci.yml

@@ -18,7 +18,9 @@ on:
   workflow_dispatch:
   pull_request:
   release:
-    types: [published]
+    types:
+      - published
+      - deleted
   push:
     branches:
       - main
@@ -27,12 +29,12 @@ on:
 
 jobs:
   ci:
-    uses: libhal/ci/.github/workflows/library.yml@4.1.0
+    uses: libhal/ci/.github/workflows/library.yml@4.x.y
     with:
       library: libhal-soft
       coverage: true
     secrets: inherit
 
   devices:
-    uses: libhal/ci/.github/workflows/deploy.yml@4.1.0
+    uses: libhal/ci/.github/workflows/deploy.yml@4.x.y
     secrets: inherit

conanfile.py

@@ -19,7 +19,7 @@
 import os
 
 
-required_conan_version = ">=1.50.0"
+required_conan_version = ">=2.0.6"
 
 
 class libhal_soft_conan(ConanFile):

test_package/CMakeLists.txt

@@ -4,7 +4,7 @@
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at
 #
-#      http://www.apache.org/licenses/LICENSE-2.0
+# http://www.apache.org/licenses/LICENSE-2.0
 #
 # Unless required by applicable law or agreed to in writing, software
 # distributed under the License is distributed on an "AS IS" BASIS,
@@ -20,5 +20,9 @@ find_package(libhal-soft REQUIRED CONFIG)
 add_executable(${PROJECT_NAME} main.cpp)
 target_include_directories(${PROJECT_NAME} PUBLIC .)
 target_compile_features(${PROJECT_NAME} PRIVATE cxx_std_20)
-set_target_properties(${PROJECT_NAME} PROPERTIES CXX_EXTENSIONS OFF)
+
+if(BAREMETAL)
+  target_link_options(${PROJECT_NAME} PRIVATE --oslib=semihost)
+endif()
+
 target_link_libraries(${PROJECT_NAME} PRIVATE libhal::soft)

test_package/conanfile.py

@@ -22,6 +22,10 @@ class TestPackageConan(ConanFile):
     settings = "os", "arch", "compiler", "build_type"
     generators = "CMakeToolchain", "CMakeDeps", "VirtualRunEnv"
 
+    @property
+    def _bare_metal(self):
+        return self.settings.os == "baremetal"
+
     def requirements(self):
         self.requires(self.tested_reference_str)
 
@@ -30,7 +34,7 @@ def layout(self):
 
     def build(self):
         cmake = CMake(self)
-        cmake.configure()
+        cmake.configure(variables={"BAREMETAL": self._bare_metal})
         cmake.build()
 
     def test(self):

test_package/main.cpp

@@ -26,6 +26,6 @@ int main()
 namespace boost {
 void throw_exception(std::exception const& e)
 {
-  std::abort();
+  hal::halt();
 }
 }  // namespace boost