build: fix x86 builds

[monstermunchkin]

This PR fixes a minor issue which leads to x86 builds failing.

The GOARCH environment variable should be 386 instead of i386.

[ghost]

Can one of the admins verify this patch?

[lucab]

ok to test

[glevand]

This looks OK for now, but the value of interpBin is really dependent on the source files used to create the stage1 image, and in general could be different for the different stage1 flavors. I'm thinking now that having these init_linux files could be problematic, and that maybe the solution is to have interpBin set by the build at configure or make.

[lucab]

The configure.ac fix is definitely a good catch, but I'm not sure I understand the other commit. interpBin is only used by stage1-coreos flavor and CoreOS (the distro) doesn't have x86 images. Why is it being added here?

Other than that, I agree with @glevand remark that interpBin constant should be coming from somewhere like usr_from_coreos.mk

[monstermunchkin]

I added the iterpBin constant here because the x86 build fails otherwise. Moving this constant out of init_*.go sounds like a good idea though. This was more of a quick fix.

Should I then just remove the second commit, and fix it in another PR?

[lucab]

Should I then just remove the second commit, and fix it in another PR?

Yes, please. It doesn't make much sense to keep multiplying files for architectures that don't even need it. amd64 & arm64 should be set via specific built-time constants, passing a default placeholder for the rest.

[monstermunchkin]

I have removed the 2nd commit.

[lucab]

LGTM