Bug 172590 - [Browser] can't compile embedded browser with firefox using N20070201-0010 (v3324)
Summary: [Browser] can't compile embedded browser with firefox using N20070201-0010 (v...
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 3.3   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 3.3 M5   Edit
Assignee: Grant Gayed CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2007-02-01 19:47 EST by Ben Konrath CLA
Modified: 2007-02-07 15:07 EST (History)
2 users (show)

See Also:


Attachments
patch against v3324 (2.19 KB, patch)
2007-02-01 19:52 EST, Ben Konrath CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ben Konrath CLA 2007-02-01 19:47:46 EST
SSIA. Patch to follow.
Comment 1 Ben Konrath CLA 2007-02-01 19:52:19 EST
Created attachment 58074 [details]
patch against v3324

In make_linux.mak, I couldn't find references to 'c.o' or 'c_stats.o' so I removed them. In build.sh I changed GECKO_{INCLUDES,LIBS} to MOZILLA_{INCLUDES,LIBS} so this file would work with the changes in make_linux.mak and I added a check for firefox-xpcom if mozilla-xpcom can't be found. It would be great to see this fix in 3.3M5. Thanks, Ben
Comment 2 Grant Gayed CLA 2007-02-02 17:45:15 EST
fixed > 0202, excluding the removal of c.o and c_stats.o.  Thanks for the patch!
Comment 3 Ben Konrath CLA 2007-02-03 16:10:08 EST
No problem. What's the deal with c.o and c_stats.o? Will I be able to compile SWT in the 3.3M5 source drop?
Comment 4 Grant Gayed CLA 2007-02-05 16:03:52 EST
Do the c.o and c_stats.o prevent you from compiling successfully?  These are needed because some of the platform-common functions from os have moved into c.  Is the source build failing to find c.o and c_stats.o when it is compiling the swt libraries?
Comment 5 Ben Konrath CLA 2007-02-06 13:37:19 EST
(In reply to comment #4)
> Do the c.o and c_stats.o prevent you from compiling successfully?  These are
> needed because some of the platform-common functions from os have moved into c.
>  Is the source build failing to find c.o and c_stats.o when it is compiling the
> swt libraries?

Yes, that is correct, c.c and c_stats.c are not found when compiling the swt libraries with the source drop. If you look at the 'compilelibs' target main build file:

http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.releng.eclipsebuilder/eclipse/buildConfigs/sdk/srcBuild/build.xml?view=markup

you will see that the swt libs are compiled from the src.zip in plugins/org.eclipse.rcp.source.${installOs}.${installWs}.${installArch}/src/org.eclipse.swt.${installWs}.${installOs}.${installArch}_*/.
I think that whatever generates this src.zip is not including the files from Eclipse SWT PI/common/library/ and should.

I'm re-opening this bug until this problem is resolved. Thanks.
Comment 6 Ben Konrath CLA 2007-02-07 14:23:26 EST
Will this be fixed in 3.3M5?
Comment 7 Grant Gayed CLA 2007-02-07 14:55:26 EST
Yes, this is fixed in HEAD (again) and will be included in today's 6:00pm build.  The c files now appear in the root of swt's source zip.
Comment 8 Ben Konrath CLA 2007-02-07 15:07:08 EST
Thanks Grant!