* [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm
@ 2012-10-06 20:35 Valentine Barshak
2012-10-24 19:22 ` Valentine Barshak
` (2 more replies)
0 siblings, 3 replies; 11+ messages in thread
From: Valentine Barshak @ 2012-10-06 20:35 UTC (permalink / raw)
To: buildroot
Signed-off-by: Valentine Barshak <gvaxon@gmail.com>
---
package/gqview/gqview.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/gqview/gqview.mk b/package/gqview/gqview.mk
index 2f64cd0..7d09fda 100644
--- a/package/gqview/gqview.mk
+++ b/package/gqview/gqview.mk
@@ -7,6 +7,7 @@ GQVIEW_VERSION = 2.1.5
GQVIEW_SOURCE = gqview-$(GQVIEW_VERSION).tar.gz
GQVIEW_SITE = http://prdownloads.sourceforge.net/gqview
GQVIEW_DEPENDENCIES = host-pkg-config libgtk2
+GQVIEW_CONF_ENV = LIBS="-lm"
$(eval $(autotools-package))
--
1.7.11.4
^ permalink raw reply related [flat|nested] 11+ messages in thread* [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm 2012-10-06 20:35 [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm Valentine Barshak @ 2012-10-24 19:22 ` Valentine Barshak 2012-10-24 20:51 ` Will Newton 2012-10-26 7:38 ` Arnout Vandecappelle 2012-11-07 19:06 ` Peter Korsgaard 2 siblings, 1 reply; 11+ messages in thread From: Valentine Barshak @ 2012-10-24 19:22 UTC (permalink / raw) To: buildroot On 10/07/2012 12:35 AM, Valentine Barshak wrote: > Signed-off-by: Valentine Barshak <gvaxon@gmail.com> > --- > package/gqview/gqview.mk | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/package/gqview/gqview.mk b/package/gqview/gqview.mk > index 2f64cd0..7d09fda 100644 > --- a/package/gqview/gqview.mk > +++ b/package/gqview/gqview.mk > @@ -7,6 +7,7 @@ GQVIEW_VERSION = 2.1.5 > GQVIEW_SOURCE = gqview-$(GQVIEW_VERSION).tar.gz > GQVIEW_SITE = http://prdownloads.sourceforge.net/gqview > GQVIEW_DEPENDENCIES = host-pkg-config libgtk2 > +GQVIEW_CONF_ENV = LIBS="-lm" > > $(eval $(autotools-package)) > > Does this work for everyone else or is it just deprecated and nobody should use it? I can't build without -lm neither in BR nor using my native host tools. Thanks, Val. ^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm 2012-10-24 19:22 ` Valentine Barshak @ 2012-10-24 20:51 ` Will Newton 2012-10-24 21:35 ` Valentine Barshak 0 siblings, 1 reply; 11+ messages in thread From: Will Newton @ 2012-10-24 20:51 UTC (permalink / raw) To: buildroot On Wed, Oct 24, 2012 at 8:22 PM, Valentine Barshak <gvaxon@gmail.com> wrote: > On 10/07/2012 12:35 AM, Valentine Barshak wrote: >> >> Signed-off-by: Valentine Barshak <gvaxon@gmail.com> >> --- >> package/gqview/gqview.mk | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/package/gqview/gqview.mk b/package/gqview/gqview.mk >> index 2f64cd0..7d09fda 100644 >> --- a/package/gqview/gqview.mk >> +++ b/package/gqview/gqview.mk >> @@ -7,6 +7,7 @@ GQVIEW_VERSION = 2.1.5 >> GQVIEW_SOURCE = gqview-$(GQVIEW_VERSION).tar.gz >> GQVIEW_SITE = http://prdownloads.sourceforge.net/gqview >> GQVIEW_DEPENDENCIES = host-pkg-config libgtk2 >> +GQVIEW_CONF_ENV = LIBS="-lm" >> >> $(eval $(autotools-package)) >> >> > > Does this work for everyone else or is it just deprecated and nobody should > use it? > > I can't build without -lm neither in BR nor using my native host tools. Which version of binutils are you using? 2.22? ^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm 2012-10-24 20:51 ` Will Newton @ 2012-10-24 21:35 ` Valentine Barshak 2012-10-25 9:02 ` Will Newton 0 siblings, 1 reply; 11+ messages in thread From: Valentine Barshak @ 2012-10-24 21:35 UTC (permalink / raw) To: buildroot On 10/25/2012 12:51 AM, Will Newton wrote: > On Wed, Oct 24, 2012 at 8:22 PM, Valentine Barshak <gvaxon@gmail.com> wrote: >> On 10/07/2012 12:35 AM, Valentine Barshak wrote: >>> >>> Signed-off-by: Valentine Barshak <gvaxon@gmail.com> >>> --- >>> package/gqview/gqview.mk | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/package/gqview/gqview.mk b/package/gqview/gqview.mk >>> index 2f64cd0..7d09fda 100644 >>> --- a/package/gqview/gqview.mk >>> +++ b/package/gqview/gqview.mk >>> @@ -7,6 +7,7 @@ GQVIEW_VERSION = 2.1.5 >>> GQVIEW_SOURCE = gqview-$(GQVIEW_VERSION).tar.gz >>> GQVIEW_SITE = http://prdownloads.sourceforge.net/gqview >>> GQVIEW_DEPENDENCIES = host-pkg-config libgtk2 >>> +GQVIEW_CONF_ENV = LIBS="-lm" >>> >>> $(eval $(autotools-package)) >>> >>> >> >> Does this work for everyone else or is it just deprecated and nobody should >> use it? >> >> I can't build without -lm neither in BR nor using my native host tools. > > Which version of binutils are you using? 2.22? > Yes, 2.22. Thanks, Val. ^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm 2012-10-24 21:35 ` Valentine Barshak @ 2012-10-25 9:02 ` Will Newton 2012-10-25 17:25 ` Valentine Barshak 0 siblings, 1 reply; 11+ messages in thread From: Will Newton @ 2012-10-25 9:02 UTC (permalink / raw) To: buildroot On Wed, Oct 24, 2012 at 10:35 PM, Valentine Barshak <gvaxon@gmail.com> wrote: > On 10/25/2012 12:51 AM, Will Newton wrote: >> >> On Wed, Oct 24, 2012 at 8:22 PM, Valentine Barshak <gvaxon@gmail.com> >> wrote: >>> >>> On 10/07/2012 12:35 AM, Valentine Barshak wrote: >>>> >>>> >>>> Signed-off-by: Valentine Barshak <gvaxon@gmail.com> >>>> --- >>>> package/gqview/gqview.mk | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/package/gqview/gqview.mk b/package/gqview/gqview.mk >>>> index 2f64cd0..7d09fda 100644 >>>> --- a/package/gqview/gqview.mk >>>> +++ b/package/gqview/gqview.mk >>>> @@ -7,6 +7,7 @@ GQVIEW_VERSION = 2.1.5 >>>> GQVIEW_SOURCE = gqview-$(GQVIEW_VERSION).tar.gz >>>> GQVIEW_SITE = http://prdownloads.sourceforge.net/gqview >>>> GQVIEW_DEPENDENCIES = host-pkg-config libgtk2 >>>> +GQVIEW_CONF_ENV = LIBS="-lm" >>>> >>>> $(eval $(autotools-package)) >>>> >>>> >>> >>> Does this work for everyone else or is it just deprecated and nobody >>> should >>> use it? >>> >>> I can't build without -lm neither in BR nor using my native host tools. >> >> >> Which version of binutils are you using? 2.22? >> > > Yes, 2.22. I believe this is caused by an issue with linking indirectly with the newer binutils. gqview relies on a library that is linked against libm but does not explicitly link against libm itself. Before 2.22 binutils would copy the DT_NEEDED entries from the library into gqview but it does not do this any more. A number of packages are affected by this. I believe the correct fix is to modify the affected packages to link against the libraries they use explicitly. ^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm 2012-10-25 9:02 ` Will Newton @ 2012-10-25 17:25 ` Valentine Barshak 2012-10-25 18:23 ` Will Newton 0 siblings, 1 reply; 11+ messages in thread From: Valentine Barshak @ 2012-10-25 17:25 UTC (permalink / raw) To: buildroot On 10/25/2012 01:02 PM, Will Newton wrote: > On Wed, Oct 24, 2012 at 10:35 PM, Valentine Barshak <gvaxon@gmail.com> wrote: >> On 10/25/2012 12:51 AM, Will Newton wrote: >>> >>> On Wed, Oct 24, 2012 at 8:22 PM, Valentine Barshak <gvaxon@gmail.com> >>> wrote: >>>> >>>> On 10/07/2012 12:35 AM, Valentine Barshak wrote: >>>>> >>>>> >>>>> Signed-off-by: Valentine Barshak <gvaxon@gmail.com> >>>>> --- >>>>> package/gqview/gqview.mk | 1 + >>>>> 1 file changed, 1 insertion(+) >>>>> >>>>> diff --git a/package/gqview/gqview.mk b/package/gqview/gqview.mk >>>>> index 2f64cd0..7d09fda 100644 >>>>> --- a/package/gqview/gqview.mk >>>>> +++ b/package/gqview/gqview.mk >>>>> @@ -7,6 +7,7 @@ GQVIEW_VERSION = 2.1.5 >>>>> GQVIEW_SOURCE = gqview-$(GQVIEW_VERSION).tar.gz >>>>> GQVIEW_SITE = http://prdownloads.sourceforge.net/gqview >>>>> GQVIEW_DEPENDENCIES = host-pkg-config libgtk2 >>>>> +GQVIEW_CONF_ENV = LIBS="-lm" >>>>> >>>>> $(eval $(autotools-package)) >>>>> >>>>> >>>> >>>> Does this work for everyone else or is it just deprecated and nobody >>>> should >>>> use it? >>>> >>>> I can't build without -lm neither in BR nor using my native host tools. >>> >>> >>> Which version of binutils are you using? 2.22? >>> >> >> Yes, 2.22. > > I believe this is caused by an issue with linking indirectly with the > newer binutils. gqview relies on a library that is linked against libm > but does not explicitly link against libm itself. Before 2.22 binutils > would copy the DT_NEEDED entries from the library into gqview but it > does not do this any more. gqview calls pow() and sqrt() directly, though it doesn't link against libm. > > A number of packages are affected by this. I believe the correct fix > is to modify the affected packages to link against the libraries they > use explicitly. > Is the above patch OK? Or do you mean configure/makefile(.in) should be patched to add libm dependency? Thanks, Val. ^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm 2012-10-25 17:25 ` Valentine Barshak @ 2012-10-25 18:23 ` Will Newton 2012-10-25 19:02 ` Valentine Barshak 0 siblings, 1 reply; 11+ messages in thread From: Will Newton @ 2012-10-25 18:23 UTC (permalink / raw) To: buildroot On Thu, Oct 25, 2012 at 6:25 PM, Valentine Barshak <gvaxon@gmail.com> wrote: > On 10/25/2012 01:02 PM, Will Newton wrote: >> >> On Wed, Oct 24, 2012 at 10:35 PM, Valentine Barshak <gvaxon@gmail.com> >> wrote: >>> >>> On 10/25/2012 12:51 AM, Will Newton wrote: >>>> >>>> >>>> On Wed, Oct 24, 2012 at 8:22 PM, Valentine Barshak <gvaxon@gmail.com> >>>> wrote: >>>>> >>>>> >>>>> On 10/07/2012 12:35 AM, Valentine Barshak wrote: >>>>>> >>>>>> >>>>>> >>>>>> Signed-off-by: Valentine Barshak <gvaxon@gmail.com> >>>>>> --- >>>>>> package/gqview/gqview.mk | 1 + >>>>>> 1 file changed, 1 insertion(+) >>>>>> >>>>>> diff --git a/package/gqview/gqview.mk b/package/gqview/gqview.mk >>>>>> index 2f64cd0..7d09fda 100644 >>>>>> --- a/package/gqview/gqview.mk >>>>>> +++ b/package/gqview/gqview.mk >>>>>> @@ -7,6 +7,7 @@ GQVIEW_VERSION = 2.1.5 >>>>>> GQVIEW_SOURCE = gqview-$(GQVIEW_VERSION).tar.gz >>>>>> GQVIEW_SITE = http://prdownloads.sourceforge.net/gqview >>>>>> GQVIEW_DEPENDENCIES = host-pkg-config libgtk2 >>>>>> +GQVIEW_CONF_ENV = LIBS="-lm" >>>>>> >>>>>> $(eval $(autotools-package)) >>>>>> >>>>>> >>>>> >>>>> Does this work for everyone else or is it just deprecated and nobody >>>>> should >>>>> use it? >>>>> >>>>> I can't build without -lm neither in BR nor using my native host tools. >>>> >>>> >>>> >>>> Which version of binutils are you using? 2.22? >>>> >>> >>> Yes, 2.22. >> >> >> I believe this is caused by an issue with linking indirectly with the >> newer binutils. gqview relies on a library that is linked against libm >> but does not explicitly link against libm itself. Before 2.22 binutils >> would copy the DT_NEEDED entries from the library into gqview but it >> does not do this any more. > > > gqview calls pow() and sqrt() directly, though it doesn't link against libm. > > >> >> A number of packages are affected by this. I believe the correct fix >> is to modify the affected packages to link against the libraries they >> use explicitly. >> > > Is the above patch OK? I think so, although it could do with a comment to say why it is needed. Peter or Thomas may disagree though? > Or do you mean configure/makefile(.in) should be patched to add libm > dependency? That is the proper long term fix that should be pushed to gqview upstream. ^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm 2012-10-25 18:23 ` Will Newton @ 2012-10-25 19:02 ` Valentine Barshak 0 siblings, 0 replies; 11+ messages in thread From: Valentine Barshak @ 2012-10-25 19:02 UTC (permalink / raw) To: buildroot On 10/25/2012 10:23 PM, Will Newton wrote: > On Thu, Oct 25, 2012 at 6:25 PM, Valentine Barshak <gvaxon@gmail.com> wrote: >> On 10/25/2012 01:02 PM, Will Newton wrote: >>> >>> On Wed, Oct 24, 2012 at 10:35 PM, Valentine Barshak <gvaxon@gmail.com> >>> wrote: >>>> >>>> On 10/25/2012 12:51 AM, Will Newton wrote: >>>>> >>>>> >>>>> On Wed, Oct 24, 2012 at 8:22 PM, Valentine Barshak <gvaxon@gmail.com> >>>>> wrote: >>>>>> >>>>>> >>>>>> On 10/07/2012 12:35 AM, Valentine Barshak wrote: >>>>>>> >>>>>>> >>>>>>> >>>>>>> Signed-off-by: Valentine Barshak <gvaxon@gmail.com> >>>>>>> --- >>>>>>> package/gqview/gqview.mk | 1 + >>>>>>> 1 file changed, 1 insertion(+) >>>>>>> >>>>>>> diff --git a/package/gqview/gqview.mk b/package/gqview/gqview.mk >>>>>>> index 2f64cd0..7d09fda 100644 >>>>>>> --- a/package/gqview/gqview.mk >>>>>>> +++ b/package/gqview/gqview.mk >>>>>>> @@ -7,6 +7,7 @@ GQVIEW_VERSION = 2.1.5 >>>>>>> GQVIEW_SOURCE = gqview-$(GQVIEW_VERSION).tar.gz >>>>>>> GQVIEW_SITE = http://prdownloads.sourceforge.net/gqview >>>>>>> GQVIEW_DEPENDENCIES = host-pkg-config libgtk2 >>>>>>> +GQVIEW_CONF_ENV = LIBS="-lm" >>>>>>> >>>>>>> $(eval $(autotools-package)) >>>>>>> >>>>>>> >>>>>> >>>>>> Does this work for everyone else or is it just deprecated and nobody >>>>>> should >>>>>> use it? >>>>>> >>>>>> I can't build without -lm neither in BR nor using my native host tools. >>>>> >>>>> >>>>> >>>>> Which version of binutils are you using? 2.22? >>>>> >>>> >>>> Yes, 2.22. >>> >>> >>> I believe this is caused by an issue with linking indirectly with the >>> newer binutils. gqview relies on a library that is linked against libm >>> but does not explicitly link against libm itself. Before 2.22 binutils >>> would copy the DT_NEEDED entries from the library into gqview but it >>> does not do this any more. >> >> >> gqview calls pow() and sqrt() directly, though it doesn't link against libm. >> >> >>> >>> A number of packages are affected by this. I believe the correct fix >>> is to modify the affected packages to link against the libraries they >>> use explicitly. >>> >> >> Is the above patch OK? > > I think so, although it could do with a comment to say why it is > needed. Peter or Thomas may disagree though? I thought it was kind of obvious. > >> Or do you mean configure/makefile(.in) should be patched to add libm >> dependency? > > That is the proper long term fix that should be pushed to gqview upstream. > Gqview is no longer supported. There's a Geeqie project that has been forked from gqview and is currently active. I still use gqview since it has all the features I need and is smaller than geeqie. I also prefer gqview because it opens a window and shows a "broken image" icon for unsupported image types, while geeqie doesn't create any window (even an error message box), which is a bit confusing. Thanks, Val. ^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm 2012-10-06 20:35 [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm Valentine Barshak 2012-10-24 19:22 ` Valentine Barshak @ 2012-10-26 7:38 ` Arnout Vandecappelle 2012-10-30 11:25 ` Markos Chandras 2012-11-07 19:06 ` Peter Korsgaard 2 siblings, 1 reply; 11+ messages in thread From: Arnout Vandecappelle @ 2012-10-26 7:38 UTC (permalink / raw) To: buildroot On 06/10/12 22:35, Valentine Barshak wrote: > Signed-off-by: Valentine Barshak<gvaxon@gmail.com> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Tested-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Problem can be reproduced on e.g. x86_64 with external toolchain. And since upstream is dead, the simple addition of -lm to LIBS is an adequate solution. Regards, Arnout > --- > package/gqview/gqview.mk | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/package/gqview/gqview.mk b/package/gqview/gqview.mk > index 2f64cd0..7d09fda 100644 > --- a/package/gqview/gqview.mk > +++ b/package/gqview/gqview.mk > @@ -7,6 +7,7 @@ GQVIEW_VERSION = 2.1.5 > GQVIEW_SOURCE = gqview-$(GQVIEW_VERSION).tar.gz > GQVIEW_SITE = http://prdownloads.sourceforge.net/gqview > GQVIEW_DEPENDENCIES = host-pkg-config libgtk2 > +GQVIEW_CONF_ENV = LIBS="-lm" > > $(eval $(autotools-package)) > -- Arnout Vandecappelle arnout at mind be Senior Embedded Software Architect +32-16-286540 Essensium/Mind http://www.mind.be G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F ^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm 2012-10-26 7:38 ` Arnout Vandecappelle @ 2012-10-30 11:25 ` Markos Chandras 0 siblings, 0 replies; 11+ messages in thread From: Markos Chandras @ 2012-10-30 11:25 UTC (permalink / raw) To: buildroot On Fri, Oct 26, 2012 at 8:38 AM, Arnout Vandecappelle <arnout@mind.be> wrote: > On 06/10/12 22:35, Valentine Barshak wrote: >> >> Signed-off-by: Valentine Barshak<gvaxon@gmail.com> > > > Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> > Tested-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> > > Problem can be reproduced on e.g. x86_64 with external toolchain. > > And since upstream is dead, the simple addition of -lm to LIBS is an > adequate solution. > > Regards, > Arnout > > Works for me as well. Tested-by: Markos Chandras <markos.chandras@imgtec.com> -- Regards, Markos ^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm 2012-10-06 20:35 [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm Valentine Barshak 2012-10-24 19:22 ` Valentine Barshak 2012-10-26 7:38 ` Arnout Vandecappelle @ 2012-11-07 19:06 ` Peter Korsgaard 2 siblings, 0 replies; 11+ messages in thread From: Peter Korsgaard @ 2012-11-07 19:06 UTC (permalink / raw) To: buildroot >>>>> "Valentine" == Valentine Barshak <gvaxon@gmail.com> writes: Valentine> Signed-off-by: Valentine Barshak <gvaxon@gmail.com> Committed, thanks. -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2012-11-07 19:06 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-10-06 20:35 [Buildroot] [PATCH] gqview: Fix build failure due to missing -lm Valentine Barshak 2012-10-24 19:22 ` Valentine Barshak 2012-10-24 20:51 ` Will Newton 2012-10-24 21:35 ` Valentine Barshak 2012-10-25 9:02 ` Will Newton 2012-10-25 17:25 ` Valentine Barshak 2012-10-25 18:23 ` Will Newton 2012-10-25 19:02 ` Valentine Barshak 2012-10-26 7:38 ` Arnout Vandecappelle 2012-10-30 11:25 ` Markos Chandras 2012-11-07 19:06 ` Peter Korsgaard
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.