xs.h is LGPL. xenctrl.h is GPL. If xs.h includes xenctrl.h it implicitly makes xs.h GPL''d. The following patches remove the xenctrl.h dependency in xs.h. There''s a hack to work around the problem of Xen''s {u,s}{8,16,32,64} type usage. It would be nicer to have a common types header file in xen/include/public. Is there a right way to do this? Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> Regards, Anthony Liguori _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Anthony Liguori wrote:> xs.h is LGPL. xenctrl.h is GPL. If xs.h includes xenctrl.h it > implicitly makes xs.h GPL''d. The following patches remove the xenctrl.h > dependency in xs.h.Why not just make all headers LGPL (or less restrictive, e.g. BSD)? Don''t many of these headers need to be referenced from OSes where software is typically BSD-licensed, anyway? (BSD without the advertising clause is compatible with GPL and LGPL.) -- David Hopwood <david.nospam.hopwood@blueyonder.co.uk> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
On 11 Oct 2005, at 17:25, David Hopwood wrote:> Anthony Liguori wrote: >> xs.h is LGPL. xenctrl.h is GPL. If xs.h includes xenctrl.h it >> implicitly makes xs.h GPL''d. The following patches remove the >> xenctrl.h >> dependency in xs.h. > > Why not just make all headers LGPL (or less restrictive, e.g. BSD)? > Don''t many of these headers need to be referenced from OSes where > software > is typically BSD-licensed, anyway? > > (BSD without the advertising clause is compatible with GPL and LGPL.)libxenctrl is GPL. Anything that includes the header file ought also to be linking with libxenctrl, so just changing the header file license is something of a hack. Getting rid of the dependency is the right answer I think. -- Keir _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel