[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Bug 976793 depends on bug 1972445, which changed state. Bug 1972445 Summary: Review Request: servus - Zeroconf discovery in C++ https://bugzilla.redhat.com/show_bug.cgi?id=1972445 What|Removed |Added

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Otto Urpelainen changed: What|Removed |Added Flags||needinfo?(jskarvad@redhat.c

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Otto Urpelainen changed: What|Removed |Added Flags||fedora-review? --- Comment #26 from

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 --- Comment #25 from Jaroslav Škarvada --- Spec URL: https://jskarvad.fedorapeople.org/lunchbox/lunchbox.spec SRPM URL: https://jskarvad.fedorapeople.org/lunchbox/lunchbox-1.17.0-2.fc33.src.rpm -- You are receiving this mail because: You

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Jaroslav Škarvada changed: What|Removed |Added Flags|needinfo?(jskarvad@redhat.c | |om)

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Otto Urpelainen changed: What|Removed |Added Flags||needinfo?(jskarvad@redhat.c

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 --- Comment #22 from Otto Urpelainen --- Jaroslav, Thank you for continuing this effort. For the tests, consider disabling only the problematic test and let others run, either by patching of whatever filtering option the test runner may have.

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 --- Comment #21 from Jaroslav Škarvada --- Spec URL: https://jskarvad.fedorapeople.org/lunchbox/lunchbox.spec SRPM URL: https://jskarvad.fedorapeople.org/lunchbox/lunchbox-1.17.0-1.fc33.src.rpm I used lunchbox name with lowercase 'l' which

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 --- Comment #20 from Jaroslav Škarvada --- Tests temporary disabled due to: https://github.com/Eyescale/Lunchbox/issues/330 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Jaroslav Škarvada changed: What|Removed |Added Depends On||1972445 Referenced Bugs:

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 --- Comment #19 from Jaroslav Škarvada --- Unfortunately upstream complicates it even more. Now it requires Servus (bug 1972445) and what's worse it's bundling it as a git submodule. I encouraged upstream to add support for building with the

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Jaroslav Škarvada changed: What|Removed |Added Flags|needinfo?(jskarvad@redhat.c | |om)

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Otto Urpelainen changed: What|Removed |Added CC||otu...@iki.fi Flags|

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Mattia Verga changed: What|Removed |Added CC||mattia.ve...@protonmail.com ---

needinfo canceled: [Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

Product: Fedora Version: rawhide Component: Package Review Jaroslav Škarvada has canceled Package Review 's request for Jaroslav Škarvada 's needinfo: Bug 976793: Review Request: libLunchbox - C++ library for multi-threaded programming https://bugzilla.redhat.com/show_bug.cgi?id=976793

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Jaroslav Škarvada changed: What|Removed |Added Flags|needinfo?(jskarvad@redhat.c | |om)

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 James Hogarth changed: What|Removed |Added Status|ASSIGNED|NEW

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 James Hogarth changed: What|Removed |Added Flags|

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Jaroslav Škarvada changed: What|Removed |Added Flags|needinfo?(jskarvad@redhat.c |

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 --- Comment #11 from Jaroslav Škarvada --- (In reply to Jonathan Wakely from comment #8) > I wondered why the upstream still has its own thread library, instead of > using std::thread / boost::thread etc. > > And indeed

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 James Hogarth changed: What|Removed |Added CC|

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Jonathan Wakely jwak...@redhat.com changed: What|Removed |Added CC||jwak...@redhat.com

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Jaroslav Škarvada jskar...@redhat.com changed: What|Removed |Added Flags|needinfo?(jskarvad@redhat.c |

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Christopher Meng cicku...@gmail.com changed: What|Removed |Added CC||jskar...@redhat.com

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Christopher Meng cicku...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Christopher Meng cicku...@gmail.com changed: What|Removed |Added CC||cicku...@gmail.com

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 --- Comment #5 from Michael Schwendt mschwe...@gmail.com --- Rest assured, it wouldn't be the first project to do that. Project name foo releases library with SONAME libfoo.so.N in tarball libfoo with headers in /usr/include/foo (#include

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 --- Comment #3 from Jaroslav Škarvada jskar...@redhat.com --- (In reply to Michael Schwendt from comment #2) Thanks for the comments, I will try to go through them probably during next week and provide new version. -- You are receiving this

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 --- Comment #4 from Jaroslav Škarvada jskar...@redhat.com --- Reply from Stefan regarding the rename to libLunchbox: Hi Jaroslav, I've looked into Lunchbox vs. libLunchbox and came to the conclusion that only the package name should change,

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 Mario Blättermann mario.blaetterm...@gmail.com changed: What|Removed |Added Summary|Review Request: Lunchbox - |Review

[Bug 976793] Review Request: libLunchbox - C++ library for multi-threaded programming

https://bugzilla.redhat.com/show_bug.cgi?id=976793 --- Comment #2 from Michael Schwendt mschwe...@gmail.com --- This will be non-trivial to review, and I dunno whether I will find enough time this weekend to do everything. How deeply have you reviewed the package already? rpmlint tells: