Re: [PATCH] Add Blender

[Ludovic Courtès]

Ricardo Wurmus <address@hidden> skribis:

> Eric Bavier <address@hidden> writes:

[...]

>>> * gnu/packages/graphics.scm (blender): New variable.
>
> [...]
>
>>> +         (add-after 'set-paths 'add-ilmbase-include-path
>>> +          (lambda* (#:key inputs #:allow-other-keys)
>>> +            ;; OpenEXR propagates ilmbase, but its include files do not 
>>> appear
>>> +            ;; in the CPATH, so we need to add "$ilmbase/include/OpenEXR/" 
>>> to
>>> +            ;; the CPATH to satisfy the dependency on "half.h".
>>> +            (setenv "CPATH"
>>> +                    (string-append (assoc-ref inputs "ilmbase")
>>> +                                   "/include/OpenEXR"
>>> +                                   ":" (getenv "CPATH")))
>>
>> Is the "half.h" header in OpenEXR, or in blender?  If the latter, would
>> it make more sense to instead patch the include directive to include
>> the OpenEXR path?
>
> “half.h” is provided by “ilmbase”.  The include is in Blender, in this
> file:
>
>     blender-2.75a/source/blender/imbuf/intern/openexr/openexr_api.cpp
>
> I could patch this instead, replacing
>
>     #include <half.h>
>
> with
>
>     #include <OpenEXR/half.h>

Normally ‘OpenEXR.pc’ provides the right -I flags:

  https://lists.gnu.org/archive/html/guix-devel/2015-09/msg00160.html

Could it be that the .pc is ignored or something?

Ludo’.