Bug 101635 – [CMake] Remove the definition of WTF_PLATFORM_WIN from the build system

• Component: Platform
• Status: RESOLVED FIXED

[Laszlo Gombos]

It seems to me that WTF_PLATFORM_WIN defined twice; in CMake build-system in cmake/OptionsWindows.cmake and in Source/WTF/wtf/Platform.h. I would find it more future-proof to have it only defined in common C++ code in Platform.h.

Related discussion in bug 101482.

[Soo-Hyun Choi]

Created attachment 182473 [details]
Patch

[Soo-Hyun Choi]

This super simple patch seemed to be faced with some kind of build error on Windows port. Any clue? I don't personally have windows build system.

[Chris Dumez]

(In reply to comment #2)
> This super simple patch seemed to be faced with some kind of build error on Windows port. Any clue? I don't personally have windows build system.

This is unrelated to the patch. The win-ews bot is in a bad state. Otherwise, the bubble would be red (not purple).

[Soo-Hyun Choi]

I guessed the win-ews is in a bad state, but wanted to doubly check. Okay, then I am trying to put it back to the review & commit queue.

[Laszlo Gombos]

Comment on attachment 182473 [details]
Patch

Looks good to me. I would like to hear from  Patrick Gansterer or someone else from the WinCE port that they support this change before this can land.

[Soo-Hyun Choi]

(In reply to comment #5)
> (From update of attachment 182473 [details])
> Looks good to me. I would like to hear from  Patrick Gansterer or someone else from the WinCE port that they support this change before this can land.

paroga doesn't seem to be in the IRC channel so often, nor do I seem to find anyone for WinCE port from http://trac.webkit.org/wiki/WebKit%20Team. 

Any advice?

[Gyuyoung Kim]

Comment on attachment 182473 [details]
Patch

You are not committer. Do not set cp+.

[Gyuyoung Kim]

If there is no comment from wince folks by tomorrow, I'd like to land this patch.

[Patrick R. Gansterer]

Comment on attachment 182473 [details]
Patch

(In reply to comment #6)
> (In reply to comment #5)
> > (From update of attachment 182473 [details] [details])
> > Looks good to me. I would like to hear from  Patrick Gansterer or someone else from the WinCE port that they support this change before this can land.
> 
> paroga doesn't seem to be in the IRC channel so often, nor do I seem to find anyone for WinCE port from http://trac.webkit.org/wiki/WebKit%20Team. 

I'm the "WinCE team". ;-)

Sorry for the long long delay, but it's not easy to find the time for WebKit if it's not your full time job. :-(

[Soo-Hyun Choi]

(In reply to comment #7)
> (From update of attachment 182473 [details])
> You are not committer. Do not set cp+.

Oops! My bad - I meant it to set cq?

[WebKit Review Bot]

Comment on attachment 182473 [details]
Patch

Clearing flags on attachment: 182473

Committed r140662: <http://trac.webkit.org/changeset/140662>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.