- edited description
Cannot build on Cygwin
Issue #191
resolved
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)
-
reporter -
reporter - edited description
-
repo owner 報告ありがとうございます。
これ、そもそも
thread_cleanup_push
とthread_cleanup_pop
の pthread 環境用の定義がここにあること自体が間違っている気がしますね(この定義自体が__try
のない MinGW をはじくためのものなので)。なので、pthread 環境では無条件で定義するようにしようかなぁと思います。手元の Cygwin では再現していないのでそれで直るのか不安ではあるのですが… -
repo owner - changed status to resolved
Forcing Cygwin to use pthread. (Fixes
#191) Removing unused C function.→ <<cset a9d1c9c76a94>>
- Log in to comment