subject:"\[Bug 1305655\] Review Request\: liblsl \- Lab streaming layer API"

[Bug 1305655] Review Request: liblsl - Lab streaming layer API

2016-02-14 Thread bugzilla

https://bugzilla.redhat.com/show_bug.cgi?id=1305655 Dmitry Mikhirev changed: What|Removed |Added Status|POST|CLOSED Resolution|---

[Bug 1305655] Review Request: liblsl - Lab streaming layer API

2016-02-14 Thread bugzilla

https://bugzilla.redhat.com/show_bug.cgi?id=1305655 --- Comment #8 from Dmitry Mikhirev --- Thank you for explanation, I added provides as you adviced. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and compone

[Bug 1305655] Review Request: liblsl - Lab streaming layer API

2016-02-14 Thread bugzilla

https://bugzilla.redhat.com/show_bug.cgi?id=1305655 --- Comment #7 from Zbigniew Jędrzejewski-Szmek --- The idea is to provide the information that package A is using code B. Would there be no bundling, this information would be provided by BuildRequires or other dependency. We want to provide

[Bug 1305655] Review Request: liblsl - Lab streaming layer API

2016-02-14 Thread bugzilla

https://bugzilla.redhat.com/show_bug.cgi?id=1305655 --- Comment #6 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/liblsl -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to t

[Bug 1305655] Review Request: liblsl - Lab streaming layer API

2016-02-14 Thread bugzilla

https://bugzilla.redhat.com/show_bug.cgi?id=1305655 --- Comment #5 from Dmitry Mikhirev --- > You should also add > Provides: bundled(boost-endian) > Provides: bundles(portable-archive) > I don't think it'll make any practical difference, but the guidelines require > > that [1], and there's no

[Bug 1305655] Review Request: liblsl - Lab streaming layer API

2016-02-13 Thread bugzilla

https://bugzilla.redhat.com/show_bug.cgi?id=1305655 Zbigniew Jędrzejewski-Szmek changed: What|Removed |Added Status|ASSIGNED|POST Flags|f

[Bug 1305655] Review Request: liblsl - Lab streaming layer API

2016-02-13 Thread bugzilla

https://bugzilla.redhat.com/show_bug.cgi?id=1305655 --- Comment #3 from Dmitry Mikhirev --- > I'm don't think boost license applies to the binary rpm You are right, the Boost license does not apply to the binary package. It is used only for headers that are not installed. > Do not use %define,

[Bug 1305655] Review Request: liblsl - Lab streaming layer API

2016-02-09 Thread bugzilla

https://bugzilla.redhat.com/show_bug.cgi?id=1305655 --- Comment #2 from Dmitry Mikhirev --- Thank you Zbigniew! > The package uses some bundled libraries: pugixml, portable_archive, boost > endian. LSL version of endian differs from upstream boost endian (probably it is a very old fork) and

[Bug 1305655] Review Request: liblsl - Lab streaming layer API

2016-02-09 Thread bugzilla

https://bugzilla.redhat.com/show_bug.cgi?id=1305655 --- Comment #1 from Zbigniew Jędrzejewski-Szmek --- Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated = MUST items = C/C++: [x]: Package does not contain kernel modules. [x]: Pac