On Fri, Oct 1, 2010 at 6:27 AM, Tru Huynh <tru at 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.ay.src.rpm > <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