unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Julien Lepiller <julien@lepiller.eu>
To: Prafulla Giri <pratheblackdiamond@gmail.com>
Cc: 43628@debbugs.gnu.org
Subject: [bug#43628] Further Info
Date: Sat, 26 Sep 2020 17:15:40 -0400	[thread overview]
Message-ID: <A6EEC292-30BC-41F8-9DA5-96905D9FCBDD@lepiller.eu> (raw)
In-Reply-To: <CAFw+=j3YByxoAVVGAdsQNGrj1UO-tV18fCi-wYnG_w1dXgP_pA@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 6154 bytes --]

Thanks! On the guix system, aegisub already works properly (at least it did when I last used it), because the alsa config allows it to find pulseaudio.

There are three issues with your proposed change:

First using derivation-outputs is a bad idea, and it probably only works because alsa-plugin doesn't depend on your modified alsa-lib (you'd get an infinite loop I think). Since alsa-plugin doesn't depend on this modified alsa-lib, you don't need such a convoluted work around. When defining the conhigure flags, you can simply use (assoc-ref %build-inputs "alsa-plugin") when "alsa-plugin" is defined as an input to the package.

Second, setting plugindir at build-time is just asking for the same kind of issues you are trying to fix, only for finding another plugin that might be needed.

Third, the way you modify the arguments is… unconventional :) try using substitute-keyword-arguments instead. There are plenty of examples in the repo.

I remember having dealt with this issue before, and pushed a patch some 3 years ago: http://issues.guix.gnu.org/31239

Could you look at what happened in the meantime? I think it was reverted in favor of the asound.conf file we have in the guix system now. You might learn something if you find the reason why it was changed.

Re my other projects, framagit is going to close next year, so I'll be migrating to my own hosting, currently at git.lepiller.eu.

Ok for the cultural difference. It's probably too soon to call me a "senior" though, I'm still pretty young: only 3 years older than you :p

Le 26 septembre 2020 16:46:50 GMT-04:00, Prafulla Giri <pratheblackdiamond@gmail.com> a écrit :
>Oh, I come from a culture that does not call seniors by their first
>names.
>(:
>Yes! guix-android looks super awesome! I'm glad that you are working on
>that. You're awesome.
>(Also, I just realized that you've also got guix-home-manager and so
>many
>more cool repositories! I need to check those out, and soon!)
>
>I did manage to resolve this issue. Kinda'. Here:
>https://github.com/peanutbutterandcrackers/guix-packages/blob/master/aegisub.scm
>
>All I had to do was re-define 'alsa-lib' to point to the /gnu/store
>path of
>alsa-plugins. The /gnu/store path had to be figured out during runtime.
>
>Could you please check how this fares in a GuixSD system? If it goes
>well,
>perhaps I should send a patch in for aegisub. But then again, I wonder
>if I
>should make the changes in alsa-lib itself, but I am not very sure
>about
>that.
>
>On Sat, Sep 26, 2020 at 8:20 PM Julien Lepiller <julien@lepiller.eu>
>wrote:
>
>> Hehe, thanks for your interest in guix-android. It's not really done
>yet,
>> but I'm very motivated :). It always feels weird when someone on the
>> internet calls you Mr ^^. No need to be so stiff, you can call me
>"Julien"
>> (and I suppose it's the norm to call people by their first name in
>this
>> kind of internet community).
>>
>> Maybe try to run aegisub with strace -f, you might find out
>something. You
>> may have to take a look at alsa's sources to find out how it looks
>for its
>> plugins exactly (maybe try to grep for PLUGIN or PLUGINDIR)?
>>
>> Le 26 septembre 2020 08:07:34 GMT-04:00, Prafulla Giri <
>> pratheblackdiamond@gmail.com> a écrit :
>>>
>>> Okay, I think I'm stuck now. lol
>>>
>>>
>>>
>>> On Sat, Sep 26, 2020 at 3:42 PM Prafulla Giri <
>>> pratheblackdiamond@gmail.com> wrote:
>>>
>>>> Hello Mr. Lepiller,
>>>>
>>>> Thank you for the tip. For the moment, I want to poke around this
>issue.
>>>> Perhaps I get absolutely tired of it (or find a good hack - and an
>ugly
>>>> patch). I did manage to learn a bit more about union-builds in
>general. So,
>>>> that's a silver lining.
>>>>
>>>> Also, your work on guix-android seems so very cool! Please keep up
>the
>>>> great work!
>>>>
>>>> On Sat, Sep 26, 2020 at 3:28 PM Julien Lepiller
><julien@lepiller.eu>
>>>> wrote:
>>>>
>>>>>
>>>>>
>>>>> Hi Prafulla,
>>>>>
>>>>> Le 26 septembre 2020 03:31:38 GMT-04:00, Prafulla Giri <
>>>>> pratheblackdiamond@gmail.com> a écrit :
>>>>> >Another information:
>>>>> >
>>>>> >I have other programs on my foreign system (installed via guix)
>that
>>>>> >produce sound well, without any issues. They include:
>>>>> >espeak and kdenlive.
>>>>> >
>>>>> >Also, I did try making the union-build use copies instead of
>symlinks
>>>>> >using
>>>>> >(union-build ... #:symlink copy-recursively) but that didn't work
>out
>>>>> >either.
>>>>> >
>>>>> >Now, I'm going to take a look at fixing the pkg-config rules, and
>will
>>>>> >be
>>>>> >updating this thread should that resolve this issue.
>>>>> >
>>>>> >On Sat, Sep 26, 2020 at 12:21 PM Prafulla Giri
>>>>> ><pratheblackdiamond@gmail.com>
>>>>> >wrote:
>>>>> >
>>>>> >> I forgot to mention, but the union-build works. If one `guix
>build`s
>>>>> >the
>>>>> >> union package, and cd into the /gnu/store/...-union-build, one
>will
>>>>> >see
>>>>> >> that things are there as it should be (lib/normal-alsa-*.so and
>>>>> >> lib/alsa-lib/*pulse*.so).
>>>>> >>
>>>>> >> It's just that aegisub seems to be looking from the
>>>>> >> /gnu/store/normal-alsa-lib-dir instead of
>>>>> >/gnu/store/union-alsa-package.
>>>>> >>
>>>>>
>>>>> I think the issue with the union-build is that it is completely
>>>>> ignored: alsa doesn't look in it's current directory for plugins,
>but in
>>>>> the directory where it's been configured (at build time) to look
>for.
>>>>>
>>>>> On the Guix System, there is a service that creates a
>/etc/asound.conf,
>>>>> which references alsa-plugins:pulseaudio. Maybe you actually want
>to
>>>>> configure that? Maybe try to copy this to a new file .asoundrc:
>>>>>
>>>>> pcm_type.pule {
>>>>>   lib
>>>>>
>"/home/foo/.guix-profile/lib/alsa-lib/libasound_module_pcm_pulse.so"
>>>>> }
>>>>>
>>>>> ctl_type.pulse {
>>>>>
>>>>>
>lib"/home/foo/.guix-profile/lib/alsa-lib/libasound_module_ctl_pulse.so"
>>>>> }
>>>>>
>>>>> pcm.!default {
>>>>>   type pulse
>>>>> }
>>>>>
>>>>> ctl.!default {
>>>>>   type pulse
>>>>> }
>>>>>
>>>>

[-- Attachment #2: Type: text/html, Size: 7760 bytes --]

  reply	other threads:[~2020-09-26 21:16 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-26  6:15 [bug#43628] Request for patch evaluation/review Prafulla Giri
2020-09-26  6:36 ` [bug#43628] Further Info Prafulla Giri
2020-09-26  7:31   ` Prafulla Giri
2020-09-26  9:43     ` Julien Lepiller
2020-09-26  9:57       ` Prafulla Giri
2020-09-26 12:07         ` Prafulla Giri
2020-09-26 14:35           ` Julien Lepiller
2020-09-26 20:46             ` Prafulla Giri
2020-09-26 21:15               ` Julien Lepiller [this message]
2020-09-27  5:56                 ` Prafulla Giri
2020-09-27 11:09                   ` Julien Lepiller
2020-09-28 14:43                     ` Prafulla Giri
2020-09-28 15:20                       ` Julien Lepiller
2020-09-26  9:51 ` [bug#43628] Progress (?) Prafulla Giri
2020-09-28 17:02 ` [bug#43628] Request for patch evaluation/review Leo Famulari

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=A6EEC292-30BC-41F8-9DA5-96905D9FCBDD@lepiller.eu \
    --to=julien@lepiller.eu \
    --cc=43628@debbugs.gnu.org \
    --cc=pratheblackdiamond@gmail.com \
    /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).