GNU bug report logs - #31733
[PATCH] gnu: encfs: Update to 1.9.5.

Previous Next

Full log

Message #11 received at 31733 <at> debbugs.gnu.org (full text, mbox):

From: Kei Kebreau <kkebreau <at> posteo.net>
To: Pierre Neidhardt <ambrevar <at> gmail.com>
Cc: 31733 <at> debbugs.gnu.org
Subject: Re: [bug#31733] [PATCH] gnu: encfs: Update to 1.9.5.
Date: Thu, 14 Jun 2018 11:04:30 -0400

[Message part 1 (text/plain, inline)]

Kei Kebreau <kkebreau <at> posteo.net> writes:

> Hi Pierre,
>
>> * gnu/packages/crypto.scm (encfs): Update to 1.9.5.
>
> Don't forget to mention the new 'make-unittests' phase!
>
>> ---
>>  gnu/packages/crypto.scm | 11 ++++++++---
>>  1 file changed, 8 insertions(+), 3 deletions(-)
>>
>> diff --git a/gnu/packages/crypto.scm b/gnu/packages/crypto.scm
>> index f96fe0490..4707355f3 100644
>> --- a/gnu/packages/crypto.scm
>> +++ b/gnu/packages/crypto.scm
>> @@ -216,7 +216,7 @@ communication.")
>>  (define-public encfs
>>    (package
>>      (name "encfs")
>> -    (version "1.9.1")
>> +    (version "1.9.5")
>>      (source
>>       (origin
>>         (method url-fetch)
>> @@ -225,7 +225,7 @@ communication.")
>>                         version "/encfs-" version ".tar.gz"))
>>         (sha256
>>          (base32
>> -         "1906254dg5hwljh0h4gyrw09ms3b57dlhjfzhfzffv50yzpkl837"))
>> +         "0qzxavvv20577bxvly8s7d3y7bqasqclc2mllp0ddfncjm9z02a7"))
>>         (modules '((guix build utils)))
>>         ;; Remove bundled dependencies in favour of proper inputs.
>>         (snippet '(for-each delete-file-recursively
>> @@ -244,7 +244,12 @@ communication.")
>>         ("openssl" ,openssl)
>>         ("tinyxml2" ,tinyxml2)))
>>      (arguments
>> -     `(#:configure-flags (list "-DUSE_INTERNAL_TINYXML=OFF")))
>> +     `(#:configure-flags (list "-DUSE_INTERNAL_TINYXML=OFF")
>> +       #:phases
>> +       (modify-phases %standard-phases
>> +         (add-before 'check 'make-unittests
>> +           (lambda _
>> +             (zero? (system* "make" "unittests")))))))
>
> Any particular reason you decided to use '(zero? (system* ...))' rather
> than '(invoke ...)' for this phase? I tried it with 'invoke' and it
> worked fine.
>
>>      (home-page "https://vgough.github.io/encfs")
>>      (synopsis "Encrypted virtual file system")
>>      (description
>
> Other than the above, this patch looks good! :)

Also, do note that the snippet needs to be updated, as bundled libraries
(now including googletest) have moved to the "vendor" subdirectory. Once
you remove the "vendor" subdirectory, you may have to add 'googletest'
from '(gnu packages check)' to the native-inputs.

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.