CentOS devs (perhaps Tru?),
I have an updated version of nx/freenx ready. The srpms are:
http://centos.toracat.org/misc/nx-freenx/5/SRPMS/freenx-0.7.3-6.el5.centos.a... http://centos.toracat.org/misc/nx-freenx/5/SRPMS/nx-3.4.0-6.el5.centos.ay.sr...
Two tarballs (nxagent and nx-X11) in nx have been update to the current version (as of Sept 2010).
One main change done in this update is addition of support for RHEL6(beta). For example,
rpmbuild -bb --target x86_64 --define 'dist .el6.centos' --define "centos_ver 6" freenx.spec
will build the x86_64 version of freenx for RHEL/CentOS-6.
Anyone wishing to give them a try can find the binaries at:
http://centos.toracat.org/misc/nx-freenx/6/i386/ http://centos.toracat.org/misc/nx-freenx/6/x86_64/
I have tested them against RHEL 6 beta2 and they seem to work fine.
Thanks,
Akemi
On Thu, Sep 30, 2010 at 02:44:25PM -0700, Akemi Yagi wrote:
CentOS devs (perhaps Tru?),
Hi,
I have an updated version of nx/freenx ready. The srpms are:
http://centos.toracat.org/misc/nx-freenx/5/SRPMS/freenx-0.7.3-6.el5.centos.a... http://centos.toracat.org/misc/nx-freenx/5/SRPMS/nx-3.4.0-6.el5.centos.ay.sr...
I pulled both and had a quick look :P
Two tarballs (nxagent and nx-X11) in nx have been update to the current version (as of Sept 2010).
ack'ed md5sums from http://www.nomachine.com/sources.php confirmed :)
One main change done in this update is addition of support for RHEL6(beta). For example,
rpmbuild -bb --target x86_64 --define 'dist .el6.centos' --define "centos_ver 6" freenx.spec
that will not work since freenx-0.7.2-centos-el6-diffs.patch is not provided in http://centos.toracat.org/misc/nx-freenx/5/SRPMS/freenx-0.7.3-6.el5.centos.a... <quote> %if "%centos_ver" == "5" Patch1: freenx-0.7.2-centos-diffs.patch %endif %if "%centos_ver" == "6" Patch1: freenx-0.7.2-centos-el6-diffs.patch %endif </quote> to only one of them is included in the src.rpm :P
attached a proposed fix
Cheers,
Tru
to only one of them is included in the src.rpm :P
Hello Akemi and Tru,
It looks to me like only one patch for CentOS-5 and CentOS-6 should be possible, then no '%if' at all is needed and further maintenance of the rpm would also stay simpler. Try the attached patch for both CentOS-5 and CentOS-6, it seems to work ok for me...
regards,
Florian La Roche
P.S.: Would it make sense to also use that src.rpm for Fedora? Merging there as well would be a great next step...
On Fri, Oct 1, 2010 at 6:27 AM, Tru Huynh tru@centos.org wrote:
On Thu, Sep 30, 2010 at 02:44:25PM -0700, Akemi Yagi wrote:
Two tarballs (nxagent and nx-X11) in nx have been update to the current version (as of Sept 2010).
ack'ed md5sums from http://www.nomachine.com/sources.php confirmed :)
Thanks for checking. :)
One main change done in this update is addition of support for RHEL6(beta). For example,
rpmbuild -bb --target x86_64 --define 'dist .el6.centos' --define "centos_ver 6" freenx.spec
that will not work since freenx-0.7.2-centos-el6-diffs.patch is not provided in http://centos.toracat.org/misc/nx-freenx/5/SRPMS/freenx-0.7.3-6.el5.centos.a...
<quote> %if "%centos_ver" == "5" Patch1: freenx-0.7.2-centos-diffs.patch %endif %if "%centos_ver" == "6" Patch1: freenx-0.7.2-centos-el6-diffs.patch %endif </quote> to only one of them is included in the src.rpm :P
attached a proposed fix
That was my oversight. :-( Indeed, your patch is the way to handle a case like this.
Akemi
On Fri, Oct 1, 2010 at 7:08 AM, Florian La Roche Florian.LaRoche@gmx.net wrote:
to only one of them is included in the src.rpm :P
Hello Akemi and Tru,
It looks to me like only one patch for CentOS-5 and CentOS-6 should be possible, then no '%if' at all is needed and further maintenance of the rpm would also stay simpler. Try the attached patch for both CentOS-5 and CentOS-6, it seems to work ok for me...
regards,
Florian La Roche
P.S.: Would it make sense to also use that src.rpm for Fedora? Merging there as well would be a great next step...
Hi Florian,
Thanks for the patch. That will eliminate the need for the "if" in the spec.
Speaking of merging with Fedora, in fact, Johnny Hughes was planning to adopting the Fedora src as a next step. Would you like to take on this "project" ?
Akemi
Sorry to bother you again :(
On Fri, Oct 1, 2010 at 7:29 AM, Akemi Yagi amyagi@gmail.com wrote:
Speaking of merging with Fedora, in fact, Johnny Hughes was planning to adopting the Fedora src as a next step. Would you like to take on this "project" ?
s/adopting/adopt/
Akemi
On Fri, Oct 1, 2010 at 7:24 AM, Akemi Yagi amyagi@gmail.com wrote:
On Fri, Oct 1, 2010 at 6:27 AM, Tru Huynh tru@centos.org wrote:
attached a proposed fix
That was my oversight. :-( Indeed, your patch is the way to handle a case like this.
My proposal is to go ahead with updating nx/freenx for CentOS-5 with your patch (either that or by removing the CentOS-6 related code). We then deal with CentOS-6 as a moving target because that needs more testing for sure.
Akemi
On Fri, Oct 1, 2010 at 9:03 AM, Akemi Yagi amyagi@gmail.com wrote:
On Fri, Oct 1, 2010 at 7:24 AM, Akemi Yagi amyagi@gmail.com wrote:
On Fri, Oct 1, 2010 at 6:27 AM, Tru Huynh tru@centos.org wrote:
attached a proposed fix
That was my oversight. :-( Indeed, your patch is the way to handle a case like this.
My proposal is to go ahead with updating nx/freenx for CentOS-5 with your patch (either that or by removing the CentOS-6 related code). We then deal with CentOS-6 as a moving target because that needs more testing for sure.
Hi Tru,
I have uploaded the version that has your fix.
http://centos.toracat.org/misc/nx-freenx/5/SRPMS/freenx-0.7.3-6.el5.centos.a...
Next, we would like to deal with CentOS-6 as per Florian's proposal.
Akemi
Thanks for the patch. That will eliminate the need for the "if" in the spec.
Ok, having a current version pushed to CentOS will be good.
Speaking of merging with Fedora, in fact, Johnny Hughes was planning to adopting the Fedora src as a next step. Would you like to take on this "project" ?
I don't have Fedora running here, so not sure I'll have time for that. I'll check on it...
regards,
Florian La Roche