Bug 99331

Summary: Poppler 0.50: Link issues when cross compiling with NSS3 enabled
Product: poppler Reporter: Nadal Gonzalo García Zavala <snorkellingcactus>
Component: generalAssignee: poppler-bugs <poppler-bugs>
Status: RESOLVED MOVED QA Contact:
Severity: normal    
Priority: medium    
Version: unspecified   
Hardware: x86-64 (AMD64)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments: suceful native build
failed cross build
A try to fix the link issues.

Description Nadal Gonzalo García Zavala 2017-01-09 15:25:35 UTC
Created attachment 128832 [details]
suceful native build

Hi.

I'm trying to find why this happens. I can successfull compile when native, but get link issues when doing cross. In the meanwhile, i have made a patch that works fine. Note that i do not have a well cmake knowledge so it can be a little dirty.

Thanks.
Comment 1 Nadal Gonzalo García Zavala 2017-01-09 15:26:43 UTC
Created attachment 128833 [details]
failed cross build
Comment 2 Nadal Gonzalo García Zavala 2017-01-09 15:31:21 UTC
Created attachment 128834 [details] [review]
A try to fix the link issues.
Comment 3 Albert Astals Cid 2017-01-09 21:59:57 UTC
Sorry but this doesn't feel right, poppler is already linking to ${NSS3_LDFLAGS} and all those other tools don't use nss directly, only through poppler so linking nss3 to them directly doesn't seem like correct to me.
Comment 4 Nadal Gonzalo García Zavala 2017-01-10 15:37:05 UTC
Okay.

 I think i understand. 
 So, i should try to link libpoppler.so to nss3 instead?.
 I will try it and go back to here. In the meanwhile, ¿why cross build could be failing?

Thanks.
Comment 5 Albert Astals Cid 2017-01-10 16:24:24 UTC
>  So, i should try to link libpoppler.so to nss3 instead?.

We already do that, don't we?
Comment 6 Nadal Gonzalo García Zavala 2017-01-11 02:23:21 UTC
(In reply to Albert Astals Cid from comment #5)
> >  So, i should try to link libpoppler.so to nss3 instead?.
> 
> We already do that, don't we?

I'm not sure since ld show undefined references.
Comment 7 Albert Astals Cid 2017-01-11 21:55:32 UTC
Well reading the code says we do, but patches that fix it welcome :)
Comment 8 GitLab Migration User 2018-08-21 11:19:34 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/poppler/poppler/issues/622.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.