Skip to content

Navigation Menu

Appearance settings

View all features
- BY COMPANY SIZE
  Enterprises
  Small and medium teams
  Startups
  Nonprofits
- BY USE CASE
  App Modernization
  DevSecOps
  DevOps
  CI/CD
  View all use cases
- BY INDUSTRY
  Healthcare
  Financial services
  Manufacturing
  Government
  View all industries
View all solutions
- EXPLORE BY TOPIC
  AI
  Software Development
  DevOps
  Security
  View all topics
- EXPLORE BY TYPE
  Customer stories
  Events & webinars
  Ebooks & reports
  Business insights
  GitHub Skills
- SUPPORT & SERVICES
  Documentation
  Customer support
  Community forum
  Trust center
  Partners
View all resources
- COMMUNITY
  GitHub SponsorsFund open source developers
- PROGRAMS
  Security Lab
  Maintainer Community
  Accelerator
  GitHub Stars
  Archive Program
- REPOSITORIES
  Topics
  Trending
  Collections
- ENTERPRISE SOLUTIONS
  Enterprise platformAI-powered developer platform
- AVAILABLE ADD-ONS
  GitHub Advanced SecurityEnterprise-grade security features
  Copilot for BusinessEnterprise-grade AI features
  Premium SupportEnterprise-grade 24/7 support
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

phoenix-rtos / libphoenix Public

Notifications You must be signed in to change notification settings
Fork 19
Star 7

Code
Issues 3
Pull requests 24
Actions
Projects
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Actions
Projects
Security
Insights

socktypes.h: define _SOCKLEN_T#408

Merged

badochov merged 1 commit intomasterphoenix-rtos/libphoenix:masterfrom

badochov/libphoenix-typedefinesphoenix-rtos/libphoenix:badochov/libphoenix-typedefinesCopy head branch name to clipboard

Mar 25, 2025

Conversation Commits1 (1)Checks Files changed

Merged

socktypes.h: define _SOCKLEN_T#408
badochov merged 1 commit intomasterfrom
badochov/libphoenix-typedefines

Conversation

Copy link

Contributor

badochov commented Mar 18, 2025

Some external program eg mbedtls check for those defines to checks if a type is available.

JIRA: RTOS-1027

Description

Motivation and Context

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)
Chore (refactoring, style fixes, git/CI config, submodule management, no code logic changes)

How Has This Been Tested?

Already covered by automatic testing.
New test added: (add PR link here).
Tested by hand on: (list targets here).

Checklist:

My change requires a change to the documentation.
I have updated the documentation accordingly.
I have added tests to cover my changes.
All new and existing linter checks and tests passed.
My changes generate no new compilation warnings for any of the targets.

Special treatment

This PR needs additional PRs to work (list the PRs, preferably in merge-order).
I will merge this PR by myself when appropriate.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

badochov requested a review from Darchiv

March 18, 2025 13:21

github-actions bot reviewed

View reviewed changes

include/setjmp.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

badochov force-pushed the badochov/libphoenix-typedefines branch from fb628d6 to 21a0bef Compare

March 18, 2025 13:28

github-actions bot reviewed

View reviewed changes

include/signal.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

badochov force-pushed the badochov/libphoenix-typedefines branch from 21a0bef to 8e21cb8 Compare

March 18, 2025 13:29

github-actions bot reviewed

View reviewed changes

include/stdint.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

include/stdint.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

include/stdint.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

include/stdint.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

badochov force-pushed the badochov/libphoenix-typedefines branch from 8e21cb8 to 6f2e994 Compare

March 18, 2025 13:33

github-actions bot reviewed

View reviewed changes

include/stdint.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

include/stdint.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

badochov force-pushed the badochov/libphoenix-typedefines branch from 6f2e994 to 7620e96 Compare

March 18, 2025 13:33

github-actions bot reviewed

View reviewed changes

include/stdlib.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

include/stdlib.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

badochov force-pushed the badochov/libphoenix-typedefines branch from 7620e96 to 96beae2 Compare

March 18, 2025 13:35

github-actions bot reviewed

View reviewed changes

include/termios.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

badochov force-pushed the badochov/libphoenix-typedefines branch from 96beae2 to 8aad691 Compare

March 18, 2025 13:36

github-actions bot reviewed

View reviewed changes

include/wchar.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

badochov force-pushed the badochov/libphoenix-typedefines branch from 8aad691 to a09797a Compare

March 18, 2025 13:43

github-actions bot reviewed

View reviewed changes

regex/regex2.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

badochov force-pushed the badochov/libphoenix-typedefines branch from a09797a to f50f5d6 Compare

March 18, 2025 13:44

github-actions bot reviewed

View reviewed changes

stdio/bignum.h Outdated Show resolved Hide resolved

Uh oh!

There was an error while loading. Please reload this page.

badochov force-pushed the badochov/libphoenix-typedefines branch from f50f5d6 to 8486a81 Compare

March 18, 2025 13:45

Copy link

github-actions bot commented Mar 18, 2025 •

edited

Loading

Unit Test Results

7 958 tests 7 434 ✅ 40m 16s ⏱️
470 suites 524 💤
1 files 0 ❌

Results for commit a6dab58.

♻️ This comment has been updated with latest results.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.


          socktypes.h: define _SOCKLEN_T

a6dab58

Add define to inform mbedtls socklen is defined.

JIRA: RTOS-1027

badochov force-pushed the badochov/libphoenix-typedefines branch from 8486a81 to a6dab58 Compare

March 18, 2025 16:52

Copy link

Contributor Author

badochov commented Mar 18, 2025

@Darchiv I changed this PR to only handle the SOCKLEN_T special case

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Darchiv approved these changes

View reviewed changes

Copy link

Member

Darchiv left a comment

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

LGTM, but please update PRs title

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

badochov changed the title ~~types: add #defines to typedefs~~ socktypes.h: define _SOCKLEN_T

badochov merged commit 307f01f into master

37 checks passed

Uh oh!

There was an error while loading. Please reload this page.

badochov deleted the badochov/libphoenix-typedefines branch

March 25, 2025 14:31

Darchiv mentioned this pull request

build: adapt to gcc 14.2 phoenix-rtos/phoenix-rtos-ports#85

Merged

14 tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

github-actions[bot] github-actions[bot] left review comments

Darchiv Darchiv approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Footer

© 2026 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Community
Docs
Contact

You can’t perform that action at this time.