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

Fix windows build (missing ssize_t type definition), take 2. #336

Merged
merged 1 commit into from Oct 7, 2017

Conversation

@ksooo
Copy link
Member

commented Oct 7, 2017

Followup to #335, which did not solve the build problem.

@ksooo ksooo added the Fix label Oct 7, 2017

@ksooo ksooo merged commit d4e599f into kodi-pvr:master Oct 7, 2017

0 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@ksooo ksooo deleted the ksooo:fix-windows-build-2 branch Oct 7, 2017

@ksooo

This comment has been minimized.

Copy link
Member Author

commented on b494b06 Oct 7, 2017

I'm getting crazy. Still does not build on Windows. Next error (and I'm clueless about this one):

[ 80%] Building CXX object CMakeFiles/pvr.hts.dir/src/tvheadend/entity/Tag.cpp.obj
Tag.cpp

C:\jenkins\workspace\WIN-32\cmake\addons\build\pvr.hts\src\tvheadend\entity\Tag.cpp(81): error C2065: 'channel_type_t': undeclared identifier
C:\jenkins\workspace\WIN-32\cmake\addons\build\pvr.hts\src\tvheadend\entity\Tag.cpp(81): error C2146: syntax error: missing ')' before identifier 'eType'
C:\jenkins\workspace\WIN-32\cmake\addons\build\pvr.hts\src\tvheadend\entity\Tag.cpp(82): error C2143: syntax error: missing ';' before '{'
C:\jenkins\workspace\WIN-32\cmake\addons\build\pvr.hts\src\tvheadend\entity\Tag.cpp(82): error C2447: '{': missing function header (old-style formal list?)
NMAKE : fatal error U1077: 'C:\PROGRA~2\MICROS~1.0\VC\bin\AMD64_~2\cl.exe' : return code '0x2'
Stop.

channel_type_tis defined in HTSPTypes.h which gets included by Tag.cpp via Tag.h...

@afedchin maybe you can help me out (I don't have a windows machine)?

This comment has been minimized.

Copy link
Member Author

replied Oct 7, 2017

Found it (I hope).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.