Le 09/01/2013 09:36, Kalle Raiskila a écrit : > On 09/01/13 03:10, ext Vincent Danjean wrote: >> I will apply the patch to remove llvm-config --ldflags in the Debian >> package as this reduce the dependency between binary packages without >> any visible effect. >> >> You can get the last patch for this here: >> http://anonscm.debian.org/gitweb/?p=collab-maint/pocl.git;a=blob;f=debian/patches/fix_linkage.patch;hb=HEAD >> or (raw data) : >> http://anonscm.debian.org/gitweb/?p=collab-maint/pocl.git;a=blob_plain;f=debian/patches/fix_linkage.patch;hb=HEAD >> >> I'm really interrested to know if this last patch works anywhere or not. > > Ok, I think we should go with this, and apply that patch to 0.8 then > (just to get 0.7 released :). I haven't tried the patch yet, but it > looks sane, so there is no reason why it shouldn't be added to pocl > direclty.
Ok. If you are interested by it for 0.8, I will apply it locally for the Debian package for 0.7 (as it minimize the dependencies) >>> Is the script location/search path issue a show-stopper for >>> the Debian package? >> >> The fact that the build directory is searched before is problematic. > > This again is one point in favour to do pocl with the API approach. Yes. I hope 0.8 will include the API solution. For 0.7, if you prefer to avoid it as it is rather intrusive, I will apply it anyway in the Debian package. I'm confident I will be able to fix any issue due to this patch on Debian system. In this case, I let you decide if you want to apply it for 0.8 or if you want to quickly move to the API solution. >> 4) debian llvm-3.2 package seems to be built with -fno-rtti. >> If this is really a problem for pocl (the basic testsuite pass, is it >> really required?) then I will need to ask llvm maintainers to change >> this (and hope they accept) > > So far I have not needed RTTI on any 'plain' pocl builds. The RTTI > requirement comes from TCE with which pocl is used, but this I guess is > not the normal Debian use-case. Great. This means I can upload my package quickly (it does not enable the TCE driver). Vincent ------------------------------------------------------------------------------ Master Java SE, Java EE, Eclipse, Spring, Hibernate, JavaScript, jQuery and much more. Keep your Java skills current with LearnJavaNow - 200+ hours of step-by-step video tutorials by Java experts. SALE $49.99 this month only -- learn more at: http://p.sf.net/sfu/learnmore_122612 _______________________________________________ pocl-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/pocl-devel
