On 29/05/2014 12:14, Brice Videau wrote:
> Well this is normal. This is part of the process we used to retro
> engineer the loader when sources were not available.
> Nonetheless maybe we could let this part out of the main build.
> I'll let Vincent decide.

  I would keep it in the mail build. If I recall correctly, this code
is here to test the *ICD Loader* itself (with a specific *fake ICD*).
That explain why there is no need to stick to the real prototypes :
the first argument must be a pointer (required by the spec), other
arguments are not used by the ICD Loader.
  Using fake (simpler) prototypes allow to generate automatic tests.

Of course, using run_dummy_icd_through_our_ICDL with a real ICD would
probably crash.

  So, for now, my position is that I will keep this test in the
Debian package and I won't patch upstream software to tweak each
prototype manually (instead of using upstream automatic code
generation) in this test.
  That said, if upstream find (or someone provides them) a way
to use the real prototypes, I will of course use them in the
Debian package. Or another possibility would be for you to tell
us how to desactivate this compiler check here (a pragma?)

  Regards,
    Vincent Danjean

> Best,
> 
> Brice
> 
> On 05/29/2014 04:53 AM, Michael Tautschnig wrote:
>> Package: ocl-icd
>> Version: 2.1.3-4
>> Usertags: goto-cc
>>
>> During a rebuild of all Debian packages in a clean sid chroot (using 
>> cowbuilder
>> and pbuilder) the build failed with the following error. Please note that we
>> use our research compiler tool-chain (using tools from the cbmc package), 
>> which
>> permits extended reporting on type inconsistencies at link time.
>>
>> [...]
>> libtool: link: gcc -Wno-deprecated-declarations -g -O2 -fstack-protector 
>> --param=ssp-buffer-size=4 -Wformat -Werror=format-security -Wl,-z -Wl,relro 
>> -o .libs/run_dummy_icd_through_our_ICDL 
>> run_dummy_icd_through_our_ICDL-run_dummy_icd.o 
>> run_dummy_icd_through_our_ICDL-run_dummy_icd_gen.o 
>> run_dummy_icd_through_our_ICDL-run_dummy_icd_weak_gen.o  
>> ./.libs/libOpenCL.so -ldl
>>
>> error: conflicting function declarations "clEnqueueBarrierWithWaitList"
>> old definition in module run_dummy_icd_gen file /usr/include/CL/cl.h line 
>> 1150
>> signed int (struct _cl_command_queue *, unsigned int, struct _cl_event * 
>> const *, struct _cl_event **)
>> new definition in module run_dummy_icd_weak_gen file 
>> run_dummy_icd_weak_gen.c line 137
>> signed int (void *arg, void *arg2)
>> Makefile:728: recipe for target 'run_dummy_icd_through_our_ICDL' failed
>> make[3]: *** [run_dummy_icd_through_our_ICDL] Error 64
>> make[3]: Leaving directory 
>> '/srv/jenkins-slave/workspace/sid-goto-cc-ocl-icd/ocl-icd-2.1.3'
>> Makefile:980: recipe for target 'all-recursive' failed
>>
>> It seems the generated code of run_dummy_icd_weak_gen.c is not in line with 
>> the
>> system headers here, and there may be further errors hidden by this incorrect
>> declaration.
>>
>> Best,
>> Michael
>>
> 


-- 
Vincent Danjean       GPG key ID 0xD17897FA         vdanj...@debian.org
GPG key fingerprint: 621E 3509 654D D77C 43F5  CA4A F6AE F2AF D178 97FA
Unofficial pkgs: http://moais.imag.fr/membres/vincent.danjean/deb.html
APT repo:  deb http://people.debian.org/~vdanjean/debian unstable main


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to