[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: Golang design session follow-up
> On Aug 28, 2020, at 4:57 PM, George Dunlap <george.dunlap@xxxxxxxxxx> wrote: > > > >> On Jul 21, 2020, at 1:35 AM, Nick Rosbrook <rosbrookn@xxxxxxxxx> wrote: >> >> # Long-term home of the package >> >> Ian: Autogenerated stuff is becoming more annoying. >> >> Delete all the libxl auto-generated stuff from staging & master, and have >> "output branch". >> >> The reason we have these in-tree is that otherwise you can't build *from >> git* if you don't >> have new enough versions of the right tools. >> >> Distribution: Make a repo on xenbits! > > So thinking about this: > > The first plan I had was to have a script in tools/golang/xenlight (and/or > the Makefile), which would be handed a directory, and would then: > > 1. Sync static files from tools/golang/xenlight into that directory > > 2. Run gengotypes.py, having the resulting generated files put into that > directory > > 3. Run `git diff` in the target directory; if there are any changes, then > automatically run `git commit` to check in the changes. > > That way you could just set up a cron job to sync things over on a regular > basis. > > Thinking about GPL considerations, however, you’d also want to include > libxl_types.idl and idl.py. And then of course, you should also include a > way to build the generated code from those two. > > At which point… would it make sense to just move the package out to its > separate repo entirely? I.e., have actual development happen in the repo > which ends up being cloned in the end? > > Obviously there are nice things about having the code in the same repo; but > there’s also something satisfying about being a full downstream. > > I was actually thinking it might make sense to put the repo at > https://gitlab.com/xen-project/go-xenlight , to try out that as a development > model. I’ve put a sort of draft module up at https://gitlab.com/martyros/go-xen ; you can test it by adding the "gitlab.com/xen-project/go-xen/xenlight” package, but adding the following line to the go.mod of the test program: replace gitlab.com/xen-project/go-xen => gitlab.com/martyros/go-xen master This builds but for me unfortunately crashes during domain creation with the following error: libxl: error: libxl_create.c:90:libxl__domain_build_info_setdefault: Unable to copy deprecated fields libxl: error: libxl_create.c:1182:libxl__domain_config_setdefault: Unable to set domain build info defaults I’ll have to dig a bit more into it next week. -George
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |