442009 - libxul-embedding.pc and libxul-embedding-unstable.pc files should require NSPR

Status: RESOLVED WONTFIX

(Toolkit Graveyard :: XULRunner, defect)

Description

[Martin Stránský [:stransky] (ni? me)]

Attachment: proposed patch to those files

libxul-embedding and libxul-embedding-unstable *.pc files should require NSPR. Embedded applications don't build w/o it.

Comment 1

[Benjamin Smedberg]

Comment on attachment 326889 [details] [diff] [review]
proposed patch to those files

Linkin NSPR is not a requirement for embedded applications... obviously the headers are required for building, but NSPR link flags should not be implied.

Comment 2

[Martin Stránský [:stransky] (ni? me)]

Honestly I know about only one application what builds as embedded and it's liferea in Fedora. It's linked with NSS/NSPR. 

If you think the embedded apps should not link with nspr (actually it makes sense) we can close this report and I'll fix the liferea package.

Comment 3

[Benjamin Smedberg]

It's not that they "shouldn't", it's that they "might not want to". I think it's perfectly fine for lifarea to link against NSPR/NSS especially when they are system libraries... just they should pull in NSPR/NSS explicitly instead of depending on libxul-embedding.pc to do it.

Make sense?

Comment 4

[Martin Stránský [:stransky] (ni? me)]

Yep, fair enough. closing as wontfix.