Bug 200929 – Add -I${includedir}/embed_base to firefox-xpcom.pc

Bug 200929 - Add -I${includedir}/embed_base to firefox-xpcom.pc

Summary:	Add -I${includedir}/embed_base to firefox-xpcom.pc

Status:	CLOSED WONTFIX

Aliases:	None

Product:	Fedora
Classification:	Fedora
Component:	firefox (Show other bugs)
Sub Component:
Version:	rawhide
Hardware:	All Linux

Priority	medium Severity medium
Target Milestone:	---
Target Release:	---
Assigned To:	Christopher Aillon
QA Contact:
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:	FC6Target 200850
	Show dependency tree / graph

Reported:	2006-08-01 13:10 EDT by Andrew Overholt
Modified:	2013-01-09 23:01 EST (History)
CC List:	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)
Add an attachment (proposed patch, testcase, etc.)

Groups: 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.