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.

Sign up for GitHub

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

Jump to bottom

cross compilation fails due to linuxInfo.Ctim.Sec #861

Closed
dkwo opened this issue May 9, 2024 · 0 comments · Fixed by #862
Closed

cross compilation fails due to linuxInfo.Ctim.Sec #861

dkwo opened this issue May 9, 2024 · 0 comments · Fixed by #862
Assignees
@stgraber
Labels
Bug Confirmed to be a bug Easy Good for new contributors
Milestone
incus-6.2

Comments

@dkwo
Copy link

dkwo commented May 9, 2024

When updating to v6.1 on void linux void-linux/void-packages#50251 , those archs for which we cross build (build on 64 host for a 32 bit target) now fail with

internal/server/instance/drivers/driver_common.go:1539:19: cannot use linuxInfo.Ctim.Sec (variable of type int32) as int64 value in argument to time.Unix
internal/server/instance/drivers/driver_common.go:1539:39: cannot use linuxInfo.Ctim.Nsec (variable of type int32) as int64 value in argument to time.Unix

Is it possible something has to be fixed in incus? see e.g. #829
Thanks.
stgraber added a commit to stgraber/incus that referenced this issue May 9, 2024
@stgraber
incusd/instance: Fix building on 32bit architectures
9c733bc 
Closes lxc#861

Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
@stgraber stgraber mentioned this issue May 9, 2024
Merged
@stgraber stgraber self-assigned this May 9, 2024
@stgraber stgraber added Bug Confirmed to be a bug Easy Good for new contributors labels May 9, 2024
@stgraber stgraber added this to the incus-6.2 milestone May 9, 2024
stgraber added a commit that referenced this issue May 27, 2024
@stgraber
incusd/instance: Fix building on 32bit architectures
86836c1 
Closes #861

Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
@dkwo dkwo mentioned this issue Jun 5, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stgraber stgraber

Labels
Bug Confirmed to be a bug Easy Good for new contributors
Milestone
incus-6.2
Development

Successfully merging a pull request may close this issue.

incusd/instance: Fix building on 32bit architectures stgraber/incus
2 participants
@stgraber @dkwo