Re: [PATCH] extensions: remove trace.c and point to the new repository

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----Original Message-----
> 在 2021年03月26日 12:33, HAGIO KAZUHITO(萩尾 一仁) 写道:
> > The trace.c extension module moved to the separate repository:
> >   https://github.com/fujitsu/crash-trace
> >
> > Remove the trace.c file from the crash repository and instead add
> > trace.c.README file to point to the new repository for a while.
> >
> > Signed-off-by: Kazuhito Hagio <k-hagio-ab@xxxxxxx>
> > ---
> > Lianbo,
> > I can see that the spec file of RHEL's crash-trace-command points to
> > the tarball in the crash-extensions repository, we can remove trace.c
> > in the crash repository?
> >
> Thank you for the information, Kazu.
> 
> Yes, please go ahead. Hatayama has put the tarball in the github:
> https://github.com/fujitsu/crash-trace/archive/v3.0/crash-trace-command-3.0.tar.gz
> 
> We can easily update the URL and Source in the spec file.

Thanks for checking this.

> 
> >  extensions/trace.c        | 2592 -------------------------------------
> >  extensions/trace.c.README |    7 +
> 
> BTW: It could be good to add a description of the new repository changes in the
> top-level README file(at end) instead of a new one. What do you think?

hmm, I think that the top-level README file (and README data in help.c) is
not very suitable to put a description of an extension module, especially
such a temporary information. (I'm going to remove the file someday.)  And
this is for users looking for trace.c, when they find there is no trace.c
there, will they read the top-level README?  The extensions/trace.c.README
will be easier to notice.

Do you have any concerns about adding the file?

Thanks,
Kazu



--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/crash-utility




[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux