462361 - Remove support for building with Cygwin

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Benjamin Smedberg]

We should drop support for building with cygwin. This means a lot of complexity related to cygwin-wrapper and perl and line endings can disappear.

Comment 1

[Mitchell Field [:Mitch]]

Attachment: Patch v1

Comment 2

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Comment on attachment 506191 [details] [diff] [review]
Patch v1

> case $target_os in
>-    os2*|msvc*|mks*|cygwin*|mingw*|darwin*|wince*|winmo*|beos*)
>+    beos*|darwin*|mingw*|os2*|wince*|winmo*)

Remove the beos from this line since you're here anyways.

I didn't look at the js/src bits, assuming they're the same.

Comment 3

[Mitchell Field [:Mitch]]

(In reply to comment #2)
> Comment on attachment 506191 [details] [diff] [review]
> Patch v1
> 
> > case $target_os in
> >-    os2*|msvc*|mks*|cygwin*|mingw*|darwin*|wince*|winmo*|beos*)
> >+    beos*|darwin*|mingw*|os2*|wince*|winmo*)
> 
> Remove the beos from this line since you're here anyways.
> 
> I didn't look at the js/src bits, assuming they're the same.

There are a lot of BeOS-related things lying around.
https://mxr.mozilla.org/mozilla-central/search?string=beos

I thought it'd make more sense to keep within this bug's scope and target BeOS in another bug report.

Comment 4

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

I agree in general, but you're touching that exact line anyways :-/

Comment 5

[Serge Gautherie (:sgautherie)]

Fwiw, either way, WinCE is in the same case too: bug 614720...

Comment 6

[Mitchell Field [:Mitch]]

http://hg.mozilla.org/projects/build-system/rev/261f1a85206a

Comment 7

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

http://hg.mozilla.org/mozilla-central/rev/261f1a85206a