https://bugzilla.redhat.com/show_bug.cgi?id=1398922
Benjamin Kircher changed:
What|Removed |Added
Status|NEW |CLOSED
Resolution|---
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
Robert-André Mauchin changed:
What|Removed |Added
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
Michal Schmidt changed:
What|Removed |Added
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #20 from Benjamin Kircher ---
Thanks, Dridi.
> Don't forget to find a sponsor in the mean time.
I'll do my best :)
--
You are receiving this mail because:
You are on the CC list for the bug.
You are
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #19 from Dridi Boukelmoune ---
I have some packaging work ahead, I'll have a look at your update then. Don't
forget to find a sponsor in the mean time. Maybe I can do the formal review,
but I can't approve
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #18 from Benjamin Kircher ---
Thanks Dridi for your work. New upstream version contains your changes so no
need to patch.
Changes:
- New upstream release 0.7
- Use %configure macro
- Do verbose build
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
Dridi Boukelmoune changed:
What|Removed |Added
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #16 from Benjamin Kircher ---
FYI, some changes related to ./configure script and Fedora packaging happened
in upstream here:
https://github.com/namhyung/uftrace/pull/98
Not in a release yet, though.
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #15 from Benjamin Kircher ---
A copr repository is available here:
https://copr.fedorainfracloud.org/coprs/bkircher/uftrace/
--
You are receiving this mail because:
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #14 from Benjamin Kircher ---
New upstream version.
New SPEC file:
https://raw.githubusercontent.com/bkircher/uftrace-fedora/master/uftrace.spec
New SRPM:
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #13 from Benjamin Kircher ---
Sorry for letting this lie around for 9 weeks.
> About GPLv2 or GPLv2+
You're right. It is GPLv2. Author confirmed this.
> 1) These BR are not needed: gcc
BR removed.
>
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #12 from Michael Schwendt ---
The fedora-review tool is equally helpful to the package maintainer as to the
reviewer.
About GPLv2 or GPLv2+, a simple grep turns up these false positives:
$ grep "or
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #11 from Benjamin Kircher ---
Hi Michael. Thank you for looking into this.
> > * I think license is GPLv2+, not GPLv2
>
> It's useless to write that without giving a rationale.
>
> Have you pointed
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #10 from Benjamin Kircher ---
New SPEC file:
https://raw.githubusercontent.com/bkircher/uftrace-fedora/master/uftrace.spec
New SRPM:
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #9 from Michael Schwendt ---
> * I think license is GPLv2+, not GPLv2
It's useless to write that without giving a rationale.
Have you pointed "fedora-review -b 1398922" at this ticket yet to see which
test
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #8 from Benjamin Kircher ---
Thank you.
> * ./configure -> %configure
> Usually, but unfortunately it's not the case right now.
Correct, configure is a shell script here.
> Main problem is that
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
Igor Gnatenko changed:
What|Removed |Added
Blocks||177841
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #6 from Benjamin Kircher ---
FYI, a link to a 6 minute lightning talk by the author about uftrace:
https://youtu.be/LNav5qvyK7I
--
You are receiving this mail because:
You are on the CC list for the
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #5 from Benjamin Kircher ---
> Sorry, as a new packager myself, I misunderstood the rules about .so files.
Oh, no problem. The question what these libraries are doing is quite justified.
--
You are
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #4 from Yunying Sun ---
(In reply to Benjamin Kircher from comment #2)
> Thank you.
>
> > 1. No version specified in %changelog.
> Fixed.
>
> > Static libraries are discouraged. Add "--disable-static" in
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #3 from Benjamin Kircher ---
I was able to talk to the author of uftrace about those libs. The libmcount*
libraries are linked into the target process (using LD_PRELOAD) rather than
uftrace itself. They
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
--- Comment #2 from Benjamin Kircher ---
Thank you.
> 1. No version specified in %changelog.
Fixed.
> Static libraries are discouraged. Add "--disable-static" in %configure, and
> add -devel package where *.so
https://bugzilla.redhat.com/show_bug.cgi?id=1398922
Yunying Sun changed:
What|Removed |Added
CC|
23 matches
Mail list logo