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 building containerinfo plugin on i386 #588

Closed
wants to merge 1 commit into from

Conversation

bzed
Copy link

@bzed bzed commented Apr 27, 2022

long is not a gint64 on i386.

long is not a gint64 on i386.
@johnwvmw
Copy link
Contributor

Thanks for finding and reporting this problem building the containerinfo plugin for the i386 architecture.

I will get the fix into the development tree ASAP.

Unfortunately the fix will not make it into the 12.0.5 open-vm-tools update, the code has been frozen. The issue will be listed in the 12.0.5 Release Notes as a known problem with a link to the fix that will be checked-in to the "devel" branch.

Copy link
Contributor

@johnwvmw johnwvmw left a comment

Choose a reason for hiding this comment

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

I believe there should be a leading space in the "us\n" string.

Will make the adjustment when checking in the patch. Thanks

johnwvmw added a commit that referenced this pull request May 6, 2022
The "gint64" type used for time values in the code is not a "long" on
i386.  The fix using the glib "G_GINT64_FORMAT" macro was provided by
the Debian OVT maintainer.

github PR:
#588
@johnwvmw
Copy link
Contributor

johnwvmw commented May 6, 2022

This pull request has been submitted to the "devel" branch at change 605c30a

@johnwvmw johnwvmw closed this May 6, 2022
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.

None yet

2 participants