Re: [Spice-devel] I want to contribute spice html5 client

2019-10-23 Thread Jeremy White
Hi, I actively maintain spice-html5 and work to review patches and see that they are committed. In the cases where I miss a patch or two, other project members usually do a good job of covering them as well. Your contributions would be welcome. Cheers, Jeremy On 10/23/19 6:10 AM, 조미리 wrot

Re: [Spice-devel] [PATCH spice-protocol v4 5/5] build-sys: Requires proper pkg-config for MingW

2019-10-23 Thread Frediano Ziglio
> > On 10/23/19 12:06 PM, Frediano Ziglio wrote: > > .pc files requires MingW version, not system one. > > > > I think it would be better if squashed into previous patch? This idea of > building the mingw packages from the same source is nice, but it needs > some more discussion. Lets keep this p

Re: [Spice-devel] [PATCH spice-protocol v4 4/5] build-sys: Allows spec file to build MingW packages

2019-10-23 Thread Victor Toso
Hi, On Wed, Oct 23, 2019 at 11:06:31AM +0100, Frediano Ziglio wrote: > Add MingW support. This allows to build MingW packages easily with a > > $ rpmbuild -ta spice-protocol-.tar.bz2 > > This is base on some work by Eduardo Lima adding MingW support to SPEC > files. > Part of dependencies,

Re: [Spice-devel] [PATCH spice-protocol v4 5/5] build-sys: Requires proper pkg-config for MingW

2019-10-23 Thread Eduardo Lima (Etrunko)
On 10/23/19 12:06 PM, Frediano Ziglio wrote: .pc files requires MingW version, not system one. I think it would be better if squashed into previous patch? This idea of building the mingw packages from the same source is nice, but it needs some more discussion. Lets keep this patch on hold fo

Re: [Spice-devel] [PATCH spice-protocol v4 3/5] build-sys: Provide spec file during build

2019-10-23 Thread Eduardo Lima (Etrunko)
On 10/23/19 12:06 PM, Frediano Ziglio wrote: Having the RPM spec file produced by the build is helpful in different situation. Different tools like mock or code scanners accept source RPMs as input. Providing this file allows easily build of the source RPM (and binaries) using a simple command li

Re: [Spice-devel] [PATCH spice-protocol v4 2/5] build-sys: Update URL in SPEC file

2019-10-23 Thread Eduardo Lima (Etrunko)
On 10/23/19 12:06 PM, Frediano Ziglio wrote: Use https instead of http, web server automatically redirects to secure version. Remove slash at the end, not needed. Signed-off-by: Frediano Ziglio --- spice-protocol.spec.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a

Re: [Spice-devel] [PATCH spice-protocol v4 1/5] build-sys: Import spec file from Fedora

2019-10-23 Thread Eduardo Lima (Etrunko)
On 10/23/19 12:06 PM, Frediano Ziglio wrote: Will be used by next commits. The SPEC file will allow to an RPM from sources with a simple rpmbuild command. Package files are useful for easy deployment. This will also be used to provide automatic package build. This commit just contain a verbatim c

[Spice-devel] I want to contribute spice html5 client

2019-10-23 Thread 조미리
Hi, I'm interested to develop spice web client. But I don't think spice-html5 project is administrated well, Is the project going well? I found out another spice web client project, eyeOS spice-web-client project. I know these two projects aren't relevant. I want to contribute any projec

[Spice-devel] [PATCH spice-protocol v4 2/5] build-sys: Update URL in SPEC file

2019-10-23 Thread Frediano Ziglio
Use https instead of http, web server automatically redirects to secure version. Remove slash at the end, not needed. Signed-off-by: Frediano Ziglio --- spice-protocol.spec.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/spice-protocol.spec.in b/spice-protocol.spec.in

[Spice-devel] [PATCH spice-protocol v4 5/5] build-sys: Requires proper pkg-config for MingW

2019-10-23 Thread Frediano Ziglio
.pc files requires MingW version, not system one. Signed-off-by: Frediano Ziglio --- spice-protocol.spec.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/spice-protocol.spec.in b/spice-protocol.spec.in index 54aacfd..2fe940b 100644 --- a/spice-protocol.spec.in +++ b/spice-p

[Spice-devel] [PATCH spice-protocol v4 3/5] build-sys: Provide spec file during build

2019-10-23 Thread Frediano Ziglio
Having the RPM spec file produced by the build is helpful in different situation. Different tools like mock or code scanners accept source RPMs as input. Providing this file allows easily build of the source RPM (and binaries) using a simple command like $ rpmbuild -ta spice-protocol-.tar.bz

[Spice-devel] [PATCH spice-protocol v4 4/5] build-sys: Allows spec file to build MingW packages

2019-10-23 Thread Frediano Ziglio
Add MingW support. This allows to build MingW packages easily with a $ rpmbuild -ta spice-protocol-.tar.bz2 This is base on some work by Eduardo Lima adding MingW support to SPEC files. Part of dependencies, description and names came from Fedora SPEC file for mingw-spice-protocol package.

[Spice-devel] [PATCH spice-protocol v4 1/5] build-sys: Import spec file from Fedora

2019-10-23 Thread Frediano Ziglio
Will be used by next commits. The SPEC file will allow to an RPM from sources with a simple rpmbuild command. Package files are useful for easy deployment. This will also be used to provide automatic package build. This commit just contain a verbatim copy of upstream SPEC file to make easier to see

Re: [Spice-devel] [spice-protocol 2/2] fixup: gitlab-ci: to fix after mingw

2019-10-23 Thread Victor Toso
On Wed, Oct 23, 2019 at 05:52:09AM -0400, Frediano Ziglio wrote: > > > > From: Victor Toso > > > > --- > > .gitlab-ci.yml | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml > > index da4f512..c7028dd 100644 > > --- a/.gitlab-ci.yml

Re: [Spice-devel] [spice-protocol 2/2] fixup: gitlab-ci: to fix after mingw

2019-10-23 Thread Frediano Ziglio
> > From: Victor Toso > > --- > .gitlab-ci.yml | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml > index da4f512..c7028dd 100644 > --- a/.gitlab-ci.yml > +++ b/.gitlab-ci.yml > @@ -4,9 +4,10 @@ variables: >DEPS_COMMON: git libtool ma

[Spice-devel] [spice-protocol 0/2] add gitlabci check

2019-10-23 Thread Victor Toso
From: Victor Toso Hi, two patches to be squashed if you agree. Both for fedora-autotools stage in gitlab-ci. Victor Toso (2): fixup: gitlab-ci: to be added fixup: gitlab-ci: to fix after mingw .gitlab-ci.yml | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) -- 2.21.0

[Spice-devel] [spice-protocol 2/2] fixup: gitlab-ci: to fix after mingw

2019-10-23 Thread Victor Toso
From: Victor Toso --- .gitlab-ci.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index da4f512..c7028dd 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -4,9 +4,10 @@ variables: DEPS_COMMON: git libtool make python3 python3-six redh

[Spice-devel] [spice-protocol 1/2] fixup: gitlab-ci: to be added

2019-10-23 Thread Victor Toso
From: Victor Toso So we can check rpm is working fine. --- .gitlab-ci.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index b3fad64..da4f512 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -3,7 +3,7 @@ image: fedora:latest variables