Thanks to visit codestin.com
Credit goes to github.com

Skip to content

main, res, tests: Fix compilation errors on FreeBSD.#917

Merged
asterisk-org-access-app[bot] merged 1 commit into
asterisk:masterfrom
InterLinked1:master-issue-916
Oct 1, 2024
Merged

main, res, tests: Fix compilation errors on FreeBSD.#917
asterisk-org-access-app[bot] merged 1 commit into
asterisk:masterfrom
InterLinked1:master-issue-916

Conversation

@InterLinked1

@InterLinked1 InterLinked1 commented Sep 29, 2024

Copy link
Copy Markdown
Contributor

asterisk.c, manager.c: Increase buffer sizes to avoid truncation warnings.
config.c: Include header file for WIFEXITED/WEXITSTATUS macros.
res_timing_kqueue: Use more portable format specifier.
test_crypto: Use non-linux limits.h header file.

Resolves: #916

@github-actions

Copy link
Copy Markdown

REMINDER: If this PR applies to other branches, please add a comment with the appropriate "cherry-pick-to" headers as per the Create a Pull Request process.

If you don't want it cherry-picked, please add a comment with cherry-pick-to: none so we don't keep asking.

If, after adding "cherry-pick-to" comments, you change your mind, please edit the comment to DELETE the header lines and add cherry-pick-to: none.

The currently active branches are now 18, 20, 21, 22 and master.

@InterLinked1

Copy link
Copy Markdown
Contributor Author

cherry-pick-to: 18
cherry-pick-to: 20
cherry-pick-to: 21
cherry-pick-to: 22

@seanbright seanbright 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.

There is also a typo in your commit message: WEXISTSTATUS should be WEXITSTATUS.

Comment thread res/res_timing_kqueue.c Outdated

@seanbright seanbright 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.

Sorry. One more.

Comment thread main/config.c Outdated
asterisk.c, manager.c: Increase buffer sizes to avoid truncation warnings.
config.c: Include header file for WIFEXITED/WEXITSTATUS macros.
res_timing_kqueue: Use more portable format specifier.
test_crypto: Use non-linux limits.h header file.

Resolves: asterisk#916
@gtjoseph gtjoseph added the cherry-pick-test Trigger dry run of cherry-picks label Sep 30, 2024
@github-actions github-actions Bot added cherry-pick-testing-in-progress Cherry-Pick tests in progress cherry-pick-checks-failed Cherry-Pick checks failed and removed cherry-pick-test Trigger dry run of cherry-picks cherry-pick-testing-in-progress Cherry-Pick tests in progress labels Sep 30, 2024
@github-actions

github-actions Bot commented Oct 1, 2024

Copy link
Copy Markdown

Successfully merged to branch master and cherry-picked to ["18","20","21","22"]

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug]: Compilation errors on FreeBSD

4 participants