Summary
The windows feature should not be set on Cygwin
Metadata
- Id: 2c61bac6368271143c3e2c5d91cf0cb5deff02cb
- Trac id: 1381
- Type: defect
- Reporter: johnwcowan
- Owner: felix
- Cc:
- Status: closed
- Component: core libraries
- Estimated difficulty: easy
- Resolution: wontfix
- Priority: major
- Milestone: 5.0
- Version: 4.12.0
- Changetime: 2018-06-08 20:57:18 UTC
- Created: 2017-06-27 17:04:22 UTC
- Keywords:
Description
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.
Changes and comments
[2017-08-25 15:11:10 UTC] sjamaan set difficulty to easy
[2017-08-25 15:11:10 UTC] sjamaan changed milestone from someday to 5.0
[2017-08-25 15:11:10 UTC] sjamaan wrote:
I think changing this for CHICKEN 4 is a bit too radical, but for CHICKEN 5 it's a good idea to change this.
[2017-11-23 10:55:56 UTC] felix changed status from new to accepted
[2017-11-23 10:55:56 UTC] felix set owner to felix
[2017-11-23 11:07:40 UTC] felix wrote:
`(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?
[2018-05-22 21:59:36 UTC] johnwcowan wrote:
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.
[2018-06-08 20:57:18 UTC] sjamaan wrote:
We've already changed `build-platform` to return `gnu` on Cygwin/gcc. `software-type` is the operating system, which is still Windows.