Linux-Trace-Devel Archive on lore.kernel.org
 help / color / Atom feed
* [ANNOUNCE] libtraceevent.git
@ 2020-10-07 17:07 Steven Rostedt
  2020-10-12 10:12 ` Jiri Olsa
  0 siblings, 1 reply; 17+ messages in thread
From: Steven Rostedt @ 2020-10-07 17:07 UTC (permalink / raw)
  To: LKML, Linux Trace Devel
  Cc: Zamir SUN, Arnaldo Carvalho de Melo, Jiri Olsa, zsun,
	Vitaly Chikunov, Tzvetomir Stoyanov, Yordan Karadzhov,
	Ben Hutchings, Sudip Mukherjee, Tony Jones, John Kacur,
	Clark Williams, powertop, Al Stone, Mauro Carvalho Chehab

I split out tools/lib/traceevent from the kernel tree using "git subtree",
which recreates all the commits of a directory and makes that directory a
stand alone. I then updated the Makefiles, and copied over some of the
header files used to build the library. I pushed this up to:

  https://git.kernel.org/pub/scm/libs/libtrace/libtraceevent.git/

My hope is that this will now be the source of all updates to the
libtraceevent library that can be used as a stand alone package that both
perf and tracecmd can use. I would also like powertop and rasdaemon to use
this as well.

-- Steve

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-07 17:07 [ANNOUNCE] libtraceevent.git Steven Rostedt
@ 2020-10-12 10:12 ` Jiri Olsa
  2020-10-12 10:19   ` Zamir SUN
  2020-10-12 15:19   ` Steven Rostedt
  0 siblings, 2 replies; 17+ messages in thread
From: Jiri Olsa @ 2020-10-12 10:12 UTC (permalink / raw)
  To: Steven Rostedt
  Cc: LKML, Linux Trace Devel, Zamir SUN, Arnaldo Carvalho de Melo,
	zsun, Vitaly Chikunov, Tzvetomir Stoyanov, Yordan Karadzhov,
	Ben Hutchings, Sudip Mukherjee, Tony Jones, John Kacur,
	Clark Williams, powertop, Al Stone, Mauro Carvalho Chehab

On Wed, Oct 07, 2020 at 01:07:50PM -0400, Steven Rostedt wrote:
> I split out tools/lib/traceevent from the kernel tree using "git subtree",
> which recreates all the commits of a directory and makes that directory a
> stand alone. I then updated the Makefiles, and copied over some of the
> header files used to build the library. I pushed this up to:
> 
>   https://git.kernel.org/pub/scm/libs/libtrace/libtraceevent.git/
> 
> My hope is that this will now be the source of all updates to the
> libtraceevent library that can be used as a stand alone package that both
> perf and tracecmd can use. I would also like powertop and rasdaemon to use
> this as well.

hi,
I'm adding this as fedora package, is there a source arhive somewhere
in git.kernel.org for libtraceevent that spec could download?

thanks,
jirka


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-12 10:12 ` Jiri Olsa
@ 2020-10-12 10:19   ` Zamir SUN
  2020-10-12 10:35     ` Jiri Olsa
  2020-10-12 15:19   ` Steven Rostedt
  1 sibling, 1 reply; 17+ messages in thread
From: Zamir SUN @ 2020-10-12 10:19 UTC (permalink / raw)
  To: Jiri Olsa
  Cc: Steven Rostedt, LKML, Linux Trace Devel,
	Arnaldo Carvalho de Melo, Ziqian SUN (Zamir),
	Vitaly Chikunov, Tzvetomir Stoyanov, Yordan Karadzhov,
	Ben Hutchings, Sudip Mukherjee, Tony Jones, John Kacur,
	Clark Williams, powertop, Al Stone, Mauro Carvalho Chehab

On Mon, Oct 12, 2020 at 6:12 PM Jiri Olsa <jolsa@redhat.com> wrote:
>
> On Wed, Oct 07, 2020 at 01:07:50PM -0400, Steven Rostedt wrote:
> > I split out tools/lib/traceevent from the kernel tree using "git subtree",
> > which recreates all the commits of a directory and makes that directory a
> > stand alone. I then updated the Makefiles, and copied over some of the
> > header files used to build the library. I pushed this up to:
> >
> >   https://git.kernel.org/pub/scm/libs/libtrace/libtraceevent.git/
> >
> > My hope is that this will now be the source of all updates to the
> > libtraceevent library that can be used as a stand alone package that both
> > perf and tracecmd can use. I would also like powertop and rasdaemon to use
> > this as well.
>
> hi,
> I'm adding this as fedora package, is there a source arhive somewhere
> in git.kernel.org for libtraceevent that spec could download?
>

Hi Jiri,

I was also working on it before. My way to get the source code is to
manually generate the tarball like what trace-cmd did before.

See my copr
https://copr.fedorainfracloud.org/coprs/zsun/test/build/1699358/
And the spec

https://download.copr.fedorainfracloud.org/results/zsun/test/fedora-rawhide-x86_64/01699358-libtraceevent/libtraceevent.spec

I haven't have time to follow-up Steven's reply of Oct 9th about the
document compiling yet. If you are willing to work on it now maybe you
can start on top of mine. Otherwise I'll follow-up the packaging
later.

HTH.

> thanks,
> jirka
>

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-12 10:19   ` Zamir SUN
@ 2020-10-12 10:35     ` Jiri Olsa
  0 siblings, 0 replies; 17+ messages in thread
From: Jiri Olsa @ 2020-10-12 10:35 UTC (permalink / raw)
  To: Zamir SUN
  Cc: Steven Rostedt, LKML, Linux Trace Devel,
	Arnaldo Carvalho de Melo, Ziqian SUN (Zamir),
	Vitaly Chikunov, Tzvetomir Stoyanov, Yordan Karadzhov,
	Ben Hutchings, Sudip Mukherjee, Tony Jones, John Kacur,
	Clark Williams, powertop, Al Stone, Mauro Carvalho Chehab

On Mon, Oct 12, 2020 at 06:19:14PM +0800, Zamir SUN wrote:
> On Mon, Oct 12, 2020 at 6:12 PM Jiri Olsa <jolsa@redhat.com> wrote:
> >
> > On Wed, Oct 07, 2020 at 01:07:50PM -0400, Steven Rostedt wrote:
> > > I split out tools/lib/traceevent from the kernel tree using "git subtree",
> > > which recreates all the commits of a directory and makes that directory a
> > > stand alone. I then updated the Makefiles, and copied over some of the
> > > header files used to build the library. I pushed this up to:
> > >
> > >   https://git.kernel.org/pub/scm/libs/libtrace/libtraceevent.git/
> > >
> > > My hope is that this will now be the source of all updates to the
> > > libtraceevent library that can be used as a stand alone package that both
> > > perf and tracecmd can use. I would also like powertop and rasdaemon to use
> > > this as well.
> >
> > hi,
> > I'm adding this as fedora package, is there a source arhive somewhere
> > in git.kernel.org for libtraceevent that spec could download?
> >
> 
> Hi Jiri,
> 
> I was also working on it before. My way to get the source code is to
> manually generate the tarball like what trace-cmd did before.
> 
> See my copr
> https://copr.fedorainfracloud.org/coprs/zsun/test/build/1699358/
> And the spec
> 
> https://download.copr.fedorainfracloud.org/results/zsun/test/fedora-rawhide-x86_64/01699358-libtraceevent/libtraceevent.spec
> 
> I haven't have time to follow-up Steven's reply of Oct 9th about the
> document compiling yet. If you are willing to work on it now maybe you
> can start on top of mine. Otherwise I'll follow-up the packaging
> later.

we discussed on irc and Zamir will continue on this ;-)
plz let me know if I can help with something

thanks,
jirka


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-12 10:12 ` Jiri Olsa
  2020-10-12 10:19   ` Zamir SUN
@ 2020-10-12 15:19   ` Steven Rostedt
  2020-10-12 15:32     ` Sudip Mukherjee
  2020-10-12 18:41     ` Tony Jones
  1 sibling, 2 replies; 17+ messages in thread
From: Steven Rostedt @ 2020-10-12 15:19 UTC (permalink / raw)
  To: Jiri Olsa
  Cc: LKML, Linux Trace Devel, Zamir SUN, Arnaldo Carvalho de Melo,
	zsun, Vitaly Chikunov, Tzvetomir Stoyanov, Yordan Karadzhov,
	Ben Hutchings, Sudip Mukherjee, Tony Jones, John Kacur,
	Clark Williams, powertop, Al Stone, Mauro Carvalho Chehab

On Mon, 12 Oct 2020 12:12:08 +0200
Jiri Olsa <jolsa@redhat.com> wrote:

> On Wed, Oct 07, 2020 at 01:07:50PM -0400, Steven Rostedt wrote:
> > I split out tools/lib/traceevent from the kernel tree using "git subtree",
> > which recreates all the commits of a directory and makes that directory a
> > stand alone. I then updated the Makefiles, and copied over some of the
> > header files used to build the library. I pushed this up to:
> > 
> >   https://git.kernel.org/pub/scm/libs/libtrace/libtraceevent.git/
> > 
> > My hope is that this will now be the source of all updates to the
> > libtraceevent library that can be used as a stand alone package that both
> > perf and tracecmd can use. I would also like powertop and rasdaemon to use
> > this as well.  
> 
> hi,
> I'm adding this as fedora package, is there a source arhive somewhere
> in git.kernel.org for libtraceevent that spec could download?
> 

Hi Jiri!

Once it's shown that it works for all the package maintainers, I will tag
it which should create the tarballs automatically on the above link. But I
wanted to fix all the packaging bugs before doing so. I hope this doesn't
make it into a catch-22. Where you can't package till there's a source
tarball, but I can't make a source tarball until I know you can package
it ;-)

-- Steve


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-12 15:19   ` Steven Rostedt
@ 2020-10-12 15:32     ` Sudip Mukherjee
  2020-10-12 18:41     ` Tony Jones
  1 sibling, 0 replies; 17+ messages in thread
From: Sudip Mukherjee @ 2020-10-12 15:32 UTC (permalink / raw)
  To: Steven Rostedt
  Cc: Jiri Olsa, LKML, Linux Trace Devel, Zamir SUN,
	Arnaldo Carvalho de Melo, zsun, Vitaly Chikunov,
	Tzvetomir Stoyanov, Yordan Karadzhov, Ben Hutchings, Tony Jones,
	John Kacur, Clark Williams, powertop, Al Stone,
	Mauro Carvalho Chehab

On Mon, Oct 12, 2020 at 4:19 PM Steven Rostedt <rostedt@goodmis.org> wrote:
>
> On Mon, 12 Oct 2020 12:12:08 +0200
> Jiri Olsa <jolsa@redhat.com> wrote:
>
> > On Wed, Oct 07, 2020 at 01:07:50PM -0400, Steven Rostedt wrote:
> > > I split out tools/lib/traceevent from the kernel tree using "git subtree",
> > > which recreates all the commits of a directory and makes that directory a
> > > stand alone. I then updated the Makefiles, and copied over some of the
> > > header files used to build the library. I pushed this up to:
> > >
> > >   https://git.kernel.org/pub/scm/libs/libtrace/libtraceevent.git/
> > >
> > > My hope is that this will now be the source of all updates to the
> > > libtraceevent library that can be used as a stand alone package that both
> > > perf and tracecmd can use. I would also like powertop and rasdaemon to use
> > > this as well.
> >
> > hi,
> > I'm adding this as fedora package, is there a source arhive somewhere
> > in git.kernel.org for libtraceevent that spec could download?
> >
>
> Hi Jiri!
>
> Once it's shown that it works for all the package maintainers, I will tag
> it which should create the tarballs automatically on the above link. But I
> wanted to fix all the packaging bugs before doing so. I hope this doesn't
> make it into a catch-22. Where you can't package till there's a source
> tarball, but I can't make a source tarball until I know you can package
> it ;-)

For Debian I have raised https://bugs.debian.org/971976 but I will
package it locally today just to check there is no packaging bugs for
our packaging.


-- 
Regards
Sudip

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-12 15:19   ` Steven Rostedt
  2020-10-12 15:32     ` Sudip Mukherjee
@ 2020-10-12 18:41     ` Tony Jones
  2020-10-12 18:52       ` Jiri Olsa
  2020-10-12 19:17       ` Steven Rostedt
  1 sibling, 2 replies; 17+ messages in thread
From: Tony Jones @ 2020-10-12 18:41 UTC (permalink / raw)
  To: Steven Rostedt
  Cc: Jiri Olsa, LKML, Linux Trace Devel, Zamir SUN,
	Arnaldo Carvalho de Melo, zsun, Vitaly Chikunov,
	Tzvetomir Stoyanov, Yordan Karadzhov, Ben Hutchings,
	Sudip Mukherjee, John Kacur, Clark Williams, powertop, Al Stone,
	Mauro Carvalho Chehab

On Mon, Oct 12, 2020 at 11:19:50AM -0400, Steven Rostedt wrote:

> Once it's shown that it works for all the package maintainers, I will tag
> it which should create the tarballs automatically on the above link.

Hi.

It builds fine for me after manually creating the tarball from git.  
Once there is an official versioned tarball I'll push it into
openSUSE.

I presume some perf Makefile changes will be forthcoming to use it,
rather than continuing to force build it out of TRACE_EVENT_DIR

Tony

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-12 18:41     ` Tony Jones
@ 2020-10-12 18:52       ` Jiri Olsa
  2020-10-12 19:13         ` Steven Rostedt
  2020-10-12 19:17       ` Steven Rostedt
  1 sibling, 1 reply; 17+ messages in thread
From: Jiri Olsa @ 2020-10-12 18:52 UTC (permalink / raw)
  To: Tony Jones
  Cc: Steven Rostedt, LKML, Linux Trace Devel, Zamir SUN,
	Arnaldo Carvalho de Melo, zsun, Vitaly Chikunov,
	Tzvetomir Stoyanov, Yordan Karadzhov, Ben Hutchings,
	Sudip Mukherjee, John Kacur, Clark Williams, powertop, Al Stone,
	Mauro Carvalho Chehab

On Mon, Oct 12, 2020 at 11:41:20AM -0700, Tony Jones wrote:
> On Mon, Oct 12, 2020 at 11:19:50AM -0400, Steven Rostedt wrote:
> 
> > Once it's shown that it works for all the package maintainers, I will tag
> > it which should create the tarballs automatically on the above link.
> 
> Hi.
> 
> It builds fine for me after manually creating the tarball from git.  
> Once there is an official versioned tarball I'll push it into
> openSUSE.
> 
> I presume some perf Makefile changes will be forthcoming to use it,
> rather than continuing to force build it out of TRACE_EVENT_DIR

right, we need to detect it in features like any other library
and use it if it's found

jirka


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-12 18:52       ` Jiri Olsa
@ 2020-10-12 19:13         ` Steven Rostedt
  0 siblings, 0 replies; 17+ messages in thread
From: Steven Rostedt @ 2020-10-12 19:13 UTC (permalink / raw)
  To: Jiri Olsa
  Cc: Tony Jones, LKML, Linux Trace Devel, Zamir SUN,
	Arnaldo Carvalho de Melo, zsun, Vitaly Chikunov,
	Tzvetomir Stoyanov, Yordan Karadzhov, Ben Hutchings,
	Sudip Mukherjee, John Kacur, Clark Williams, powertop, Al Stone,
	Mauro Carvalho Chehab

On Mon, 12 Oct 2020 20:52:17 +0200
Jiri Olsa <jolsa@redhat.com> wrote:

> > I presume some perf Makefile changes will be forthcoming to use it,
> > rather than continuing to force build it out of TRACE_EVENT_DIR  
> 
> right, we need to detect it in features like any other library
> and use it if it's found

I need to do the same with trace-cmd.

-- Steve

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-12 18:41     ` Tony Jones
  2020-10-12 18:52       ` Jiri Olsa
@ 2020-10-12 19:17       ` Steven Rostedt
  2020-10-12 20:24         ` Sudip Mukherjee
  2020-10-13  3:06         ` Zamir SUN
  1 sibling, 2 replies; 17+ messages in thread
From: Steven Rostedt @ 2020-10-12 19:17 UTC (permalink / raw)
  To: Tony Jones
  Cc: Jiri Olsa, LKML, Linux Trace Devel, Zamir SUN,
	Arnaldo Carvalho de Melo, zsun, Vitaly Chikunov,
	Tzvetomir Stoyanov, Yordan Karadzhov, Ben Hutchings,
	Sudip Mukherjee, John Kacur, Clark Williams, Al Stone,
	Mauro Carvalho Chehab


[ Removing the powertop mailing list because it's rejecting everything ]

On Mon, 12 Oct 2020 11:41:20 -0700
Tony Jones <tonyj@suse.de> wrote:

> On Mon, Oct 12, 2020 at 11:19:50AM -0400, Steven Rostedt wrote:
> 
> > Once it's shown that it works for all the package maintainers, I will tag
> > it which should create the tarballs automatically on the above link.  
> 
> Hi.
> 
> It builds fine for me after manually creating the tarball from git.  
> Once there is an official versioned tarball I'll push it into
> openSUSE.
> 
> I presume some perf Makefile changes will be forthcoming to use it,
> rather than continuing to force build it out of TRACE_EVENT_DIR
> 

Zamir found this issue with the Documentation man pages:

Note, I'm not sure the proper way to fix this. I think this is the last
issue I need to resolve before making the tag.

-- Steve


On Thu, 8 Oct 2020 17:50:19 +0800
Zamir SUN <sztsian@gmail.com> wrote:

> Hi,
> 
> When I try to compiling the document of libtraceevent with the fix I 
> mentioned in [1] applied, make doc-install fails with errors like
> 
> "/usr/bin/install: cannot stat 'libtraceevent-record_parse.3': No such 
> file or directory"
> 
> Checking the compiled documents I see a lot of tep_*.3 generated, and 
> some libtraceevent_*.html. However no libtraceevent_*.3.
> 
> $ ls Documentation/*3 | head
> Documentation/libtraceevent.3
> Documentation/tep_alloc.3
> Documentation/tep_clear_flag.3
> Documentation/tep_cmdline_pid.3
> Documentation/tep_data_comm_from_pid.3
> Documentation/tep_data_flags.3
> Documentation/tep_data_pid.3
> Documentation/tep_data_pid_from_comm.3
> Documentation/tep_data_preempt_count.3
> Documentation/tep_data_type.3
> ls Documentation/*html | head
> Documentation/libtraceevent-commands.html
> Documentation/libtraceevent-cpus.html
> Documentation/libtraceevent-endian_read.html
> Documentation/libtraceevent-event_find.html
> Documentation/libtraceevent-event_get.html
> Documentation/libtraceevent-event_list.html
> Documentation/libtraceevent-event_print.html
> Documentation/libtraceevent-field_find.html
> Documentation/libtraceevent-field_get_val.html
> Documentation/libtraceevent-field_print.html
> 
> I also tried to port the Makefile from trace-cmd/Documentation to the 
> document dir, and it still only generates tep_*.3 files, so I feel this 
> is not the issue with the patch from [1].
> 
> As for my local environment, I have xmlto and asciidoc installed[2], but 
> not asciidoctor. I expect asciidoc could generate the documentations 
> like what it did in trace-cmd before.
> 
> Any idea if this is issue with my environment or it's something that 
> need to be implemented in the Makefile?
>  

Hi Zamir,

Thanks a lot for looking into this. I took your advice and reverted my
blind copy of the Makefiles, scripts and include headers, and instead
copied over their full history from the Linux kernel Tools directory.

You can see that update no (which now includes the utilities.mak as well).

Can you see if this patch fixes your current issue?

-- Steve

diff --git a/Documentation/Makefile b/Documentation/Makefile
index edb8623..3a981be 100644
--- a/Documentation/Makefile
+++ b/Documentation/Makefile
@@ -157,7 +157,7 @@ endif
 do-install-man: man
 	$(call QUIET_INSTALL, Documentation-man) \
 		$(INSTALL) -d -m 755 $(DESTDIR)$(man3dir); \
-		$(INSTALL) -m 644 $(DOC_MAN3) $(DESTDIR)$(man3dir);
+		$(INSTALL) -m 644 $(OUTPUT)*.3 $(DESTDIR)$(man3dir);
 
 install-man: check-man-tools man do-install-man
 

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-12 19:17       ` Steven Rostedt
@ 2020-10-12 20:24         ` Sudip Mukherjee
  2020-10-13  3:06         ` Zamir SUN
  1 sibling, 0 replies; 17+ messages in thread
From: Sudip Mukherjee @ 2020-10-12 20:24 UTC (permalink / raw)
  To: Steven Rostedt
  Cc: Tony Jones, Jiri Olsa, LKML, Linux Trace Devel, Zamir SUN,
	Arnaldo Carvalho de Melo, zsun, Vitaly Chikunov,
	Tzvetomir Stoyanov, Yordan Karadzhov, Ben Hutchings, John Kacur,
	Clark Williams, Al Stone, Mauro Carvalho Chehab

Hi Steve,

On Mon, Oct 12, 2020 at 8:17 PM Steven Rostedt <rostedt@goodmis.org> wrote:
>
>
> [ Removing the powertop mailing list because it's rejecting everything ]
>
> On Mon, 12 Oct 2020 11:41:20 -0700
> Tony Jones <tonyj@suse.de> wrote:
>
> > On Mon, Oct 12, 2020 at 11:19:50AM -0400, Steven Rostedt wrote:
> >
> > > Once it's shown that it works for all the package maintainers, I will tag
> > > it which should create the tarballs automatically on the above link.
> >

Works for me.

> Can you see if this patch fixes your current issue?
>
> -- Steve
>
> diff --git a/Documentation/Makefile b/Documentation/Makefile
> index edb8623..3a981be 100644
> --- a/Documentation/Makefile
> +++ b/Documentation/Makefile
> @@ -157,7 +157,7 @@ endif
>  do-install-man: man
>         $(call QUIET_INSTALL, Documentation-man) \
>                 $(INSTALL) -d -m 755 $(DESTDIR)$(man3dir); \
> -               $(INSTALL) -m 644 $(DOC_MAN3) $(DESTDIR)$(man3dir);
> +               $(INSTALL) -m 644 $(OUTPUT)*.3 $(DESTDIR)$(man3dir);
>
>  install-man: check-man-tools man do-install-man
>

I faced the same problem and this patch worked for me.


-- 
Regards
Sudip

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-12 19:17       ` Steven Rostedt
  2020-10-12 20:24         ` Sudip Mukherjee
@ 2020-10-13  3:06         ` Zamir SUN
  2020-10-13 13:02           ` Steven Rostedt
  1 sibling, 1 reply; 17+ messages in thread
From: Zamir SUN @ 2020-10-13  3:06 UTC (permalink / raw)
  To: Steven Rostedt
  Cc: Tony Jones, Jiri Olsa, LKML, Linux Trace Devel,
	Arnaldo Carvalho de Melo, Ziqian SUN (Zamir),
	Vitaly Chikunov, Tzvetomir Stoyanov, Yordan Karadzhov,
	Ben Hutchings, Sudip Mukherjee, John Kacur, Clark Williams,
	Al Stone, Mauro Carvalho Chehab

On Tue, Oct 13, 2020 at 3:17 AM Steven Rostedt <rostedt@goodmis.org> wrote:
>
>
> [ Removing the powertop mailing list because it's rejecting everything ]
>
> On Mon, 12 Oct 2020 11:41:20 -0700
> Tony Jones <tonyj@suse.de> wrote:
>
> > On Mon, Oct 12, 2020 at 11:19:50AM -0400, Steven Rostedt wrote:
> >
> > > Once it's shown that it works for all the package maintainers, I will tag
> > > it which should create the tarballs automatically on the above link.
> >
> > Hi.
> >
> > It builds fine for me after manually creating the tarball from git.
> > Once there is an official versioned tarball I'll push it into
> > openSUSE.
> >
> > I presume some perf Makefile changes will be forthcoming to use it,
> > rather than continuing to force build it out of TRACE_EVENT_DIR
> >
>
> Zamir found this issue with the Documentation man pages:
>
> Note, I'm not sure the proper way to fix this. I think this is the last
> issue I need to resolve before making the tag.
>

If anyone interested in, the discussion of this document compiling is
in the thread "libtraceevent: make doc-install tries different file
names than generated".

With the patch Steve mentioned, I can package this in RPM already. And
it's pending review in Fedora now.
https://bugzilla.redhat.com/show_bug.cgi?id=1887470

So, for me, there is no more issue for Fedora packaging.

Thanks.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-13  3:06         ` Zamir SUN
@ 2020-10-13 13:02           ` Steven Rostedt
  2020-10-14 10:08             ` Sudip Mukherjee
  2020-10-14 12:56             ` Zamir SUN
  0 siblings, 2 replies; 17+ messages in thread
From: Steven Rostedt @ 2020-10-13 13:02 UTC (permalink / raw)
  To: Zamir SUN
  Cc: Tony Jones, Jiri Olsa, LKML, Linux Trace Devel,
	Arnaldo Carvalho de Melo, Ziqian SUN (Zamir),
	Vitaly Chikunov, Tzvetomir Stoyanov, Yordan Karadzhov,
	Ben Hutchings, Sudip Mukherjee, John Kacur, Clark Williams,
	Al Stone, Mauro Carvalho Chehab

On Tue, 13 Oct 2020 11:06:16 +0800
Zamir SUN <sztsian@gmail.com> wrote:

> On Tue, Oct 13, 2020 at 3:17 AM Steven Rostedt <rostedt@goodmis.org> wrote:
> >
> >
> > [ Removing the powertop mailing list because it's rejecting everything ]
> >
> > On Mon, 12 Oct 2020 11:41:20 -0700
> > Tony Jones <tonyj@suse.de> wrote:
> >  
> > > On Mon, Oct 12, 2020 at 11:19:50AM -0400, Steven Rostedt wrote:
> > >  
> > > > Once it's shown that it works for all the package maintainers, I will tag
> > > > it which should create the tarballs automatically on the above link.  
> > >
> > > Hi.
> > >
> > > It builds fine for me after manually creating the tarball from git.
> > > Once there is an official versioned tarball I'll push it into
> > > openSUSE.
> > >
> > > I presume some perf Makefile changes will be forthcoming to use it,
> > > rather than continuing to force build it out of TRACE_EVENT_DIR
> > >  
> >
> > Zamir found this issue with the Documentation man pages:
> >
> > Note, I'm not sure the proper way to fix this. I think this is the last
> > issue I need to resolve before making the tag.
> >  
> 
> If anyone interested in, the discussion of this document compiling is
> in the thread "libtraceevent: make doc-install tries different file
> names than generated".
> 
> With the patch Steve mentioned, I can package this in RPM already. And
> it's pending review in Fedora now.
> https://bugzilla.redhat.com/show_bug.cgi?id=1887470
> 
> So, for me, there is no more issue for Fedora packaging.
> 

So should I just add that one patch and tag it?

-- Steve

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-13 13:02           ` Steven Rostedt
@ 2020-10-14 10:08             ` Sudip Mukherjee
  2020-10-14 13:44               ` Steven Rostedt
  2020-10-14 12:56             ` Zamir SUN
  1 sibling, 1 reply; 17+ messages in thread
From: Sudip Mukherjee @ 2020-10-14 10:08 UTC (permalink / raw)
  To: Steven Rostedt
  Cc: Zamir SUN, Tony Jones, Jiri Olsa, LKML, Linux Trace Devel,
	Arnaldo Carvalho de Melo, Ziqian SUN (Zamir),
	Vitaly Chikunov, Tzvetomir Stoyanov, Yordan Karadzhov,
	Ben Hutchings, John Kacur, Clark Williams, Al Stone,
	Mauro Carvalho Chehab

Hi Steve,

On Tue, Oct 13, 2020 at 2:02 PM Steven Rostedt <rostedt@goodmis.org> wrote:
>
> On Tue, 13 Oct 2020 11:06:16 +0800
> Zamir SUN <sztsian@gmail.com> wrote:
>
> > On Tue, Oct 13, 2020 at 3:17 AM Steven Rostedt <rostedt@goodmis.org> wrote:
> > >
<snip>
> > So, for me, there is no more issue for Fedora packaging.
> >
>
> So should I just add that one patch and tag it?

Just a thought, if you see
https://repology.org/project/linux-tools/versions then you will notice
that libtracevent has been packaged by the distros with a version of
v5.x+, and I will have the same problem for Debian also. Do you think
it makes sense to start  with a version of v6.x when you tag it? If
that is not possible then we will have to use epoch like we did for
libbpf.


-- 
Regards
Sudip

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-13 13:02           ` Steven Rostedt
  2020-10-14 10:08             ` Sudip Mukherjee
@ 2020-10-14 12:56             ` Zamir SUN
  2020-10-14 17:25               ` Steven Rostedt
  1 sibling, 1 reply; 17+ messages in thread
From: Zamir SUN @ 2020-10-14 12:56 UTC (permalink / raw)
  To: Steven Rostedt
  Cc: Tony Jones, Jiri Olsa, LKML, Linux Trace Devel,
	Arnaldo Carvalho de Melo, Ziqian SUN (Zamir),
	Vitaly Chikunov, Tzvetomir Stoyanov, Yordan Karadzhov,
	Ben Hutchings, Sudip Mukherjee, John Kacur, Clark Williams,
	Al Stone, Mauro Carvalho Chehab



On 10/13/20 9:02 PM, Steven Rostedt wrote:
> On Tue, 13 Oct 2020 11:06:16 +0800
> Zamir SUN <sztsian@gmail.com> wrote:
> 
>> On Tue, Oct 13, 2020 at 3:17 AM Steven Rostedt <rostedt@goodmis.org> wrote:
>>>
>>>
>>> [ Removing the powertop mailing list because it's rejecting everything ]
>>>
>>> On Mon, 12 Oct 2020 11:41:20 -0700
>>> Tony Jones <tonyj@suse.de> wrote:
>>>   
>>>> On Mon, Oct 12, 2020 at 11:19:50AM -0400, Steven Rostedt wrote:
>>>>   
>>>>> Once it's shown that it works for all the package maintainers, I will tag
>>>>> it which should create the tarballs automatically on the above link.
>>>>
>>>> Hi.
>>>>
>>>> It builds fine for me after manually creating the tarball from git.
>>>> Once there is an official versioned tarball I'll push it into
>>>> openSUSE.
>>>>
>>>> I presume some perf Makefile changes will be forthcoming to use it,
>>>> rather than continuing to force build it out of TRACE_EVENT_DIR
>>>>   
>>>
>>> Zamir found this issue with the Documentation man pages:
>>>
>>> Note, I'm not sure the proper way to fix this. I think this is the last
>>> issue I need to resolve before making the tag.
>>>   
>>
>> If anyone interested in, the discussion of this document compiling is
>> in the thread "libtraceevent: make doc-install tries different file
>> names than generated".
>>
>> With the patch Steve mentioned, I can package this in RPM already. And
>> it's pending review in Fedora now.
>> https://bugzilla.redhat.com/show_bug.cgi?id=1887470
>>
>> So, for me, there is no more issue for Fedora packaging.
>>
> 
> So should I just add that one patch and tag it?
> 

That would be great, at least for Fedora packaging.

Thanks!

-- 
Zamir SUN
Fedora user
GPG : 1D86 6D4A 49CE 4BBD 72CF FCF5 D856 6E11 F2A0 525E

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-14 10:08             ` Sudip Mukherjee
@ 2020-10-14 13:44               ` Steven Rostedt
  0 siblings, 0 replies; 17+ messages in thread
From: Steven Rostedt @ 2020-10-14 13:44 UTC (permalink / raw)
  To: Sudip Mukherjee
  Cc: Zamir SUN, Tony Jones, Jiri Olsa, LKML, Linux Trace Devel,
	Arnaldo Carvalho de Melo, Ziqian SUN (Zamir),
	Vitaly Chikunov, Tzvetomir Stoyanov, Yordan Karadzhov,
	Ben Hutchings, John Kacur, Clark Williams, Al Stone,
	Mauro Carvalho Chehab

On Wed, 14 Oct 2020 11:08:48 +0100
Sudip Mukherjee <sudipm.mukherjee@gmail.com> wrote:

> Just a thought, if you see
> https://repology.org/project/linux-tools/versions then you will notice
> that libtracevent has been packaged by the distros with a version of
> v5.x+, and I will have the same problem for Debian also. Do you think
> it makes sense to start  with a version of v6.x when you tag it? If
> that is not possible then we will have to use epoch like we did for
> libbpf.

Grumble. This is another reason I wish this was not part of the kernel. It
should not have a versioning based on the kernel. Yeah, this may be an
issue, especially, since library versions have real meaning with respect
to compatibility, where the Linux kernel version numbers do not.

We may need to use the epoch on this, because 5.7 has no meaning compared
to 5.8 and 5.9. I didn't even realize this was being shipped yet.

Yeah, I want to make this 1.1.0 as I've been tracking changes internally
with this.

-- Steve

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [ANNOUNCE] libtraceevent.git
  2020-10-14 12:56             ` Zamir SUN
@ 2020-10-14 17:25               ` Steven Rostedt
  0 siblings, 0 replies; 17+ messages in thread
From: Steven Rostedt @ 2020-10-14 17:25 UTC (permalink / raw)
  To: Zamir SUN
  Cc: Tony Jones, Jiri Olsa, LKML, Linux Trace Devel,
	Arnaldo Carvalho de Melo, Ziqian SUN (Zamir),
	Vitaly Chikunov, Tzvetomir Stoyanov, Yordan Karadzhov,
	Ben Hutchings, Sudip Mukherjee, John Kacur, Clark Williams,
	Al Stone, Mauro Carvalho Chehab

On Wed, 14 Oct 2020 20:56:53 +0800
Zamir SUN <sztsian@gmail.com> wrote:

> > 
> > So should I just add that one patch and tag it?
> >   
> 
> That would be great, at least for Fedora packaging.

I'm going with version 1.1.0 and not following the kernel versioning, as
that would just add to the confusion.

Here's the tarball:

  https://git.kernel.org/pub/scm/libs/libtrace/libtraceevent.git/snapshot/libtraceevent-1.1.0.tar.gz

-- Steve


^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, back to index

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-07 17:07 [ANNOUNCE] libtraceevent.git Steven Rostedt
2020-10-12 10:12 ` Jiri Olsa
2020-10-12 10:19   ` Zamir SUN
2020-10-12 10:35     ` Jiri Olsa
2020-10-12 15:19   ` Steven Rostedt
2020-10-12 15:32     ` Sudip Mukherjee
2020-10-12 18:41     ` Tony Jones
2020-10-12 18:52       ` Jiri Olsa
2020-10-12 19:13         ` Steven Rostedt
2020-10-12 19:17       ` Steven Rostedt
2020-10-12 20:24         ` Sudip Mukherjee
2020-10-13  3:06         ` Zamir SUN
2020-10-13 13:02           ` Steven Rostedt
2020-10-14 10:08             ` Sudip Mukherjee
2020-10-14 13:44               ` Steven Rostedt
2020-10-14 12:56             ` Zamir SUN
2020-10-14 17:25               ` Steven Rostedt

Linux-Trace-Devel Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-trace-devel/0 linux-trace-devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-trace-devel linux-trace-devel/ https://lore.kernel.org/linux-trace-devel \
		linux-trace-devel@vger.kernel.org
	public-inbox-index linux-trace-devel

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-trace-devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git