[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#42229] [PATCH 8/9] gnu: jami: Fix compilation by providing necessar

From: Jan Wielkiewicz
Subject: [bug#42229] [PATCH 8/9] gnu: jami: Fix compilation by providing necessary webchat files.
Date: Tue, 7 Jul 2020 23:22:59 +0200


On Mon, 06 Jul 2020 22:19:42 +0100
Christopher Baines <> wrote:

> I'd perhaps just say instead that these files are packaged as
> jami-chatview which is used by the jami package, that's probably
> what's useful to know here.

> Because substitute* uses regular expressions, characters like . should
> probably be escaped. However, you might want to try something like the
> following:
>                (substitute* "CMakeLists.txt"
>                  (("src/web-chatview/.*") ""))
I use to forget this, don't know why. Testing how it works right now.

> It would be good if this could be clearer as to what the user gets if
> they install this package. I'm guessing because it's an input to Jami,
> it's probably not useful to have it in your profile?
Users don't have to know this. I made it private by changing
define-public to define.

> Also, if it's going to describe the contents of the package, I'd do it
> in a full sentance, so something like: "This package provides the
> HTML, CSS and JavaScript files for the web chatview for Jami".

> I think there's some Expat stuff in there too.
Yes, there is, added expat to license list. Shouldn't Jami files
contain explicit license statement though?

> Rather than copying, could this be a symlink? That way you're just
> using another store item, rather than copying the files from one to
> another.
Working on this. The build fails on the ".eslintrc.json". Guix for some
reason doesn't like dotfiles.

Jan Wielkiewicz

reply via email to

[Prev in Thread] Current Thread [Next in Thread]