linux-trace-devel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Steven Rostedt <rostedt@goodmis.org>
To: Troy Engel <troyengel@gmail.com>
Cc: Linux Trace Devel <linux-trace-devel@vger.kernel.org>
Subject: Re: Update home url for trace-cmd and kernelshark
Date: Mon, 22 Jul 2019 21:17:40 -0400	[thread overview]
Message-ID: <20190722211740.54493d12@oasis.local.home> (raw)
In-Reply-To: <CAO7q0mnKAAv9u+vGrzvnu=w4GBRvw7Aq+43BFmE6YKkY_XzP-A@mail.gmail.com>

On Mon, 22 Jul 2019 18:26:16 -0500
Troy Engel <troyengel@gmail.com> wrote:

> Hi Steven,
> 
> Can do -- right now we have the stable pinned at 2.7 which doesn't exist in
> the new upstream repo, it appears the tags started with 2.8. I edited the

Right. I just pushed trace-cmd-v2.7 tag to the new repo ;-)

> build file locally but it's probably a dead end with 2.7 (it'll die out
> naturally) - 2.7 is pinned because we have the older GTK kernelshark binary
> in there, trace-cmd 2.8.x will release with KS 1.0 to supply a clean
> upgrade to users. It's also a good clean break if those Python 3 patches
> merge, we use the new hotness to introduce that change as well (gtk -> qt,
> py2 -> py3, 1pkg -> 2pkgs).

I need to push the patches. Haven't done that yet. I will tomorrow.

> 
> For the trace-cmd-git package working off HEAD, now that trace-cmd has to

Note, please use the trace-cmd-stable-v2.8 branch. HEAD for master will
soon be holding development code for 2.9, and will be quite unstable.

> be installed first to compile kernelshark 1.0 we need to break this apart
> into two separate packages which I just haven't done yet; I have it also

It's good to have the separation. We plan on working on splitting
KernelShark out of the trace-cmd git repo soon. But I don't think it's
a good idea to make the build of KernelShark depend on trace-cmd being
installed.


> edited locally in the master build file - once y'all tag KS 1.0 then we
> split this single/monolithic version we've been working with out into two
> packages solving the current chicken & egg problem, which then gets pushed
> to stable (non-git). Based on your projections in the other email, within a
> week or so but no pressure.

:-)  Last year I said KernelShark 1.0 will be released by August.
August came and went and people asked me where KernelShark 1.0 is? I
told them, I did not specify *which* August ;-) But I better not let
that happen again.

> 
> tl;dr - will do, just can't push the commits right away; we'll get the URL
> changed with the packaging split/rebuild needed for trace-cmd 2.8.x / KS
> 1.0 happening soon.

With a trace-cmd-v2.7 tag in the main repo, can you make the update?

Thanks Troy!

-- Steve

      parent reply	other threads:[~2019-07-23  1:17 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-22 19:03 Update home url for trace-cmd and kernelshark Steven Rostedt
     [not found] ` <CAO7q0mnKAAv9u+vGrzvnu=w4GBRvw7Aq+43BFmE6YKkY_XzP-A@mail.gmail.com>
2019-07-23  1:17   ` Steven Rostedt [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190722211740.54493d12@oasis.local.home \
    --to=rostedt@goodmis.org \
    --cc=linux-trace-devel@vger.kernel.org \
    --cc=troyengel@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).