[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH RFC 59/59] tools/xenlight: Create interface for xenlight info
On Thu, Dec 29, 2016 at 10:52:42AM +0000, Wei Liu wrote: > On Thu, Dec 29, 2016 at 10:34:34AM +0000, George Dunlap wrote: > > On 29/12/16 01:14, Ronald Rojas wrote: > > > Create interface to interact with libxl_verions_info > > > and libxl_physinfo. Also introduce proper error > > > handling. > > > > > > Signed-off-by: Ronald Rojas <ronladred@xxxxxxxxx> > > > > Hey Ronald, > > > > When you send a series to a public open-source project, you want it to > > be a sort of idealized set of changes that make it easy to "see the > > forest for the trees", as it were, not the actual local development > > path. We don't need the full development history of libxl.go -- a > > simple dump will be fine; and we definitely don't need all the other > > stuff from schedbench in the xen.git history. :-) > > > > All you needed to do to "import" my existing libxl code from schedbench was: > > > > $ cd xen.git > > $ cp ../schedbench.git/controller/libxl.go tools/golang/xenlight/xenlight.go > > $ git add tools/golang/xenlight/xenlight.go This is much easier then what I was doing. I did it another way because I thought you would have wanted to keep the git commit history. I can resubmit the patch into 2 commits so it'll be easier to read. 1 for importing the code from schedbench and then the changes that I made afterward. How does that sound? > > > > +1 for this. _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |