unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: ng0 <ng0@we.make.ritual.n0.is>
To: guix-devel <guix-devel@gnu.org>
Subject: Re: torsocks 2.0 bug
Date: Mon, 01 Aug 2016 15:37:07 +0000	[thread overview]
Message-ID: <87twf4pkkc.fsf@we.make.ritual.n0.is> (raw)
In-Reply-To: <87k2h9brtq.fsf@we.make.ritual.n0.is>

A short note to link threads:
It is likely that 2.2.x release will fix the current bug, see
https://lists.gnu.org/archive/html/guix-devel/2016-07/msg01452.html

ng0@we.make.ritual.n0.is writes:

> Hi,
>
> I found this thread while I was deleting old threads.
>
> I worked on torsocks-2.1.0 without knowing there was
> a discussion before. As expected I ran into problems
> too.
>
> The only change I did so far was version bumping, which
> when building torsocks-2.1.0 only fails tests.
>
> This is because our torsocks-2.0 disable dns patch no
> longer works but so far I have found no solution to
> fix it.
>
> Problems appear to be exclusive to tests/unit/test_dns.c ,
> line 59.
> I suggest that whoever wrote the torsocks patch creates
> one for torsocks-2.1.0
>
> Andreas Enge writes:
>
>> On Wed, Feb 03, 2016 at 07:08:32PM +0100, swedebugia@riseup.net wrote:
>>>       '(#:phases (modify-phases %standard-phases
>>> +                  ; FIXME. this phase is still needed but fails. I dont
>>> +                  ; understand why. It works fine when done manually in an
>>> +                  ; environment.
>>>                    (add-before 'configure 'bootstrap
>>>                      (lambda _
>>> -                      (system* "autoreconf" "-vfi"))))))
>>> +                      (system* "autoreconf" "-i")
>>
>> The problem is here. First of all, there was no need to drop the options "v"
>> and "f". But the real problem comes from dropping the parentheses.
>> So now what follows is part of the "lambda" function:
>>
>>> +       ; FIXME. test_fd_passing fail.
>>> +       #:tests? #f)))))
>>
>> And it ends with "#f", so the return value of the function is false, and the
>> phase fails.
>>
>> I tried closing more parentheses, and then the build passes, but the tests
>> still fail. This would need to be investigated more.
>>
>> Andreas
>>
>>


-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

      reply	other threads:[~2016-08-01 15:37 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-29 10:45 torsocks 2.0 bug swedebugia
2016-01-29 11:28 ` Efraim Flashner
2016-01-29 15:28   ` swedebugia
2016-02-03  9:40     ` Ludovic Courtès
2016-02-03 18:08       ` swedebugia
2016-02-03 19:33         ` Andreas Enge
2016-06-28  9:12           ` ng0
2016-08-01 15:37             ` ng0 [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87twf4pkkc.fsf@we.make.ritual.n0.is \
    --to=ng0@we.make.ritual.n0.is \
    --cc=guix-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).