conda-forge / libtheora-feedstock

A conda-smithy repository for libtheora.
BSD 3-Clause "New" or "Revised" License
0 stars 10 forks source link

Build on Windows #7

Closed jakirkham closed 4 years ago

jakirkham commented 6 years ago

Fixes https://github.com/conda-forge/libtheora-feedstock/issues/4

Drops the skip and starts building on Windows.

@conda-forge-admin, please re-render.

Checklist

conda-forge-linter commented 6 years ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

jakirkham commented 6 years ago

Seems to failing. Any ideas?

CMake Error at CMakeLists.txt:84 (add_library):
  Cannot find source file:
    lib/collect.c
  Tried extensions .c .C .c++ .cc .cpp .cxx .cu .m .M .mm .h .hh .h++ .hm
  .hpp .hxx .in .txx
CMake Error at CMakeLists.txt:84 (add_library):
  No SOURCES given to target: THEORA

ref: https://ci.appveyor.com/project/conda-forge/libtheora-feedstock/build/1.0.5

jakirkham commented 6 years ago

Appears we cannot use inline assembly with Visual Studio on 64-bit. Not sure how to proceed. 😕

ref: https://stackoverflow.com/a/14292031 ref: https://ci.appveyor.com/project/conda-forge/libtheora-feedstock/build/1.0.9

ihnorton commented 5 years ago

vcpkg has a working CMakeList for theora:

https://github.com/Microsoft/vcpkg/blob/master/ports/libtheora/CMakeLists.txt

big differences are they build the VC-specific fragments from lib/x86_vc, and only set -DOC_X86_AS on x86.

jakirkham commented 4 years ago

This was addressed in PR ( https://github.com/conda-forge/libtheora-feedstock/pull/10 ). Closing...