Skip to content

impl: fix QNX test issues, restore CI tests#278

Merged
arkjedrz merged 4 commits into
eclipse-score:mainfrom
qorix-group:arkjedrz_remove-libc-dep
Jun 22, 2026
Merged

impl: fix QNX test issues, restore CI tests#278
arkjedrz merged 4 commits into
eclipse-score:mainfrom
qorix-group:arkjedrz_remove-libc-dep

Conversation

@arkjedrz

@arkjedrz arkjedrz commented Jun 17, 2026

Copy link
Copy Markdown
Contributor
  • Remove libc dependency - not available for QNX x86_64.
  • Fix implementation and tests.
  • Restore tests in CI.

Resolves #251

@arkjedrz arkjedrz self-assigned this Jun 17, 2026
@github-project-automation github-project-automation Bot moved this to In Progress in BAS - Baselibs FT Jun 17, 2026
@arkjedrz arkjedrz requested a deployment to workflow-approval June 17, 2026 13:15 — with GitHub Actions Waiting
@arkjedrz arkjedrz requested a deployment to workflow-approval June 17, 2026 13:15 — with GitHub Actions Waiting
@arkjedrz arkjedrz force-pushed the arkjedrz_remove-libc-dep branch from 47553f7 to cef587f Compare June 17, 2026 13:15
@arkjedrz arkjedrz temporarily deployed to workflow-approval June 17, 2026 13:15 — with GitHub Actions Inactive
@arkjedrz arkjedrz temporarily deployed to workflow-approval June 17, 2026 13:15 — with GitHub Actions Inactive
@github-actions

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

@arkjedrz arkjedrz temporarily deployed to workflow-approval June 18, 2026 05:41 — with GitHub Actions Inactive
@arkjedrz arkjedrz temporarily deployed to workflow-approval June 18, 2026 05:41 — with GitHub Actions Inactive
@arkjedrz arkjedrz marked this pull request as ready for review June 18, 2026 05:59
@arkjedrz arkjedrz requested a review from Copilot June 18, 2026 05:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 12 changed files in this pull request and generated 3 comments.

Comment thread src/pal/lib.rs Outdated
Comment on lines +24 to +30
pub type c_int = i32;
pub type c_long = i64;
pub type c_ulong = u64;
pub type size_t = usize;
pub type c_void = core::ffi::c_void;
pub type pid_t = i32;
pub type pthread_t = c_ulong;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@arkjedrz makse sens or ?

Comment thread src/containers/storage/heap.rs
Comment thread src/thread/thread.rs Outdated
@arkjedrz arkjedrz mentioned this pull request Jun 21, 2026
1 task
arkjedrz added 3 commits June 21, 2026 14:41
Removed libc dependency - not available for QNX x86_64.
- Fix implementation and tests.
- Restore tests in CI.
@arkjedrz arkjedrz force-pushed the arkjedrz_remove-libc-dep branch from eca01e5 to fbe7bed Compare June 21, 2026 12:41
@arkjedrz arkjedrz temporarily deployed to workflow-approval June 21, 2026 12:41 — with GitHub Actions Inactive
@arkjedrz arkjedrz temporarily deployed to workflow-approval June 21, 2026 12:41 — with GitHub Actions Inactive
@github-project-automation github-project-automation Bot moved this from In Progress to On Hold in BAS - Baselibs FT Jun 22, 2026

@pawelrutkaq pawelrutkaq left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

On small comment

@arkjedrz arkjedrz temporarily deployed to workflow-approval June 22, 2026 07:41 — with GitHub Actions Inactive
@arkjedrz arkjedrz temporarily deployed to workflow-approval June 22, 2026 07:41 — with GitHub Actions Inactive
@pawelrutkaq pawelrutkaq added this pull request to the merge queue Jun 22, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 22, 2026
@arkjedrz arkjedrz added this pull request to the merge queue Jun 22, 2026
Merged via the queue into eclipse-score:main with commit 41a6e86 Jun 22, 2026
20 checks passed
@arkjedrz arkjedrz deleted the arkjedrz_remove-libc-dep branch June 22, 2026 10:37
@github-project-automation github-project-automation Bot moved this from On Hold to Done in BAS - Baselibs FT Jun 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Investigate failing rust tests

3 participants