uyjulian / spc2it

Convert SPC files to IT (Impulse Tracker) files
84 stars 10 forks source link

PATH_MAX undeclared identifier #19

Closed jeffythedragonslayer closed 3 years ago

jeffythedragonslayer commented 3 years ago

When I try to build this with the "meson compile -C build" command on windows, I get these errors:

Activating VS 16.11.2
ninja: Entering directory `build'
[1/2] Compiling C object spc2it.exe.p/main.c.obj
FAILED: spc2it.exe.p/main.c.obj
"cl" "-Ispc2it.exe.p" "-I." "-I.." "/MDd" "/nologo" "/showIncludes" "/W2" "/O2" "/Gw" "/Zi" "/Fdspc2it.exe.p\main.c.pdb" /Fospc2it.exe.p/main.c.obj "/c" ../main.c
../main.c(60): error C2065: 'PATH_MAX': undeclared identifier
../main.c(60): error C2057: expected constant expression
../main.c(60): error C2466: cannot allocate an array of constant size 0
../main.c(60): error C2133: 'fn': unknown size
../main.c(156): error C2065: 'PATH_MAX': undeclared identifier
ninja: build stopped: subcommand failed.
uyjulian commented 3 years ago

The specified build environment and commands are located here: https://github.com/uyjulian/spc2it/blob/master/.github/workflows/ci.yml#L53 If you want to add support for alternative build environment, I accept pull requests

jeffythedragonslayer commented 3 years ago

I'm confused. Line 47 is build-win32 but it's followed by Ubuntu commands. Do you mean that you support building inside WSL when on windows, but not native windows?

On Sun, Sep 5, 2021, 1:54 AM Julian Uy @.***> wrote:

The specified build environment and commands are located here: https://github.com/uyjulian/spc2it/blob/master/.github/workflows/ci.yml#L53 If you want to add support for alternative build environment, I accept pull requests

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/uyjulian/spc2it/issues/19#issuecomment-913091676, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADFDUGQMO3BIKIXKSMJVMKTUAMAZHANCNFSM5DOKAK2Q . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

uyjulian commented 3 years ago

Yes, that is correct

jeffythedragonslayer commented 3 years ago

Thank you.