search

guacsec / guac

GUAC aggregates software security metadata into a high fidelity graph database.
https://guac.sh
Apache License 2.0
1.27k stars 172 forks source link

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

Open jeffmendoza opened 1 year ago

jeffmendoza commented 1 year ago

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.

lumjjb commented 1 year ago

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 commented 1 year ago

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