guac icon indicating copy to clipboard operation
guac copied to clipboard

[feature - ent] See if 32-bit support is possible

Open jeffmendoza opened this issue 2 years ago • 2 comments

https://github.com/ent/ent/issues/3125 Implies that the ent folks know they don't compile on 32-bit, and are not concerned (Asking about use case), but the issue is open, they maybe would accept a pr? Looks like just some casting is needed around bit shifts. The line we are getting is https://github.com/ent/contrib/blob/v0.4.5/entgql/template/node.tmpl#L271 in particular, but sounds like there are multiple of these in ent.

Looking at https://github.com/guacsec/guac/pull/910#discussion_r1303123801 Is there a workaround for this? different ID types? If so, we may want to do that. If not, let's discuss further if we want to pick this up with ent.

jeffmendoza avatar Aug 30 '23 15:08 jeffmendoza

Based on looking at https://github.com/droyo/styx/pull/17/files, would this be an upstream fix to ent to change that to uint32?

lumjjb avatar Sep 05 '23 14:09 lumjjb

Also , would we be able to skip building ent for 32 bit if we use build constraints? https://pkg.go.dev/go/build#hdr-Build_Constraints

lumjjb avatar Sep 05 '23 16:09 lumjjb