[Bug 2121585] Review Request: janus - An open source general purpose WebRTC server.

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

 



https://bugzilla.redhat.com/show_bug.cgi?id=2121585



--- Comment #27 from Renich Bon Ciric <renich@xxxxxxxxxxxxxxxx> ---
(In reply to Benson Muite from comment #25)
> Should the spec file have a Requires section? Requires brings in dependent
> packages needed when using Janus.  At the moment,
> only BuildRequires are listed.
> 
> Perhaps consider only adding demo files that have minimal Javascript
> dependencies, or Javascript dependencies that are already packaged?

Hello, Benson.

Well, the tools package has a requirement. Outside of it, janus, mostly, is
standalone, AFAIK. I'll double-check, though. 

(In reply to Benson Muite from comment #26)
> Typically .so files go in -devel packages and are softlinks to .so.* files,
> this also is true for subpackages.

In this case, remember that the .so files (the versioned ones) are plugins.
Very similar to NginX's modules. These are packaged in their respective
sub-packages. 

I am hoping to add a meta-package so that all plugins can be installed easily. 

> Issues when running fedora-review:
> a)
>     Note: Directories without known owners: /usr/include/janus/events,
>      /usr/share/janus/javascript, /usr/lib64/janus/plugins,
>      /usr/lib64/janus/transports, /usr/lib64/janus/events,
>      /usr/lib64/janus/loggers, /usr/share/janus/duktape,
>      /usr/include/janus, /usr/share/janus, /usr/include/janus/loggers,
>      /usr/include/janus/plugins, /usr/include/janus/transports,
>      /usr/lib64/janus, /usr/share/janus/streams,
>      /usr/share/janus/recordings, /etc/janus, /usr/share/janus/lua

Ah, the directories. I will add them. Thanks for htis. 

> ...
> /builddir/janus-plugins-textroom-debuginfo-1.1.0-1.fc38.x86_64.rpm
> ERROR: Command failed: 
>  # /usr/bin/systemd-nspawn -q -M d79e9c67ebb847689873e491ada18b81 -D
> /var/lib/mock/fedora-rawhide-x86_64-bootstrap/root -a
> --capability=cap_ipc_lock --bind=/tmp/mock-resolv.yfyj9_8u:/etc/resolv.conf
> --console=pipe --setenv=TERM=vt100 --setenv=SHELL=/bin/bash
> --setenv=HOME=/var/lib/mock/fedora-rawhide-x86_64/root/installation-homedir
> --setenv=HOSTNAME=mock --setenv=PATH=/usr/bin:/bin:/usr/sbin:/sbin
> --setenv=PROMPT_COMMAND=printf "\033]0;<mock-chroot>\007"
> --setenv=PS1=<mock-chroot> \s-\v\$  --setenv=LANG=C.UTF-8
> --setenv=LC_MESSAGES=C.UTF-8 --resolv-conf=off /usr/bin/dnf --installroot
> ...

It's not clear to me which command failed here and what the error was. :s


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2121585
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux