Skip to content

uname: allow setting the system name to CYGWIN#97

Merged
lazka merged 1 commit into
msys2:msys2-3_3_5-releasefrom
lazka:uname-cygwin
Jul 4, 2022
Merged

uname: allow setting the system name to CYGWIN#97
lazka merged 1 commit into
msys2:msys2-3_3_5-releasefrom
lazka:uname-cygwin

Conversation

@lazka

@lazka lazka commented Jul 3, 2022

Copy link
Copy Markdown
Member

We are currently trying to move our cygwin build environment closer
to cygwin and some autotools/bash based build systems call "uname -s"
to figure out the OS and in many cases only handle the cygwin case, so
we have to patch them.

With this instead of patching we can set MSYSTEM=CYGWIN and change
uname output that way.

The next step would be to always output CYGWIN in an msys env by default,
but for now this allows us to get rid of all the patches without
affecting users.

We are currently trying to move our cygwin build environment closer
to cygwin and some autotools/bash based build systems call "uname -s"
to figure out the OS and in many cases only handle the cygwin case, so
we have to patch them.

With this instead of patching we can set MSYSTEM=CYGWIN and change
uname output that way.

The next step would be to always output CYGWIN in an msys env by default,
but for now this allows us to get rid of all the patches without
affecting users.
@lazka

lazka commented Jul 3, 2022

Copy link
Copy Markdown
Member Author

There is also uname -o -> Msys, which I'm not sure about... I'll have another look there.

@lazka lazka marked this pull request as draft July 3, 2022 21:15
@lazka

lazka commented Jul 4, 2022

Copy link
Copy Markdown
Member Author

There is also uname -o -> Msys, which I'm not sure about... I'll have another look there.

ah, that's hardcoded in coreutils, so nothing we can do here.

@lazka lazka marked this pull request as ready for review July 4, 2022 07:44
@lazka

lazka commented Jul 4, 2022

Copy link
Copy Markdown
Member Author

The "uname -o" part is handled here now: msys2/MSYS2-packages#3080

@dscho dscho left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good!

@lazka

lazka commented Jul 4, 2022

Copy link
Copy Markdown
Member Author

thanks!

@lazka lazka merged commit 041f56f into msys2:msys2-3_3_5-release Jul 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants