Re: [PATCH spice-protocol 0/5] SPEC integration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/14/19 10:18 AM, Frediano Ziglio wrote:

Hi,

On Fri, Oct 11, 2019 at 10:56:24AM +0100, Frediano Ziglio wrote:
This series is part from a former series.
There's an initial import commit to better understand the changes.
It contains some work from Eduardo for MingW packaging.
Final patches are really minor.

Should I/we wait for Eduardo's review?


No idea what to suggest here. But surely I'd like some comments from
him. Part of the implementation came from his job.


Sorry, I missed this thread completely.

The thing that may be important to think about having a spec file in the upstream source code is that we need to be open to contributions from other distros as well.


Frediano Ziglio (5):
   build-sys: Import spec file from Fedora
   build-sys: Provide spec file during build
   build-sys: Allows spec file to build MingW packages
   build-sys: Update URL in SPEC file
   build-sys: Requires proper pkg-config for MingW

  Makefile.am            |   1 +
  configure.ac           |   1 +
  spice-protocol.spec.in | 224 +++++++++++++++++++++++++++++++++++++++++

Let's add to meson too?


I didn't though about. All SPEC files I started with are using autoconf
and I never tested distribution with Meson (I did instead with spice-gtk).
But it seems a bit of a follow up at the moment, it would require some
additional scripts to make Meson distribute processed files and other changes.
Not considering that if the SPEC at the end is using autoconf a Meson
build would have to add generated "configure" and other relative files
which is a bit odd and complex.


Why not taking this opportunity and dropping the autotools support in favor of meson, at least for protocol? Spice-gtk already did it, and it is much more complex than this one.

--
Eduardo de Barros Lima (Etrunko)
Software Engineer - Red Hat
etrunko@xxxxxxxxxx
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]