<html> <head> <meta content="text/html; charset=utf-8" http-equiv="Content-Type"> </head> <body text="#000000" bgcolor="#FFFFFF"> Il 29/02/2016 12:26, Yury Shvedov ha scritto: <blockquote cite="mid:56D42AE1.3030003@lvk.cs.msu.su" type="cite">Hi, Fabio! Take look at my latest commit It now merged with latest master version and successfully compiles with ./configure --enable-spice-compositor. But unfortunately it doesn't work due to new spice API. I hope, this evening it will!. </blockquote> Thanks for your work about it. I make the new diff in other test branch: <a class="moz-txt-link-freetext" href="https://github.com/Fantu/compositor-spice/tree/test2">https://github.com/Fantu/compositor-spice/tree/test2</a> And I have some questions: <meta name="qrichtext" content="1"> - src/Makefile.am was removed in newer weston and now unused, I suppose to be removed - Makefile.am missed monitor renderer additions, must be added or monitor renderer is not needed anymore? - src/compositor-rdp.c: I suppose is not needed and not related changes to be removed, right? - src/spice/Makefile.am: I suppose is unused now that thing are added in Makefile.am, to be removed, right? Can be the monitor renderer missed/incomplete the cause of "run test" failed? About spice-server api I did't found good docs to make update simply and fast but with a fast search I found this xspice (similar project for xorg instead) commit that probably can be faster update some deprecrated spice functions: <a class="moz-txt-link-freetext" href="https://cgit.freedesktop.org/xorg/driver/xf86-video-qxl/commit/?id=70884bd353c34c0be23c2b21eec320cd8c637f4f">https://cgit.freedesktop.org/xorg/driver/xf86-video-qxl/commit/?id=70884bd353c34c0be23c2b21eec320cd8c637f4f</a> I don't have sufficent free time for try to change it and test build/use today. After update to newer api I suppose will be good add also a required spice-server version check in configure based on newer api, I found this that seems will make fast see at what version was added any api: <a class="moz-txt-link-freetext" href="https://cgit.freedesktop.org/spice/spice/tree/server/spice-server.syms">https://cgit.freedesktop.org/spice/spice/tree/server/spice-server.syms</a> Another important note if you don't know it, spice-server recently is under heavy changes, latest version (0.13.0) is like a "devel snapshot". Latest stable version that I think is good to use also with this project for now is 0.12.6. Thanks for any reply and sorry for my bad english.  <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <style type="text/css"> p, li { white-space: pre-wrap; } </style> <blockquote cite="mid:56D42AE1.3030003@lvk.cs.msu.su" type="cite"> On 02/29/2016 12:22 PM, Daniel Stone wrote: <blockquote type="cite">Hi Fabio, On 27 February 2016 at 18:02, Fabio Fantoni <a class="moz-txt-link-rfc2396E" href="mailto:fabio.fantoni@m2r.biz"><fabio.fantoni@m2r.biz></a> wrote: <blockquote type="cite">Hi, long time ago I saw an interesting project for weston, the spice compositor: <a class="moz-txt-link-freetext" href="https://github.com/ein-shved/compositor-spice">https://github.com/ein-shved/compositor-spice</a> It is now abandoned because the developer has been involved in another project. As no other has continued it, despite my low knowledge and time I would try to update, test and possibly improve it. </blockquote> Great! <blockquote type="cite">I did a new branch with only 2 commit on top of latest upstream commit: <a class="moz-txt-link-freetext" href="https://github.com/Fantu/compositor-spice/commits/test">https://github.com/Fantu/compositor-spice/commits/test</a> and I tried to do a fast rebase on latest upstream commit (1.10) instead of master (development branch) for decrease the risk regression on first build/use tests: <a class="moz-txt-link-freetext" href="https://github.com/Fantu/compositor-spice/commits/rebase/spice-1.10">https://github.com/Fantu/compositor-spice/commits/rebase/spice-1.10</a> Solving conflict about configure and makefile parts I have some doubts (as also reported in the description of each commit): About first commit (Add Spice compositor) <a class="moz-txt-link-freetext" href="https://github.com/Fantu/compositor-spice/commit/f589ab264e80d43fa0853770481b6ddcadf5505b">https://github.com/Fantu/compositor-spice/commit/f589ab264e80d43fa0853770481b6ddcadf5505b</a> - in configure.ac some changes seems strange, including LIBS and CFLAGS that seems "double" </blockquote> I think this can be removed. Usually setting LIBS/CFLAGS and foo_save_LIBS/foo_save_CFLAGS is used for an AC_CHECK_* call, which relies on LIBS and CFLAGS already being set. I guess there may have been a call here which has since been removed. <blockquote type="cite">About the second commit (Monitor renderer) <a class="moz-txt-link-freetext" href="https://github.com/Fantu/compositor-spice/commit/2632b8b8067e46ac69b5ad1bc2164d90ced5e19f">https://github.com/Fantu/compositor-spice/commit/2632b8b8067e46ac69b5ad1bc2164d90ced5e19f</a> - Makefile things seems fully changed, tried to adapt them but I'm not sure if I did it correct. - Add -g to AM_CPPFLAGS in Makefile.am is really needed? not added for now </blockquote> No, this is a debugging feature only. <blockquote type="cite">- add of "-Wl,--wrap=pixman_renderer_init" to LDFLAGS of many other backend is really needed? not added for now, if needed is good understand why to add it also to new things added since this start commit done 3 years ago </blockquote> This should be solved in a different way if required. <blockquote type="cite">- src/compositor-rdp.c changes is really needed? if not I'll remove them I also searched documentation about api and/or internal weston functions changed any versions but I not found them. </blockquote> There is no documentation on the change, no. As you can see, several functions have changed: - weston_output_finish_frame now takes a struct timespec rather than an integer number of milliseconds (trivial conversion) - the output repaint function now returns an integer marking success or failure - the compositor interface has now changed to weston_backend, and you can see examples of the changes required in commit 954f183e Hope this helps: just pick out the warnings and errors one by one, and try to figure them out - searching git commits for anything relevant always helps - until you get something that builds. Cheers, Daniel </blockquote> -- Kind Regards, Yury Shvedov </blockquote> </body> </html>