Alex Kost <alezost@gmail.com> writes:

> John J Foerch (2016-07-04 06:47 +0300) wrote:
>
>> John J Foerch <jjfoerch@earthlink.net> writes:
>>
>>> Leo Famulari <leo@famulari.name> writes:
>>>
>>>> On Sat, Jul 02, 2016 at 03:13:52PM -0400, John J Foerch wrote:
>>>>> * gnu/packages/admin.scm (di): New variable.
>>>>
>>>> Thanks!
>>>>
>>>>> +    (arguments
>>>>> +     `(#:phases
>>>>> +       (modify-phases %standard-phases
>>>>> +         (delete 'configure)
>>>>> +         (delete 'check)
>>>>
>>>> Did you try setting `#:test-target "test"` in the arguments? The
>>>> Makefile has a test target.
>>>>
>>>
>>> I didn't know about that.  I'll try it out.
>>>
>> A test relating to an included perl module Filesys::di fails.  I hadn't
>> intended to include that module in the package, as all I'm interested in
>> is the 'di' binary.  Given that, I would suggest that we bypass the
>> tests.
>
> As for me, I would also disable the tests.  All these handmade Makefiles
> in all sub-directories are a nightmare.  I think trying to figure out
> what is needed to run the tests doesn't worth it.
>
> Also don't forget to add a copyright line for yourself in the beginning
> of "admin.scm".

Here is an updated patch: