Opened 2 years ago

Closed 13 months ago

#1381 closed defect (wontfix)

The windows feature should not be set on Cygwin

Reported by: johnwcowan Owned by: felix
Priority: major Milestone: 5.0
Component: core libraries Version: 4.12.0
Keywords: Cc:
Estimated difficulty: easy


In general, the assumption that Cygwin is Windows is a bad idea. Cygwin doesn't like Windows paths, though it can tolerate them in some circumstances, and process execution is done via bash, not via cmd. On a Cygwin build, the windows: entry in (features) should not be present.

Change History (5)

comment:1 Changed 23 months ago by sjamaan

  • Estimated difficulty set to easy
  • Milestone changed from someday to 5.0

I think changing this for CHICKEN 4 is a bit too radical, but for CHICKEN 5 it's a good idea to change this.

comment:2 Changed 20 months ago by felix

  • Owner set to felix
  • Status changed from new to accepted

comment:3 Changed 20 months ago by felix

(software-type) returns #:windows on Cygwin systems (because that is what it is.) I'd rather change those places that exhibit wrong behaviour by assuming a windows feature implies a non-cygwin system.

Is there specific beahviour that is wrong, or is this just a philosophical issue?

comment:4 Changed 14 months ago by johnwcowan

Well, specifically build-platform's outputs don't make sense. 'cygwin', 'gnu', and 'clang' are all possible results, but you can build on Cygwin with either gcc or clang. By promoting Cygwin to an OS, we can get sensible results from build-platform.

comment:5 Changed 13 months ago by sjamaan

  • Resolution set to wontfix
  • Status changed from accepted to closed

We've already changed build-platform to return gnu on Cygwin/gcc. software-type is the operating system, which is still Windows.

Note: See TracTickets for help on using tickets.