From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp2 ([2001:41d0:2:4a6f::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms11 with LMTPS id sKfACrcpcF9wMQAA0tVLHw (envelope-from ) for ; Sun, 27 Sep 2020 05:57:11 +0000 Received: from aspmx1.migadu.com ([2001:41d0:2:4a6f::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp2 with LMTPS id iHaHBrcpcF+kMwAAB5/wlQ (envelope-from ) for ; Sun, 27 Sep 2020 05:57:11 +0000 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id 8A14E9402B0 for ; Sun, 27 Sep 2020 05:57:10 +0000 (UTC) Received: from localhost ([::1]:43658 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kMPgL-0006nO-Gi for larch@yhetil.org; Sun, 27 Sep 2020 01:57:09 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:55086) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kMPgE-0006nG-LA for guix-patches@gnu.org; Sun, 27 Sep 2020 01:57:02 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:36615) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1kMPgE-00004x-C5 for guix-patches@gnu.org; Sun, 27 Sep 2020 01:57:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1kMPgE-0000jH-BS for guix-patches@gnu.org; Sun, 27 Sep 2020 01:57:02 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#43628] Further Info Resent-From: Prafulla Giri Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Sun, 27 Sep 2020 05:57:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 43628 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: To: Julien Lepiller Cc: 43628@debbugs.gnu.org Received: via spool by 43628-submit@debbugs.gnu.org id=B43628.16011862132788 (code B ref 43628); Sun, 27 Sep 2020 05:57:02 +0000 Received: (at 43628) by debbugs.gnu.org; 27 Sep 2020 05:56:53 +0000 Received: from localhost ([127.0.0.1]:48161 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1kMPg4-0000io-CN for submit@debbugs.gnu.org; Sun, 27 Sep 2020 01:56:53 -0400 Received: from mail-pj1-f41.google.com ([209.85.216.41]:54111) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1kMPg1-0000iR-4h for 43628@debbugs.gnu.org; Sun, 27 Sep 2020 01:56:50 -0400 Received: by mail-pj1-f41.google.com with SMTP id t7so1725615pjd.3 for <43628@debbugs.gnu.org>; Sat, 26 Sep 2020 22:56:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=pUERo0RMtXw6Ml3paCUpHnzT1aDkKCDRUgtxNgNrZm8=; b=ZAM9fr+1zBAsmRdxwDoIdMAHYzc5Qxv79C+5OcF4J8ZaMqENxFKzSLR5ar0b7MLnBn kH5S7t0a4Bqb+FyTfyPk2pHK7tlcKBQeGoL/YkAq9pWFeYDrlGsHF2JitEfg026UW4Jc deQnCHROMEeeIOPLsNgLW1N0ey6McU3PinAxpEgQdiVVqV5So3b//CqpaZznzbWNrV3E TEGFyAn6MJ0r8nZcIgcTRlS0f1barVrhlRW7tJERa2EaV4dq7QWjxCoKVCA/2xW9GI+O oWAEZTMZzV4GY8ijZz33xH09B0+gj/dirrBgEEnPa2DegEWE5JRH8Q6+5QTU2i1yTaTA dyEA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=pUERo0RMtXw6Ml3paCUpHnzT1aDkKCDRUgtxNgNrZm8=; b=h6mpUSlfQjRylil+IakGAQkTQwS5quRptLPHhQoLHsodEZCBmdAh4QPheFlFlI6yV8 ERNpoHZdVky9wNtbWknrnxJl5BKV/xEAK+MwIGbbyOxP96yhm6jElWPsYG7skSikWmil 4Lw/2IQl8rNcKUJUGtzJZuLmSPY4IS5a31kGDGaAhHYQ2YtNrnpd4H5kF319LWWfXsmH mnyrXbApqbA1vtJW/kYtCPtDYdqrXluaBkdN30sq9uIgmhUazHADq+hmKFtXKMx8lLx0 2qwjyrgG5clKASGthgD3ql6E7CEHBF86GyA8uOdXgabUgyNHsYP27yV1cQaU6ZAAHfUr 5WpQ== X-Gm-Message-State: AOAM5321l9PPafBSXtLc2oBOGX/peJiHG8E5Lb2dNlfJj9gFI1/bMflk ukffWbH+tKqBhnBO6nEx0DupjDazMPQLVeRNm4Rhp5ZJd5xIXA== X-Google-Smtp-Source: ABdhPJw6inC5EH6dArOsfWSKroeo58jmkXXEezMb27rvYpg1PIgb5c8Ia1UOt5XlGTDlrssAbiJ0amsqi+5H6LRXLAg= X-Received: by 2002:a17:90b:ecc:: with SMTP id gz12mr4261447pjb.219.1601186202988; Sat, 26 Sep 2020 22:56:42 -0700 (PDT) MIME-Version: 1.0 References: <561466E3-006A-4460-9DEA-61A8DC16E7A8@lepiller.eu> <5B02C136-670D-4930-A461-731DA3D5221F@lepiller.eu> In-Reply-To: From: Prafulla Giri Date: Sun, 27 Sep 2020 11:41:31 +0545 Message-ID: Content-Type: multipart/alternative; boundary="000000000000d9341505b04536cc" X-Spam-Score: 0.0 (/) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-Spam-Score: -1.0 (-) X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+larch=yhetil.org@gnu.org Sender: "Guix-patches" X-Scanner: scn0 Authentication-Results: aspmx1.migadu.com; dkim=fail (rsa verify failed) header.d=gmail.com header.s=20161025 header.b=ZAM9fr+1; dmarc=fail reason="SPF not aligned (relaxed)" header.from=gmail.com (policy=none); spf=pass (aspmx1.migadu.com: domain of guix-patches-bounces@gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=guix-patches-bounces@gnu.org X-Spam-Score: 0.09 X-TUID: b6E2C0v4Yxqm --000000000000d9341505b04536cc Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable First and foremost: wow. A post-doc at just three years older than me. *cries in high-school drop-out* Super awesome! I have cleaned up my package definition as per your review: https://github.com/peanutbutterandcrackers/guix-packages/blob/master/aegisu= b.scm I will look into the matter then. Might take me a while, but I'll try. It's sad that framagit is closing. I think they were doing a great job with peertube and all... On Sun, Sep 27, 2020 at 3:00 AM Julien Lepiller wrote: > 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 a= n > 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 mig= ht > be needed. > > Third, the way you modify the arguments is=E2=80=A6 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 migh= t > 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 =C3=A9crit : >> >> 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 man= y >> 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/aeg= isub.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 runtim= e. >> >> 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 ver= y >> sure about that. >> >> On Sat, Sep 26, 2020 at 8:20 PM Julien Lepiller >> 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 t= he >>> internet calls you Mr ^^. No need to be so stiff, you can call me "Juli= en" >>> (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 =C3=A9crit : >>>> >>>> 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 - an= d 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 th= e >>>>> great work! >>>>> >>>>> On Sat, Sep 26, 2020 at 3:28 PM Julien Lepiller >>>>> wrote: >>>>> >>>>>> >>>>>> >>>>>> Hi Prafulla, >>>>>> >>>>>> Le 26 septembre 2020 03:31:38 GMT-04:00, Prafulla Giri < >>>>>> pratheblackdiamond@gmail.com> a =C3=A9crit : >>>>>> >Another information: >>>>>> > >>>>>> >I have other programs on my foreign system (installed via guix) tha= t >>>>>> >produce sound well, without any issues. They include: >>>>>> >espeak and kdenlive. >>>>>> > >>>>>> >Also, I did try making the union-build use copies instead of symlin= ks >>>>>> >using >>>>>> >(union-build ... #:symlink copy-recursively) but that didn't work o= ut >>>>>> >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 >>>>>> > >>>>>> >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 wi= ll >>>>>> >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, bu= t 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 yo= u >>>>>> actually want to configure that? Maybe try to copy this to a new fil= e >>>>>> .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 >>>>>> } >>>>>> >>>>> --000000000000d9341505b04536cc Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
First and foremost: wow. A post-doc at just three yea= rs older than me. *cries in high-school drop-out* Super awesome!
=
I have cleaned up my package definition as per your review: = https://github.com/peanutbutterandcrackers/guix-packages/= blob/master/aegisub.scm

I will look into the m= atter then. Might take me a while, but I'll try.

It's sad that framagit is closing. I think they were doing a great j= ob with peertube and all...

<= div dir=3D"ltr" class=3D"gmail_attr">On Sun, Sep 27, 2020 at 3:00 AM Julien= Lepiller <julien@lepiller.eu&= gt; wrote:
= 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 usi= ng derivation-outputs is a bad idea, and it probably only works because als= a-plugin doesn't depend on your modified alsa-lib (you'd get an inf= inite 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 t= he conhigure flags, you can simply use (assoc-ref %build-inputs "alsa-= plugin") when "alsa-plugin" is defined as an input to the pa= ckage.

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

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

I remember having dealt w= ith 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 i= t 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 d= ifference. It's probably too soon to call me a "senior" thoug= h, I'm still pretty young: only 3 years older than you :p

Le 26 septembre 2020 16:46:50 GMT-04:00, Prafulla Giri= <prat= heblackdiamond@gmail.com> a =C3=A9crit :
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.
(Al= so, 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!)


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 sys= tem? If it goes well, perhaps I should send a patch in for aegisub. But the= n 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 n= ot really done yet, but I'm very motivated :). It always feels weird wh= en 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 t= o take a look at alsa's sources to find out how it looks for its plugin= s exactly (maybe try to grep for PLUGIN or PLUGINDIR)?

Le 26 septembre 2020 08:07:34 GMT-04:00, Prafulla Giri <pratheblack= diamond@gmail.com> a =C3=A9crit :
Okay, I think I'm stuck now. lol



On Sat, Sep 26, 2020 at 3:42 PM Prafulla Giri <pratheblackdiam= ond@gmail.com> wrote:
Hello Mr. Lepiller,

=
Thank you for the tip. For the moment, I want to poke around this issu= e. 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 gu= ix-android seems so very cool! Please keep up the great work!

On S= at, 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 =C3=A9crit :
>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 o= ut
>either.
>
>Now, I'm going to take a look at fixing the pkg-config rules, and w= ill
>be
>updating this thread should that resolve this issue.
>
>On Sat, Sep 26, 2020 at 12:21 PM Prafulla Giri
><p= ratheblackdiamond@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 wil= l
>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: al= sa doesn't look in it's current directory for plugins, but in the d= irectory 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, whi= ch references alsa-plugins:pulseaudio. Maybe you actually want to configure= that? Maybe try to copy this to a new file .asoundrc:

pcm_type.pule {
=C2=A0 lib "/home/foo/.guix-profile/lib/alsa-lib/libasound_module_pcm_= pulse.so"
}

ctl_type.pulse {
=C2=A0 lib"/home/foo/.guix-profile/lib/alsa-lib/libasound_module_ctl_p= ulse.so"
}

pcm.!default {
=C2=A0 type pulse
}

ctl.!default {
=C2=A0 type pulse
}
--000000000000d9341505b04536cc--