Closed Bug 344504 Opened 18 years ago Closed 18 years ago

Mingw build error in nsDataObj.cpp

Categories

(Core :: DOM: Copy & Paste and Drag & Drop, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: martijn.martijn, Unassigned)

References

Details

I updated my tree, and now I get this build error: SYMBOLS=1 -DMOZ_USER_DIR=\"Mozilla\" -DHAVE_STDINT_H=1 -DHAVE_INTTYPES_H=1 -DHAV E_UINT64_T=1 -DMOZ_XUL=1 -DMOZ_PROFILELOCKING=1 -DMOZ_MORKREADER=1 -DMOZ_DLL_SUF FIX=\".dll\" -DJS_THREADSAFE=1 -DMOZ_REFLOW_PERF=1 -DMOZ_REFLOW_PERF_DSP=1 -DMOZ ILLA_LOCALE_VERSION=\"1.9a1\" -DMOZILLA_REGION_VERSION=\"1.9a1\" -DMOZILLA_SKIN_ VERSION=\"1.8\" -D_MOZILLA_CONFIG_H_ -DMOZILLA_CLIENT /cygdrive/c/mozilla/mozil la/widget/src/windows/nsDataObj.cpp In file included from c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:46: c:/mozilla/mozilla/widget/src/windows/nsDataObj.h:119: error: expected class-nam e before '{' token c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:265: warning: `CLSID_nsDataO bj' initialized and declared `extern' c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:265: warning: `__cdecl__' at tribute only applies to function types c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp: In member function `virtual HRESULT nsDataObj::QueryInterface(const IID&, void**)': c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:332: error: `IID_IAsyncOpera tion' was not declared in this scope c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:333: error: `IAsyncOperation ' has not been declared c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:333: error: expected `>' bef ore '*' token c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:333: error: expected `(' bef ore '*' token c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:333: error: expected primary -expression before '>' token c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:333: error: expected `)' bef ore ';' token c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:332: warning: unused variabl e 'IID_IAsyncOperation' c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp: In member function `virtual HRESULT nsDataObj::GetFileDescriptor_IStreamA(FORMATETC&, STGMEDIUM&)': c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:1607: error: `FD_PROGRESSUI' was not declared in this scope c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:1607: warning: unused variab le 'FD_PROGRESSUI' c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp: In member function `virtual HRESULT nsDataObj::GetFileDescriptor_IStreamW(FORMATETC&, STGMEDIUM&)': c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:1643: error: `FD_PROGRESSUI' was not declared in this scope c:/mozilla/mozilla/widget/src/windows/nsDataObj.cpp:1643: warning: unused variab le 'FD_PROGRESSUI' make[6]: *** [nsDataObj.o] Error 1 make[6]: Leaving directory `/cygdrive/c/mozilla/mozilla/widget/src/windows' make[5]: *** [libs] Error 2 make[5]: Leaving directory `/cygdrive/c/mozilla/mozilla/widget/src' make[4]: *** [libs] Error 2 make[4]: Leaving directory `/cygdrive/c/mozilla/mozilla/widget' make[3]: *** [libs_tier_9] Error 2 make[3]: Leaving directory `/cygdrive/c/mozilla/mozilla' make[2]: *** [tier_9] Error 2 make[2]: Leaving directory `/cygdrive/c/mozilla/mozilla' make[1]: *** [alldep] Error 2 make[1]: Leaving directory `/cygdrive/c/mozilla/mozilla' make: *** [alldep] Error 2 Regression from bug 267426, I guess.
Think this is a mingw bug, those are standard windows enums and interfaces - should be present. We could explicitly define them in nsDataObj.h or somewhere to fix this build error. Don't have mingw build system installed right now. How are bugs like this fixed?
Sometimes defines are added in Mozilla's codebase, see bug 328182. Sometimes mingw is being fixed and everyone is forced to upgrade, see bug 328499. But I think that's something that only can happen on trunk. And for bug 267426 the plan is it to land on branch, right?
(In reply to comment #2) > 328499. But I think that's something that only can happen on trunk. And for bug > 267426 the plan is it to land on branch, right? > Yes, there is such a plan. So it'd be better to define stuff inplace, am I right? By the way, where mingw build instructions are?
You can find build instructions here: http://gemal.dk/mozilla/build.html To be honest, I'm not sure if you're responsible for fixing this. Mingw is not an officially supported build platform.
Blocks: mingw
Please note, even if this bug is fixed we have also bug 341128 where we wait for its patch to be checked in.
did the last patch for bug 203307 fixed this error? if so mark it fixed.
(In reply to comment #6) > did the last patch for bug 203307 fixed this error? if so mark it fixed. Thanks! I believe you. If it isn't fixed, I'll notice it soon enough.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.