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

Fixed error: unknown type name ‘ssize_t’ on Solaris #212

Merged
merged 1 commit into from
Feb 23, 2024

Conversation

larsewi
Copy link
Contributor

@larsewi larsewi commented Feb 22, 2024

After removing sequence.h include dependency from string_lib.h, it appears that there is no definition of ssize_t on SPARC 64 Solaris 11. Including sys/types.h should fix this issue.

After removing sequence.h include dependency from string_lib.h, it
appears that there is no definition of `ssize_t` on SPARC 64 Solaris
11. Including sys/types.h should fix this issue.

Ticket: None
Changelog: None
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
@mender-test-bot
Copy link

There was an error running your pipeline, see logs for details.

@larsewi larsewi requested a review from olehermanse February 22, 2024 13:53
@larsewi
Copy link
Contributor Author

larsewi commented Feb 22, 2024

@cf-bottom Jenkins with exotics please :)

@cf-bottom
Copy link

@larsewi
Copy link
Contributor Author

larsewi commented Feb 22, 2024

Build Status

@larsewi
Copy link
Contributor Author

larsewi commented Feb 23, 2024

HP-UX failure looks like rsync failures. Sequential test is marked as unstable.

@larsewi larsewi requested a review from vpodzime February 23, 2024 10:22
Copy link
Contributor

@vpodzime vpodzime left a comment

Choose a reason for hiding this comment

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

ACK

@larsewi larsewi merged commit 58705c5 into NorthernTechHQ:master Feb 23, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants