Remove support for building with Cygwin

RESOLVED FIXED

Status

defect
--
trivial
RESOLVED FIXED
11 years ago
Last year

People

(Reporter: benjamin, Assigned: Mitch)

Tracking

Trunk
x86
Windows XP

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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.
Assignee: benjamin → nobody
Severity: normal → trivial
Flags: in-testsuite-
Version: unspecified → Trunk
Depends on: 604266
Assignee: nobody → mitchell.field
Status: NEW → ASSIGNED
Summary: Remove support for building with cygwin → Remove support for building with Cygwin
Posted patch Patch v1Splinter Review
Attachment #506191 - Flags: review?(khuey)
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.
Attachment #506191 - Flags: review?(khuey) → review+
(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.
I agree in general, but you're touching that exact line anyways :-/
Fwiw, either way, WinCE is in the same case too: bug 614720...
http://hg.mozilla.org/mozilla-central/rev/261f1a85206a
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
No longer depends on: 604266
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.