Cannot build on Cygwin

Issue #191 resolved
Yuki Okumura created an issue

src/sagittarius/thread.h は、pthread環境の検出のために!defined(_WIN32) || !defined(_WIN64)使用しています。これはCygwinでのビルドをext/filewatchで失敗させます。

FilewatchでOSヘッダを最後にincludeする https://bitbucket.org/okuoku/yunibase-impl-stable/commits/799afb877696bdb430ccb4d2eb358cd1d9f276e8

または、sagittarius/thread.h__CYGWIN__にpthread環境を強制するのが良いと思います。

Checked on: 0.7.6, https://bitbucket.org/ktakashi/sagittarius-scheme/commits/a1ac8341ddc750a94590f55a58a0bfd83e8c6081

Comments (4)

  1. Takashi Kato repo owner

    報告ありがとうございます。

    これ、そもそも thread_cleanup_pushthread_cleanup_pop の pthread 環境用の定義がここにあること自体が間違っている気がしますね(この定義自体が __try のない MinGW をはじくためのものなので)。なので、pthread 環境では無条件で定義するようにしようかなぁと思います。手元の Cygwin では再現していないのでそれで直るのか不安ではあるのですが…

  2. Log in to comment