Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Work around build errors on musl libc #4535

Merged
merged 2 commits into from
Aug 15, 2019
Merged

Conversation

r-gr
Copy link
Contributor

@r-gr r-gr commented Aug 13, 2019

Purpose and Motivation

This continues on from and replaces #4391.

Currently musl does not support GLOB_TILDE but it is on their roadmap [1]. This
patch is no fix and changes functionality for musl. A slightly changed
functionality seems better than being unable to build.

For glibc and future versions of musl this change is a noop.

[1] https://wiki.musl-libc.org/roadmap.html

Tests 10 (server_test_run) and 16 (perf_counter_test_run) from make test fail on my machine but they also fail without these changes.

Types of changes

  • Bug fix
  • New feature

To-do list

  • Code is tested
  • All tests are passing
  • Updated documentation
  • This PR is ready for review

Currently musl does not support GLOB_TILDE but it is on their roadmap
[1]. This patch is no fix and changes functionality for musl. A slightly
changed functionality seems better than being unable to build.

For glibc and future versions of musl this change is a noop.

[1] https://wiki.musl-libc.org/roadmap.html
@r-gr
Copy link
Contributor Author

r-gr commented Aug 13, 2019

Sorry about the force push. Messed up the commit message but shouldn't impact anyone.

Copy link
Contributor

@mossheim mossheim left a comment

Choose a reason for hiding this comment

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

Thanks @r-gr !

common/SC_Filesystem_unix.cpp Outdated Show resolved Hide resolved
@mossheim mossheim added the comp: build CMake build system label Aug 14, 2019
Copy link
Contributor

@mossheim mossheim left a comment

Choose a reason for hiding this comment

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

Thanks!

@mossheim mossheim merged commit 834934a into supercollider:3.10 Aug 15, 2019
jpburstrom pushed a commit to jpburstrom/supercollider that referenced this pull request Aug 21, 2019
* Work around build errors on musl libc

Currently musl does not support GLOB_TILDE but it is on their roadmap
[1]. This patch is no fix and changes functionality for musl. A slightly
changed functionality seems better than being unable to build.

For glibc and future versions of musl this change is a noop.

[1] https://wiki.musl-libc.org/roadmap.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp: build CMake build system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants