| Message ID | 164267787600.9929.14996075092071019143.stgit@work (mailing list archive) |
|---|---|
| Headers | show |
| Series | (Trivial) README fix and libtracecmd SONAME | expand |
On Thu, 20 Jan 2022 12:34:08 +0100 Dario Faggioli <dfaggioli@suse.com> wrote: > Hi! > > So, patch 1 of this small series just get rid of a stale mention to > libparsevent in the README file. > > About patch 2. Some distributions have the requirement that the package > name contains/is related to the SONAME (for packages that ship libraries, > of course). > > Currently, the SONAME of libtracecmd is something like libtracecmd.so.0.0.1 > or libtracecmd.so.1.0.0. This is not a problem, but I think libtracecmd.so.0 > (libtracecmd.so.1) is a bit more convenient. > > It is also how things are for libtracefs and libtraceevent. > > So, patch 2 does exactly that for libtracecmd, if that could be > interesting. :-) > > If, instead, it was intentional and there is an actual reason for the > SONAME of libtracecmd to be the entire file name of the .so file, then I > guess I've missed it. In which case, sorry for the noise. > > Applied. Thanks Dario. Although I had to tweak the subjects to have them start with a capital letter (like a sentence). And I also fixed a typo in the change log of the second patch. -- Steve
Hi! So, patch 1 of this small series just get rid of a stale mention to libparsevent in the README file. About patch 2. Some distributions have the requirement that the package name contains/is related to the SONAME (for packages that ship libraries, of course). Currently, the SONAME of libtracecmd is something like libtracecmd.so.0.0.1 or libtracecmd.so.1.0.0. This is not a problem, but I think libtracecmd.so.0 (libtracecmd.so.1) is a bit more convenient. It is also how things are for libtracefs and libtraceevent. So, patch 2 does exactly that for libtracecmd, if that could be interesting. :-) If, instead, it was intentional and there is an actual reason for the SONAME of libtracecmd to be the entire file name of the .so file, then I guess I've missed it. In which case, sorry for the noise. Regards --- Dario Faggioli (2): README: remove stale mention to libparsevent libtracecmd: use major version as SONAME lib/trace-cmd/Makefile | 2 +- scripts/utils.mk | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) -- Dario Faggioli, Ph.D http://about.me/dario.faggioli Virtualization Software Engineer SUSE Labs, SUSE https://www.suse.com/ ------------------------------------------------------------------- <<This happens because _I_ choose it to happen!>> (Raistlin Majere)