* [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support @ 2021-04-12 19:55 Peter Seiderer 2021-04-12 19:55 ` [Buildroot] [PATCH v1 2/3] package/libtomcrypt: compile with -fPIC to enable linking to dynamic libraries/exectuables Peter Seiderer ` (2 more replies) 0 siblings, 3 replies; 17+ messages in thread From: Peter Seiderer @ 2021-04-12 19:55 UTC (permalink / raw) To: buildroot Signed-off-by: Peter Seiderer <ps.report@gmx.net> --- package/libtomcrypt/libtomcrypt.mk | 3 +++ package/libtomcrypt/libtomcrypt.pc.in | 10 ++++++++++ 2 files changed, 13 insertions(+) create mode 100644 package/libtomcrypt/libtomcrypt.pc.in diff --git a/package/libtomcrypt/libtomcrypt.mk b/package/libtomcrypt/libtomcrypt.mk index fccd4b051c..fa6dde64cd 100644 --- a/package/libtomcrypt/libtomcrypt.mk +++ b/package/libtomcrypt/libtomcrypt.mk @@ -27,6 +27,9 @@ define LIBTOMCRYPT_INSTALL_STAGING_CMDS $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) DESTDIR="$(STAGING_DIR)" \ PREFIX=/usr NODOCS=1 INSTALL_USER=$(shell id -u) \ INSTALL_GROUP=$(shell id -g) install + $(INSTALL) -D -m 0644 package/libtomcrypt/libtomcrypt.pc.in \ + $(STAGING_DIR)/usr/lib/pkgconfig/libtomcrypt.pc + $(SED) "s/@PACKAGE_VERSION@/${LIBTOMCRYPT_VERSION}/" $(STAGING_DIR)/usr/lib/pkgconfig/libtomcrypt.pc endef $(eval $(generic-package)) diff --git a/package/libtomcrypt/libtomcrypt.pc.in b/package/libtomcrypt/libtomcrypt.pc.in new file mode 100644 index 0000000000..14a010e216 --- /dev/null +++ b/package/libtomcrypt/libtomcrypt.pc.in @@ -0,0 +1,10 @@ +prefix=/usr +exec_prefix=${prefix} +libdir=${exec_prefix}/lib +includedir=${prefix}/include + +Name: LibTomCrypt +Description: public domain open source cryptographic toolkit +Version: @PACKAGE_VERSION@ +Libs: -L${libdir} -ltomcrypt +Cflags: -I${includedir} -- 2.31.1 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 2/3] package/libtomcrypt: compile with -fPIC to enable linking to dynamic libraries/exectuables 2021-04-12 19:55 [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support Peter Seiderer @ 2021-04-12 19:55 ` Peter Seiderer 2021-04-12 20:15 ` Thomas Petazzoni 2021-05-01 13:19 ` Arnout Vandecappelle 2021-04-12 19:55 ` [Buildroot] [PATCH v1 3/3] package/libtomcrypt: update home page URL to https Peter Seiderer 2021-04-12 20:14 ` [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support Thomas Petazzoni 2 siblings, 2 replies; 17+ messages in thread From: Peter Seiderer @ 2021-04-12 19:55 UTC (permalink / raw) To: buildroot Fixes: .../x86_64-buildroot-linux-gnu/bin/ld: .../host/x86_64-buildroot-linux-gnu/sysroot/usr/lib/libtomcrypt.a(md5.o): relocation R_X86_64_32 against `.rodata.str1.1' can not be used when making a shared object; recompile with -fPIC Signed-off-by: Peter Seiderer <ps.report@gmx.net> --- package/libtomcrypt/libtomcrypt.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package/libtomcrypt/libtomcrypt.mk b/package/libtomcrypt/libtomcrypt.mk index fa6dde64cd..b6ad77daec 100644 --- a/package/libtomcrypt/libtomcrypt.mk +++ b/package/libtomcrypt/libtomcrypt.mk @@ -17,7 +17,7 @@ LIBTOMCRYPT_DEPENDENCIES = libtommath # 0001-fix-CVE-2019-17362.patch LIBTOMCRYPT_IGNORE_CVES += CVE-2019-17362 -LIBTOMCRYPT_CFLAGS = -I./src/headers $(TARGET_CFLAGS) -DLTC_SOURCE -DLTM_DESC +LIBTOMCRYPT_CFLAGS = -I./src/headers $(TARGET_CFLAGS) -fPIC -DLTC_SOURCE -DLTM_DESC define LIBTOMCRYPT_BUILD_CMDS $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) $(TARGET_CONFIGURE_OPTS) CFLAGS="$(LIBTOMCRYPT_CFLAGS)" -- 2.31.1 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 2/3] package/libtomcrypt: compile with -fPIC to enable linking to dynamic libraries/exectuables 2021-04-12 19:55 ` [Buildroot] [PATCH v1 2/3] package/libtomcrypt: compile with -fPIC to enable linking to dynamic libraries/exectuables Peter Seiderer @ 2021-04-12 20:15 ` Thomas Petazzoni 2021-04-12 20:40 ` Peter Seiderer 2021-05-01 13:19 ` Arnout Vandecappelle 1 sibling, 1 reply; 17+ messages in thread From: Thomas Petazzoni @ 2021-04-12 20:15 UTC (permalink / raw) To: buildroot On Mon, 12 Apr 2021 21:55:26 +0200 Peter Seiderer <ps.report@gmx.net> wrote: > Fixes: > > .../x86_64-buildroot-linux-gnu/bin/ld: .../host/x86_64-buildroot-linux-gnu/sysroot/usr/lib/libtomcrypt.a(md5.o): relocation R_X86_64_32 against `.rodata.str1.1' can not be used when making a shared object; recompile with -fPIC > > Signed-off-by: Peter Seiderer <ps.report@gmx.net> In which situations? Are there some autobuilder failures? If not, what configuration fails? Thomas -- Thomas Petazzoni, co-owner and CEO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 2/3] package/libtomcrypt: compile with -fPIC to enable linking to dynamic libraries/exectuables 2021-04-12 20:15 ` Thomas Petazzoni @ 2021-04-12 20:40 ` Peter Seiderer 0 siblings, 0 replies; 17+ messages in thread From: Peter Seiderer @ 2021-04-12 20:40 UTC (permalink / raw) To: buildroot Hello Thomas, On Mon, 12 Apr 2021 22:15:57 +0200, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote: > On Mon, 12 Apr 2021 21:55:26 +0200 > Peter Seiderer <ps.report@gmx.net> wrote: > > > Fixes: > > > > .../x86_64-buildroot-linux-gnu/bin/ld: .../host/x86_64-buildroot-linux-gnu/sysroot/usr/lib/libtomcrypt.a(md5.o): relocation R_X86_64_32 against `.rodata.str1.1' can not be used when making a shared object; recompile with -fPIC > > > > Signed-off-by: Peter Seiderer <ps.report@gmx.net> > > In which situations? Are there some autobuilder failures? If not, what > configuration fails? Failure with an custom project which links libtomcrpyt to/into a dynamic library (x86_64)...., sorry no autobuilder failure (there is only one libtomcrypt cosumer in buildroot until now, dopbear)... Regards, Peter > > Thomas ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 2/3] package/libtomcrypt: compile with -fPIC to enable linking to dynamic libraries/exectuables 2021-04-12 19:55 ` [Buildroot] [PATCH v1 2/3] package/libtomcrypt: compile with -fPIC to enable linking to dynamic libraries/exectuables Peter Seiderer 2021-04-12 20:15 ` Thomas Petazzoni @ 2021-05-01 13:19 ` Arnout Vandecappelle 2021-05-02 10:15 ` Peter Seiderer 1 sibling, 1 reply; 17+ messages in thread From: Arnout Vandecappelle @ 2021-05-01 13:19 UTC (permalink / raw) To: buildroot On 12/04/2021 21:55, Peter Seiderer wrote: > Fixes: > > .../x86_64-buildroot-linux-gnu/bin/ld: .../host/x86_64-buildroot-linux-gnu/sysroot/usr/lib/libtomcrypt.a(md5.o): relocation R_X86_64_32 against `.rodata.str1.1' can not be used when making a shared object; recompile with -fPIC > > Signed-off-by: Peter Seiderer <ps.report@gmx.net> > --- > package/libtomcrypt/libtomcrypt.mk | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/package/libtomcrypt/libtomcrypt.mk b/package/libtomcrypt/libtomcrypt.mk > index fa6dde64cd..b6ad77daec 100644 > --- a/package/libtomcrypt/libtomcrypt.mk > +++ b/package/libtomcrypt/libtomcrypt.mk > @@ -17,7 +17,7 @@ LIBTOMCRYPT_DEPENDENCIES = libtommath > # 0001-fix-CVE-2019-17362.patch > LIBTOMCRYPT_IGNORE_CVES += CVE-2019-17362 > > -LIBTOMCRYPT_CFLAGS = -I./src/headers $(TARGET_CFLAGS) -DLTC_SOURCE -DLTM_DESC > +LIBTOMCRYPT_CFLAGS = -I./src/headers $(TARGET_CFLAGS) -fPIC -DLTC_SOURCE -DLTM_DESC On some architectures, -fPIC is incompatible with static builds. So this should probably only be done in non-static builds. Marked as Changes Requested. Regards, Arnout > > define LIBTOMCRYPT_BUILD_CMDS > $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) $(TARGET_CONFIGURE_OPTS) CFLAGS="$(LIBTOMCRYPT_CFLAGS)" > ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 2/3] package/libtomcrypt: compile with -fPIC to enable linking to dynamic libraries/exectuables 2021-05-01 13:19 ` Arnout Vandecappelle @ 2021-05-02 10:15 ` Peter Seiderer 0 siblings, 0 replies; 17+ messages in thread From: Peter Seiderer @ 2021-05-02 10:15 UTC (permalink / raw) To: buildroot Hello Arnout, On Sat, 1 May 2021 15:19:37 +0200, Arnout Vandecappelle <arnout@mind.be> wrote: > On 12/04/2021 21:55, Peter Seiderer wrote: > > Fixes: > > > > .../x86_64-buildroot-linux-gnu/bin/ld: .../host/x86_64-buildroot-linux-gnu/sysroot/usr/lib/libtomcrypt.a(md5.o): relocation R_X86_64_32 against `.rodata.str1.1' can not be used when making a shared object; recompile with -fPIC > > > > Signed-off-by: Peter Seiderer <ps.report@gmx.net> > > --- > > package/libtomcrypt/libtomcrypt.mk | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/package/libtomcrypt/libtomcrypt.mk b/package/libtomcrypt/libtomcrypt.mk > > index fa6dde64cd..b6ad77daec 100644 > > --- a/package/libtomcrypt/libtomcrypt.mk > > +++ b/package/libtomcrypt/libtomcrypt.mk > > @@ -17,7 +17,7 @@ LIBTOMCRYPT_DEPENDENCIES = libtommath > > # 0001-fix-CVE-2019-17362.patch > > LIBTOMCRYPT_IGNORE_CVES += CVE-2019-17362 > > > > -LIBTOMCRYPT_CFLAGS = -I./src/headers $(TARGET_CFLAGS) -DLTC_SOURCE -DLTM_DESC > > +LIBTOMCRYPT_CFLAGS = -I./src/headers $(TARGET_CFLAGS) -fPIC -DLTC_SOURCE -DLTM_DESC > > On some architectures, -fPIC is incompatible with static builds. So this should > probably only be done in non-static builds. O.k., will fix it...., thanks for review! Regards, Peter > > Marked as Changes Requested. > > Regards, > Arnout > > > > > define LIBTOMCRYPT_BUILD_CMDS > > $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) $(TARGET_CONFIGURE_OPTS) CFLAGS="$(LIBTOMCRYPT_CFLAGS)" > > ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 3/3] package/libtomcrypt: update home page URL to https 2021-04-12 19:55 [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support Peter Seiderer 2021-04-12 19:55 ` [Buildroot] [PATCH v1 2/3] package/libtomcrypt: compile with -fPIC to enable linking to dynamic libraries/exectuables Peter Seiderer @ 2021-04-12 19:55 ` Peter Seiderer 2021-05-01 13:19 ` Arnout Vandecappelle 2021-05-07 7:33 ` Peter Korsgaard 2021-04-12 20:14 ` [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support Thomas Petazzoni 2 siblings, 2 replies; 17+ messages in thread From: Peter Seiderer @ 2021-04-12 19:55 UTC (permalink / raw) To: buildroot - update home page URL to https Signed-off-by: Peter Seiderer <ps.report@gmx.net> --- package/libtomcrypt/Config.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package/libtomcrypt/Config.in b/package/libtomcrypt/Config.in index a7c5f50015..2459cd21e8 100644 --- a/package/libtomcrypt/Config.in +++ b/package/libtomcrypt/Config.in @@ -8,4 +8,4 @@ config BR2_PACKAGE_LIBTOMCRYPT functions, chaining modes, pseudo-random number generators, public key cryptography and a plethora of other routines. - http://www.libtom.net + https://www.libtom.net -- 2.31.1 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 3/3] package/libtomcrypt: update home page URL to https 2021-04-12 19:55 ` [Buildroot] [PATCH v1 3/3] package/libtomcrypt: update home page URL to https Peter Seiderer @ 2021-05-01 13:19 ` Arnout Vandecappelle 2021-05-07 7:33 ` Peter Korsgaard 1 sibling, 0 replies; 17+ messages in thread From: Arnout Vandecappelle @ 2021-05-01 13:19 UTC (permalink / raw) To: buildroot On 12/04/2021 21:55, Peter Seiderer wrote: > - update home page URL to https > > Signed-off-by: Peter Seiderer <ps.report@gmx.net> Applied to master, thanks. Regards, Arnout > --- > package/libtomcrypt/Config.in | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/package/libtomcrypt/Config.in b/package/libtomcrypt/Config.in > index a7c5f50015..2459cd21e8 100644 > --- a/package/libtomcrypt/Config.in > +++ b/package/libtomcrypt/Config.in > @@ -8,4 +8,4 @@ config BR2_PACKAGE_LIBTOMCRYPT > functions, chaining modes, pseudo-random number generators, > public key cryptography and a plethora of other routines. > > - http://www.libtom.net > + https://www.libtom.net > ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 3/3] package/libtomcrypt: update home page URL to https 2021-04-12 19:55 ` [Buildroot] [PATCH v1 3/3] package/libtomcrypt: update home page URL to https Peter Seiderer 2021-05-01 13:19 ` Arnout Vandecappelle @ 2021-05-07 7:33 ` Peter Korsgaard 1 sibling, 0 replies; 17+ messages in thread From: Peter Korsgaard @ 2021-05-07 7:33 UTC (permalink / raw) To: buildroot >>>>> "Peter" == Peter Seiderer <ps.report@gmx.net> writes: > - update home page URL to https > Signed-off-by: Peter Seiderer <ps.report@gmx.net> Committed to 2021.02.x, thanks. -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support 2021-04-12 19:55 [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support Peter Seiderer 2021-04-12 19:55 ` [Buildroot] [PATCH v1 2/3] package/libtomcrypt: compile with -fPIC to enable linking to dynamic libraries/exectuables Peter Seiderer 2021-04-12 19:55 ` [Buildroot] [PATCH v1 3/3] package/libtomcrypt: update home page URL to https Peter Seiderer @ 2021-04-12 20:14 ` Thomas Petazzoni 2021-04-12 21:15 ` Peter Seiderer 2 siblings, 1 reply; 17+ messages in thread From: Thomas Petazzoni @ 2021-04-12 20:14 UTC (permalink / raw) To: buildroot On Mon, 12 Apr 2021 21:55:25 +0200 Peter Seiderer <ps.report@gmx.net> wrote: > Signed-off-by: Peter Seiderer <ps.report@gmx.net> Is there any specific motivation? Does this had a chance of going upstream ? Thanks! Thomas -- Thomas Petazzoni, co-owner and CEO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support 2021-04-12 20:14 ` [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support Thomas Petazzoni @ 2021-04-12 21:15 ` Peter Seiderer 2021-05-01 13:18 ` Arnout Vandecappelle 0 siblings, 1 reply; 17+ messages in thread From: Peter Seiderer @ 2021-04-12 21:15 UTC (permalink / raw) To: buildroot Hello Thomas, On Mon, 12 Apr 2021 22:14:53 +0200, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote: > On Mon, 12 Apr 2021 21:55:25 +0200 > Peter Seiderer <ps.report@gmx.net> wrote: > > > Signed-off-by: Peter Seiderer <ps.report@gmx.net> > > Is there any specific motivation? Does this had a chance of going > upstream ? For an custom project, previously developed on with cmake on ubuntu (which provides the pkgconfig file).... Can try to suggest upstream... Regards, Peter > > Thanks! > > Thomas ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support 2021-04-12 21:15 ` Peter Seiderer @ 2021-05-01 13:18 ` Arnout Vandecappelle 2021-05-02 10:27 ` Peter Seiderer 0 siblings, 1 reply; 17+ messages in thread From: Arnout Vandecappelle @ 2021-05-01 13:18 UTC (permalink / raw) To: buildroot On 12/04/2021 23:15, Peter Seiderer wrote: > Hello Thomas, > > On Mon, 12 Apr 2021 22:14:53 +0200, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote: > >> On Mon, 12 Apr 2021 21:55:25 +0200 >> Peter Seiderer <ps.report@gmx.net> wrote: >> >>> Signed-off-by: Peter Seiderer <ps.report@gmx.net> >> >> Is there any specific motivation? Does this had a chance of going >> upstream ? > > For an custom project, previously developed on with cmake on ubuntu (which > provides the pkgconfig file).... > > Can try to suggest upstream... Adding a .pc file in Buildroot usually doesn't make much sense, because no package that depends on it is going to use it. So yes, please organise this upstream. Therefore, I've marked this patch as Rejected. Regards, Arnout ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support 2021-05-01 13:18 ` Arnout Vandecappelle @ 2021-05-02 10:27 ` Peter Seiderer 2021-05-03 7:58 ` Arnout Vandecappelle 0 siblings, 1 reply; 17+ messages in thread From: Peter Seiderer @ 2021-05-02 10:27 UTC (permalink / raw) To: buildroot Hello Arnout, On Sat, 1 May 2021 15:18:42 +0200, Arnout Vandecappelle <arnout@mind.be> wrote: > On 12/04/2021 23:15, Peter Seiderer wrote: > > Hello Thomas, > > > > On Mon, 12 Apr 2021 22:14:53 +0200, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote: > > > >> On Mon, 12 Apr 2021 21:55:25 +0200 > >> Peter Seiderer <ps.report@gmx.net> wrote: > >> > >>> Signed-off-by: Peter Seiderer <ps.report@gmx.net> > >> > >> Is there any specific motivation? Does this had a chance of going > >> upstream ? > > > > For an custom project, previously developed on with cmake on ubuntu (which > > provides the pkgconfig file).... > > > > Can try to suggest upstream... > > Adding a .pc file in Buildroot usually doesn't make much sense, because no > package that depends on it is going to use it. So yes, please organise this > upstream. But a buildroot provided library should be usable for buildroot packages AND out-of-tree/custom packages? Nevertheless, did take a second look at the libtomcrypt sources, already providing a libtomcrypt.pc.in template (but only used/installed in case of 'makefile.shared'/'makefile.unix' and the buildroot compile uses the default 'makefile' one)... Regards, Peter > > Therefore, I've marked this patch as Rejected. > > Regards, > Arnout ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support 2021-05-02 10:27 ` Peter Seiderer @ 2021-05-03 7:58 ` Arnout Vandecappelle 2021-05-03 19:38 ` Peter Seiderer 0 siblings, 1 reply; 17+ messages in thread From: Arnout Vandecappelle @ 2021-05-03 7:58 UTC (permalink / raw) To: buildroot On 02/05/2021 12:27, Peter Seiderer wrote: > Hello Arnout, > > On Sat, 1 May 2021 15:18:42 +0200, Arnout Vandecappelle <arnout@mind.be> wrote: > >> On 12/04/2021 23:15, Peter Seiderer wrote: >>> Hello Thomas, >>> >>> On Mon, 12 Apr 2021 22:14:53 +0200, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote: >>> >>>> On Mon, 12 Apr 2021 21:55:25 +0200 >>>> Peter Seiderer <ps.report@gmx.net> wrote: >>>> >>>>> Signed-off-by: Peter Seiderer <ps.report@gmx.net> >>>> >>>> Is there any specific motivation? Does this had a chance of going >>>> upstream ? >>> >>> For an custom project, previously developed on with cmake on ubuntu (which >>> provides the pkgconfig file).... >>> >>> Can try to suggest upstream... >> >> Adding a .pc file in Buildroot usually doesn't make much sense, because no >> package that depends on it is going to use it. So yes, please organise this >> upstream. > > But a buildroot provided library should be usable for buildroot packages AND > out-of-tree/custom packages? What I mean is the following. A .pc file is useless unless a package that uses libtomcrypt actually uses pkg-config to configure it. If the .pc file is not provided by libtomcrypt itself, but only by buildroot, then no package that depends on libtomcrypt is going to use pkg-config, because they "think" there is no .pc file for libtomcrypt. So we would have to patch each and every user of libtomcrypt to use pkg-config instead of whatever discovery mechanism they have. And none of these patches are upstreamable, because there is no .pc file outside of buildroot. > Nevertheless, did take a second look at the libtomcrypt sources, already > providing a libtomcrypt.pc.in template (but only used/installed in case > of 'makefile.shared'/'makefile.unix' and the buildroot compile uses the > default 'makefile' one)... That of course changes the story completely. Do you know if there is a reason for us not to use makefile.unix, so we don't need to sed the .pc file ourselves? Regards, Arnout > > Regards, > Peter > >> >> Therefore, I've marked this patch as Rejected. >> >> Regards, >> Arnout > ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support 2021-05-03 7:58 ` Arnout Vandecappelle @ 2021-05-03 19:38 ` Peter Seiderer 2021-05-04 7:28 ` Arnout Vandecappelle 0 siblings, 1 reply; 17+ messages in thread From: Peter Seiderer @ 2021-05-03 19:38 UTC (permalink / raw) To: buildroot Hello Arnout, On Mon, 3 May 2021 09:58:26 +0200, Arnout Vandecappelle <arnout@mind.be> wrote: > On 02/05/2021 12:27, Peter Seiderer wrote: > > Hello Arnout, > > > > On Sat, 1 May 2021 15:18:42 +0200, Arnout Vandecappelle <arnout@mind.be> wrote: > > > >> On 12/04/2021 23:15, Peter Seiderer wrote: > >>> Hello Thomas, > >>> > >>> On Mon, 12 Apr 2021 22:14:53 +0200, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote: > >>> > >>>> On Mon, 12 Apr 2021 21:55:25 +0200 > >>>> Peter Seiderer <ps.report@gmx.net> wrote: > >>>> > >>>>> Signed-off-by: Peter Seiderer <ps.report@gmx.net> > >>>> > >>>> Is there any specific motivation? Does this had a chance of going > >>>> upstream ? > >>> > >>> For an custom project, previously developed on with cmake on ubuntu (which > >>> provides the pkgconfig file).... > >>> > >>> Can try to suggest upstream... > >> > >> Adding a .pc file in Buildroot usually doesn't make much sense, because no > >> package that depends on it is going to use it. So yes, please organise this > >> upstream. > > > > But a buildroot provided library should be usable for buildroot packages AND > > out-of-tree/custom packages? > > What I mean is the following. A .pc file is useless unless a package that uses > libtomcrypt actually uses pkg-config to configure it. If the .pc file is not > provided by libtomcrypt itself, but only by buildroot, then no package that > depends on libtomcrypt is going to use pkg-config, because they "think" there is > no .pc file for libtomcrypt. So we would have to patch each and every user of > libtomcrypt to use pkg-config instead of whatever discovery mechanism they have. > And none of these patches are upstreamable, because there is no .pc file outside > of buildroot. > > > > Nevertheless, did take a second look at the libtomcrypt sources, already > > providing a libtomcrypt.pc.in template (but only used/installed in case > > of 'makefile.shared'/'makefile.unix' and the buildroot compile uses the > > default 'makefile' one)... > > That of course changes the story completely. > > Do you know if there is a reason for us not to use makefile.unix, so we don't > need to sed the .pc file ourselves? No..., it is used since the introduction of libtomcrypt (2016), maybe because it is the default or because it uses less/no predefined compile vars which needs to be patched/overwritten... By the way, libtomcrypt depends on libtommath which used -fPIC unconditionally... Regards, Peter > > Regards, > Arnout > > > > > Regards, > > Peter > > > >> > >> Therefore, I've marked this patch as Rejected. > >> > >> Regards, > >> Arnout > > > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support 2021-05-03 19:38 ` Peter Seiderer @ 2021-05-04 7:28 ` Arnout Vandecappelle 2021-05-06 19:45 ` Peter Seiderer 0 siblings, 1 reply; 17+ messages in thread From: Arnout Vandecappelle @ 2021-05-04 7:28 UTC (permalink / raw) To: buildroot On 03/05/2021 21:38, Peter Seiderer wrote: > By the way, libtomcrypt depends on libtommath which used -fPIC unconditionally... So, apparently I'm just full of s**t when I say -fPIC sometimes fails in static builds... Weird, I really *do* remember that... Ah, here I have it: [1] It doesn't lead to build failures, it's just less performant and useless in static builds. It turns out, however, that we unconditionally pass -fPIC in dozens of packages. So one more doesn't really hurt I guess. Regards, Arnout [1] https://patchwork.ozlabs.org/project/buildroot/patch/20210209104726.20132-1-kremneva at synopsys.com/ ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support 2021-05-04 7:28 ` Arnout Vandecappelle @ 2021-05-06 19:45 ` Peter Seiderer 0 siblings, 0 replies; 17+ messages in thread From: Peter Seiderer @ 2021-05-06 19:45 UTC (permalink / raw) To: buildroot Hello Arnout, On Tue, 4 May 2021 09:28:35 +0200, Arnout Vandecappelle <arnout@mind.be> wrote: > On 03/05/2021 21:38, Peter Seiderer wrote: > > By the way, libtomcrypt depends on libtommath which used -fPIC unconditionally... > > So, apparently I'm just full of s**t when I say -fPIC sometimes fails in static > builds... Weird, I really *do* remember that... > > Ah, here I have it: [1] It doesn't lead to build failures, it's just less > performant and useless in static builds. It turns out, however, that we > unconditionally pass -fPIC in dozens of packages. So one more doesn't really > hurt I guess. Thanks for insight, can send (at least) a patch for libtommath to change the -fPIC usage only for the non static case... Regards, Peter > > Regards, > Arnout > > > [1] > https://patchwork.ozlabs.org/project/buildroot/patch/20210209104726.20132-1-kremneva at synopsys.com/ > > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2021-05-07 7:33 UTC | newest] Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-04-12 19:55 [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support Peter Seiderer 2021-04-12 19:55 ` [Buildroot] [PATCH v1 2/3] package/libtomcrypt: compile with -fPIC to enable linking to dynamic libraries/exectuables Peter Seiderer 2021-04-12 20:15 ` Thomas Petazzoni 2021-04-12 20:40 ` Peter Seiderer 2021-05-01 13:19 ` Arnout Vandecappelle 2021-05-02 10:15 ` Peter Seiderer 2021-04-12 19:55 ` [Buildroot] [PATCH v1 3/3] package/libtomcrypt: update home page URL to https Peter Seiderer 2021-05-01 13:19 ` Arnout Vandecappelle 2021-05-07 7:33 ` Peter Korsgaard 2021-04-12 20:14 ` [Buildroot] [PATCH v1 1/3] package/libtomcrypt: add pkgconfig support Thomas Petazzoni 2021-04-12 21:15 ` Peter Seiderer 2021-05-01 13:18 ` Arnout Vandecappelle 2021-05-02 10:27 ` Peter Seiderer 2021-05-03 7:58 ` Arnout Vandecappelle 2021-05-03 19:38 ` Peter Seiderer 2021-05-04 7:28 ` Arnout Vandecappelle 2021-05-06 19:45 ` Peter Seiderer
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.