Re: golang bindings dirty in tree after libxl build

[George Dunlap <George.Dunlap@xxxxxxxxxx>]

> On Jun 12, 2020, at 12:00 PM, Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx> wrote:
> 
> Hello,
> 
> I've just done a libxl build and got things such as:
> 
> --- a/tools/golang/xenlight/helpers.gen.go
> +++ b/tools/golang/xenlight/helpers.gen.go
> @@ -431,14 +431,14 @@ x.Evtch = int(xc.evtch)
>  x.Rref = int(xc.rref)
>  x.Connection = ChannelConnection(xc.connection)
>  switch x.Connection{
> -case ChannelConnectionUnknown:
> -x.ConnectionUnion = nil
>  case ChannelConnectionPty:
>  var connectionPty ChannelinfoConnectionUnionPty
>  if err := connectionPty.fromC(xc);err != nil {
>   return fmt.Errorf("converting field connectionPty: %v", err)
>  }
>  x.ConnectionUnion = connectionPty
> +case ChannelConnectionUnknown:
> +x.ConnectionUnion = nil
>  case ChannelConnectionSocket:
>  x.ConnectionUnion = nil
>  default:
> 
> dirty in tree.  They are all case labels, and only their order in the
> switch has changed.
> 
> Does the current binding generation rely on the order of entries in a
> python dictionary by any chance?

Not explicitly, but obviously somewhat implicitly.

Is this a python2/3 issue, or would different versions of python within 2/3 end 
up with different sort orders?

If python3 will always put them in the same order, then we might consider just 
switching the script to being explicitly python3.  Otherwise, we’ll probably 
have to add sorts.

 -George