Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix compile issue on FreeBSD 14 #925

Merged
merged 1 commit into from
Dec 17, 2024

Conversation

moonetic
Copy link
Contributor

This PR fixes compilation fail on FreeBSD 14, described on issue by hxw The problem is, that there's no getSendQueueLen function defined for bsd platform, which causes compile failure. The fix adds new, sockio_bsd file with corresponding function implementation and build tags.

This PR fixes compilation fail on FreeBSD 14, described on [issue by hxw](uber#914)
The problem is, that there's no getSendQueueLen function defined for bsd platform, which causes compile failure. The fix adds new, sockio_bsd file with corresponding function implementation and build tags.
@CLAassistant
Copy link

CLAassistant commented Oct 31, 2024

CLA assistant check
All committers have signed the CLA.

@moonetic moonetic changed the title Fix compile issue on FreeBSD 14 (#914) Fix compile issue on FreeBSD 14 Oct 31, 2024
@ibarrajo ibarrajo added the bug label Nov 7, 2024
@ibarrajo ibarrajo requested a review from ChenX1993 November 7, 2024 18:57
Copy link
Collaborator

@cinchurge cinchurge left a comment

Choose a reason for hiding this comment

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

LGTM

@cinchurge cinchurge merged commit 48b1eb9 into uber:dev Dec 17, 2024
3 checks passed
moonetic added a commit to moonetic/tchannel-go that referenced this pull request Dec 19, 2024
This PR fixes compilation fail on FreeBSD 14, described on [issue by hxw](uber#914)
The problem is, that there's no getSendQueueLen function defined for bsd platform, which causes compile failure. The fix adds new, sockio_bsd file with corresponding function implementation and build tags.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants