Re: [PATCH] gnu: Add chromium-bsu.

[Alex Kost]

Leo Famulari (2016-06-22 00:44 +0300) wrote:

> On Tue, Jun 21, 2016 at 09:59:31AM -0400, Kei Kebreau wrote:
>> * gnu/packages/games (chromium-bsu): New variable.
>
>> +    (arguments
>> +     `(#:phases (modify-phases %standard-phases
>> +                  (add-after 'set-paths 'set-sdl-paths
>> +                             (lambda* (#:key inputs #:allow-other-keys)
>> +                               (setenv "CPATH"
>> +                                       (string-append (assoc-ref inputs 
>> "sdl-union")
>> +                                                      "/include/SDL"))
>> +                               #t)))))
>
> If you move (modify-phases) underneath #:phases, then this will be
> comfortably shorter than 80 characters :)

I agree with Leo, I would also reformat it like this:

  (arguments
   `(#:phases
     (modify-phases %standard-phases
       (add-after 'set-paths 'set-sdl-paths
         (lambda* (#:key inputs #:allow-other-keys)
           (setenv "CPATH"
                   (string-append (assoc-ref inputs "sdl-union")
                                  "/include/SDL"))
           #t)))))

-- 
Alex