Bug 200929 - Add -I${includedir}/embed_base to firefox-xpcom.pc
Add -I${includedir}/embed_base to firefox-xpcom.pc
Status: CLOSED WONTFIX
Product: Fedora
Classification: Fedora
Component: firefox (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Christopher Aillon
:
Depends On:
Blocks: FC6Target 200850
  Show dependency treegraph
 
Reported: 2006-08-01 13:10 EDT by Andrew Overholt
Modified: 2013-01-09 23:01 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2006-09-11 13:38:57 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Andrew Overholt 2006-08-01 13:10:46 EDT
Description of problem:
SWT needs nsEmbedAPI.h to be in the compiler's -I flags for building its mozilla
bindings.  If I hack firefox-xpcom.pc to include it, the build proceeds as
expected.  As this file is in embed_base, can it be added to firefox-xpcom.pc?

Version-Release number of selected component (if applicable):
firefox-1.5.0.5-8
Comment 1 Jesse Keating 2006-09-10 11:14:02 EDT
Whats the status here?  This is blocking the eclipse bug, that we'd like to get
fixed for Test3...  
Comment 2 Ben Konrath 2006-09-10 11:25:45 EDT
(In reply to comment #1)
> Whats the status here?  This is blocking the eclipse bug, that we'd like to get
> fixed for Test3...  

No, we have a workaround in place for this. It would still be nice to get this
fixe, but it's not critical for Test3.

3.2.0-5 has a fix for bug # 201778 and it's building now.
Comment 4 Christopher Aillon 2006-09-11 13:38:57 EDT
nsEmbedAPI.h is not part of the Gecko SDK.  It shouldn't have been available
before.  Use nsXPCOM.h for the stuff you need.  See also bug 201778 comment 6.
Comment 5 Ben Konrath 2006-09-19 09:03:36 EDT
(In reply to comment #4)
> nsEmbedAPI.h is not part of the Gecko SDK.  It shouldn't have been available
> before.  Use nsXPCOM.h for the stuff you need.  See also bug 201778 comment 6.

I had planned to keep both the NS_InitXPCOM3 and the NS_InitEmbedding in SWT so
we still needed this. Looking into the situation again, I think it's best to
remove the NS_InitEmbedding entirely as you suggest. Our latest package,
eclipse-1:3.2.0-6.fc6, does not use nsEmbedAPI.h. Thanks.

Note You need to log in before you can comment on or make changes to this bug.