GNU bug report logs - #26752
Ansible & others' problems with wrapped '.ansible-real' scripts

Previous Next

Package: guix;

Reported by: Jelle Licht <jlicht <at> fsfe.org>

Date: Tue, 2 May 2017 20:10:02 UTC

Severity: important

Merged with 31299

Done: Brice Waegeneire <brice <at> waegenei.re>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 26752 in the body.
You can then email your comments to 26752 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-guix <at> gnu.org:
bug#26752; Package guix. (Tue, 02 May 2017 20:10:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Jelle Licht <jlicht <at> fsfe.org>:
New bug report received and forwarded. Copy sent to bug-guix <at> gnu.org. (Tue, 02 May 2017 20:10:02 GMT) Full text and rfc822 format available.

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

From: Jelle Licht <jlicht <at> fsfe.org>
To: bug-guix <at> gnu.org
Subject: Ansible & others' problems with wrapped '.ansible-real' scripts
Date: Tue, 2 May 2017 22:08:38 +0200
[Message part 1 (text/plain, inline)]
Hi,

I had some problems getting current ansible package to work. It seems that
the bin/ansible script which is created as part of the python-build-system
via a call to `wrap-program' interferes with certain expectations ansible
has regarding how it and its subcommands are called.

This mechanism does not work well with our generated created .ansible-real.
See [1] for a similar issue that has since been worked around in the newest
version of ansible.

For now, I have a similar workaround that add some guix-specific checks to
ansible looking for being called as .ansible-real, though I do consider
this a hack. This problem is indicative of a bigger issue: the fact that
wrap-program currently leads to subtle bugs for lots of scripts. There has
been some noise on #guix about solving this problem in general.

Thanks,
Jelle

[1]: https://github.com/ansible/ansible/issues/22261
[Message part 2 (text/html, inline)]

Information forwarded to bug-guix <at> gnu.org:
bug#26752; Package guix. (Wed, 03 May 2017 10:05:01 GMT) Full text and rfc822 format available.

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

From: ludo <at> gnu.org (Ludovic Courtès)
To: Jelle Licht <jlicht <at> fsfe.org>
Cc: 26752 <at> debbugs.gnu.org
Subject: Re: bug#26752: Ansible & others' problems with wrapped
 '.ansible-real' scripts
Date: Wed, 03 May 2017 12:04:17 +0200
Jelle Licht <jlicht <at> fsfe.org> skribis:

> I had some problems getting current ansible package to work. It seems that
> the bin/ansible script which is created as part of the python-build-system
> via a call to `wrap-program' interferes with certain expectations ansible
> has regarding how it and its subcommands are called.
>
> This mechanism does not work well with our generated created .ansible-real.
> See [1] for a similar issue that has since been worked around in the newest
> version of ansible.
>
> For now, I have a similar workaround that add some guix-specific checks to
> ansible looking for being called as .ansible-real, though I do consider
> this a hack. This problem is indicative of a bigger issue: the fact that
> wrap-program currently leads to subtle bugs for lots of scripts. There has
> been some noise on #guix about solving this problem in general.

For the record, the discussion is visible here:
<https://gnunet.org/bot/log/guix/2017-05-02#T1370586>.

I think it’s bad for a program to rely on argv[0], but I also think
those .thing-real are kinda ugly.  :-)

Ludo’.




Severity set to 'important' from 'normal' Request was from ludo <at> gnu.org (Ludovic Courtès) to control <at> debbugs.gnu.org. (Mon, 29 May 2017 11:09:02 GMT) Full text and rfc822 format available.

Information forwarded to bug-guix <at> gnu.org:
bug#26752; Package guix. (Thu, 22 Jun 2017 16:03:01 GMT) Full text and rfc822 format available.

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

From: Jelle Licht <jlicht <at> fsfe.org>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 26752 <at> debbugs.gnu.org
Subject: Re: bug#26752: Ansible & others' problems with wrapped
 '.ansible-real' scripts
Date: Thu, 22 Jun 2017 18:02:07 +0200
[Message part 1 (text/plain, inline)]
The current ansible package is still brokenin the same way.

Is there already an acceptable way of working around this problem?
Otherwise I could send my (extremely hacky) workaround that adds a specific
condition in the ansible source code to check for .ansible-real.

Thanks,
Jelle

2017-05-03 12:04 GMT+02:00 Ludovic Courtès <ludo <at> gnu.org>:

> Jelle Licht <jlicht <at> fsfe.org> skribis:
>
> > I had some problems getting current ansible package to work. It seems
> that
> > the bin/ansible script which is created as part of the
> python-build-system
> > via a call to `wrap-program' interferes with certain expectations ansible
> > has regarding how it and its subcommands are called.
> >
> > This mechanism does not work well with our generated created
> .ansible-real.
> > See [1] for a similar issue that has since been worked around in the
> newest
> > version of ansible.
> >
> > For now, I have a similar workaround that add some guix-specific checks
> to
> > ansible looking for being called as .ansible-real, though I do consider
> > this a hack. This problem is indicative of a bigger issue: the fact that
> > wrap-program currently leads to subtle bugs for lots of scripts. There
> has
> > been some noise on #guix about solving this problem in general.
>
> For the record, the discussion is visible here:
> <https://gnunet.org/bot/log/guix/2017-05-02#T1370586>.
>
> I think it’s bad for a program to rely on argv[0], but I also think
> those .thing-real are kinda ugly.  :-)
>
> Ludo’.
>
[Message part 2 (text/html, inline)]

Information forwarded to bug-guix <at> gnu.org:
bug#26752; Package guix. (Thu, 22 Jun 2017 19:04:01 GMT) Full text and rfc822 format available.

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

From: ludo <at> gnu.org (Ludovic Courtès)
To: Jelle Licht <jlicht <at> fsfe.org>
Cc: 26752 <at> debbugs.gnu.org
Subject: Re: bug#26752: Ansible & others' problems with wrapped
 '.ansible-real' scripts
Date: Thu, 22 Jun 2017 21:03:33 +0200
Hi,

Jelle Licht <jlicht <at> fsfe.org> skribis:

> The current ansible package is still brokenin the same way.
>
> Is there already an acceptable way of working around this problem?
> Otherwise I could send my (extremely hacky) workaround that adds a specific
> condition in the ansible source code to check for .ansible-real.

For now I think we have to go with the hack.  Make sure to add a comment
linking to this bug report.

Thanks,
Ludo’.




Merged 26752 31299. Request was from ludo <at> gnu.org (Ludovic Courtès) to control <at> debbugs.gnu.org. (Sun, 29 Apr 2018 16:20:03 GMT) Full text and rfc822 format available.

Information forwarded to bug-guix <at> gnu.org:
bug#26752; Package guix. (Sun, 29 Apr 2018 16:21:01 GMT) Full text and rfc822 format available.

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

From: ludo <at> gnu.org (Ludovic Courtès)
To: Jelle Licht <jlicht <at> fsfe.org>
Cc: 31299 <at> debbugs.gnu.org, 26752 <at> debbugs.gnu.org
Subject: Re: bug#31299: Ansible depends on $0,
 which does not work for wrapped python scripts.
Date: Sun, 29 Apr 2018 18:20:38 +0200
Merging with <https://bugs.gnu.org/26752>, which you submitted a while
back.

Ludo’.




Information forwarded to bug-guix <at> gnu.org:
bug#26752; Package guix. (Mon, 07 May 2018 17:26:02 GMT) Full text and rfc822 format available.

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

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Jelle Licht <jlicht <at> fsfe.org>
Cc: 26752 <at> debbugs.gnu.org
Subject: Re: bug#26752: Ansible & others' problems with wrapped
 '.ansible-real' scripts
Date: Mon, 7 May 2018 19:25:51 +0200
[Message part 1 (text/plain, inline)]
Just a heads-up, if we make the Python wrap-program do the wrapping inline (bug# 29856), this problem here will also vanish.

See https://lists.gnu.org/archive/html/guix-devel/2017-11/msg00027.html for the newest possible approach.

[Message part 2 (application/pgp-signature, inline)]

Information forwarded to bug-guix <at> gnu.org:
bug#26752; Package guix. (Mon, 04 Feb 2019 07:51:02 GMT) Full text and rfc822 format available.

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

From: Ricardo Wurmus <rekado <at> elephly.net>
To: ludo <at> gnu.org (Ludovic Courtès)
Cc: Jelle Licht <jlicht <at> fsfe.org>, 26752 <at> debbugs.gnu.org, 29951 <at> debbugs.gnu.org
Subject: Re: bug#26752: Ansible & others' problems with wrapped
 '.ansible-real' scripts
Date: Mon, 04 Feb 2019 08:50:14 +0100
ludo <at> gnu.org (Ludovic Courtès) writes:
> Jelle Licht <jlicht <at> fsfe.org> skribis:
>
>> The current ansible package is still brokenin the same way.
>>
>> Is there already an acceptable way of working around this problem?
>> Otherwise I could send my (extremely hacky) workaround that adds a specific
>> condition in the ansible source code to check for .ansible-real.
>
> For now I think we have to go with the hack.  Make sure to add a comment
> linking to this bug report.

There is a way to get around this.  It’s presented in #29951:

    https://issues.guix.info/issue/29951

Open issues with this include a hard-coded store prefix, but it’s no big
obstacle.  Should we give this a try on core-updates?





Added indication that bug 26752 blocks29951 Request was from Ricardo Wurmus <rekado <at> elephly.net> to control <at> debbugs.gnu.org. (Mon, 04 Feb 2019 08:04:02 GMT) Full text and rfc822 format available.

Added blocking bug(s) 29951 Request was from Ricardo Wurmus <rekado <at> elephly.net> to control <at> debbugs.gnu.org. (Mon, 04 Feb 2019 10:32:01 GMT) Full text and rfc822 format available.

Information forwarded to bug-guix <at> gnu.org:
bug#26752; Package guix. (Mon, 04 Feb 2019 10:33:02 GMT) Full text and rfc822 format available.

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

From: Arun Isaac <arunisaac <at> systemreboot.net>
To: Jelle Licht <jlicht <at> fsfe.org>
Cc: Ricardo Wurmus <rekado <at> elephly.net>, 26752 <at> debbugs.gnu.org
Subject: Re: bug#26752: Ansible & others' problems with wrapped
 '.ansible-real' scripts
Date: Mon, 04 Feb 2019 16:02:44 +0530
[Message part 1 (text/plain, inline)]
> The current ansible package is still brokenin the same way.

FWIW, I fixed our ansible package in
https://issues.guix.info/issue/33777 . This is not a general solution,
but at least our ansible package works at the moment.
[signature.asc (application/pgp-signature, inline)]

Information forwarded to bug-guix <at> gnu.org:
bug#26752; Package guix. (Mon, 04 Feb 2019 18:06:01 GMT) Full text and rfc822 format available.

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: Jelle Licht <jlicht <at> fsfe.org>, 26752 <at> debbugs.gnu.org, 29951 <at> debbugs.gnu.org
Subject: Re: bug#26752: Ansible & others' problems with wrapped
 '.ansible-real' scripts
Date: Mon, 04 Feb 2019 19:05:47 +0100
Hi,

Ricardo Wurmus <rekado <at> elephly.net> skribis:

> ludo <at> gnu.org (Ludovic Courtès) writes:
>> Jelle Licht <jlicht <at> fsfe.org> skribis:
>>
>>> The current ansible package is still brokenin the same way.
>>>
>>> Is there already an acceptable way of working around this problem?
>>> Otherwise I could send my (extremely hacky) workaround that adds a specific
>>> condition in the ansible source code to check for .ansible-real.
>>
>> For now I think we have to go with the hack.  Make sure to add a comment
>> linking to this bug report.
>
> There is a way to get around this.  It’s presented in #29951:
>
>     https://issues.guix.info/issue/29951
>
> Open issues with this include a hard-coded store prefix, but it’s no big
> obstacle.  Should we give this a try on core-updates?

Definitely, please do!

Thanks,
Ludo’.




Information forwarded to bug-guix <at> gnu.org:
bug#26752; Package guix. (Wed, 06 Feb 2019 22:15:02 GMT) Full text and rfc822 format available.

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: Jelle Licht <jlicht <at> fsfe.org>, 26752 <at> debbugs.gnu.org, 29951 <at> debbugs.gnu.org
Subject: Re: bug#26752: Ansible & others' problems with wrapped
 '.ansible-real' scripts
Date: Wed, 06 Feb 2019 23:14:04 +0100
Ricardo Wurmus <rekado <at> elephly.net> skribis:

> ludo <at> gnu.org (Ludovic Courtès) writes:
>> Jelle Licht <jlicht <at> fsfe.org> skribis:
>>
>>> The current ansible package is still brokenin the same way.
>>>
>>> Is there already an acceptable way of working around this problem?
>>> Otherwise I could send my (extremely hacky) workaround that adds a specific
>>> condition in the ansible source code to check for .ansible-real.
>>
>> For now I think we have to go with the hack.  Make sure to add a comment
>> linking to this bug report.
>
> There is a way to get around this.  It’s presented in #29951:
>
>     https://issues.guix.info/issue/29951

BTW, there’s also this patch series on this topic:

  https://issues.guix.info/issue/27003

What to do?

Ludo’.




Information forwarded to bug-guix <at> gnu.org:
bug#26752; Package guix. (Sat, 21 Mar 2020 20:26:02 GMT) Full text and rfc822 format available.

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

From: Brice Waegeneire <brice <at> waegenei.re>
To: 26752 <at> debbugs.gnu.org
Subject: Re: bug#26752: Ansible & others' problems with wrapped
 '.ansible-real' scripts
Date: Sat, 21 Mar 2020 20:25:15 +0000
Hello,

Arun Isaac <arunisaac <at> systemreboot.net> writes:
> FWIW, I fixed our ansible package in
> https://issues.guix.info/issue/33777 . This is not a general solution,
> but at least our ansible package works at the moment.

If this bug is about ansible malfunctioning then it should be closed,
Arun fixed it in 01cb4d47570c38812492bbc331b7b818e1b69fbb. I've tested
it myself successfully with the command from the IRC report[0] (linked
in the merged issue[2]).
On the other hand If it's about fixing it with the new wrap-script from
Ricardo I can send a patch, but the derivation will need guile as a new
input.

[0]: http://logs.guix.gnu.org/guix/2018-04-28.log#013741
[1]: https://issues.guix.info/issue/31299

Brice.




Information forwarded to bug-guix <at> gnu.org:
bug#26752; Package guix. (Sat, 21 Mar 2020 21:46:02 GMT) Full text and rfc822 format available.

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

From: Jelle Licht <jlicht <at> fsfe.org>
To: Brice Waegeneire <brice <at> waegenei.re>, 26752 <at> debbugs.gnu.org
Subject: Re: bug#26752: Ansible & others' problems with wrapped
 '.ansible-real' scripts
Date: Sat, 21 Mar 2020 22:45:27 +0100
Hi

Brice Waegeneire <brice <at> waegenei.re> writes:

> Hello,
>
> Arun Isaac <arunisaac <at> systemreboot.net> writes:
>> FWIW, I fixed our ansible package in
>> https://issues.guix.info/issue/33777 . This is not a general solution,
>> but at least our ansible package works at the moment.
>
> If this bug is about ansible malfunctioning then it should be closed,
> Arun fixed it in 01cb4d47570c38812492bbc331b7b818e1b69fbb. I've tested
> it myself successfully with the command from the IRC report[0] (linked
> in the merged issue[2]).
> On the other hand If it's about fixing it with the new wrap-script from
> Ricardo I can send a patch, but the derivation will need guile as a new
> input.

I would be okay with closing it, as ansible works fine right now. If the
current solution is deemed too brittle, or if the intent behind the new
wrap-script is that we use it everywhere we can, then not of course.

- Jelle




bug closed, send any further explanations to 26752 <at> debbugs.gnu.org and Jelle Licht <jlicht <at> fsfe.org> Request was from Brice Waegeneire <brice <at> waegenei.re> to control <at> debbugs.gnu.org. (Sat, 28 Mar 2020 10:40:02 GMT) Full text and rfc822 format available.

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Sat, 25 Apr 2020 11:24:04 GMT) Full text and rfc822 format available.

This bug report was last modified 3 years and 337 days ago.

Previous Next


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