search

buggins / ddbc

DDBC is DB Connector for D language (similar to JDBC)
78 stars 45 forks source link

travis builds should use dmd, ldc, and gdc. on multiple platforms #32

Closed SingingBush closed 8 years ago

SingingBush commented 8 years ago

this is the quickest way to ensure that the project works with various os & compiler combinations

SingingBush commented 8 years ago

The OSX builds currently fail:

Undefined symbols for architecture x86_64:
  "_sqlite3_column_database_name", referenced from:     _D4ddbc7drivers10sqliteddbc23SQLITEPreparedStatement14createMetadataMFZC4ddbc4core17ResultSetMetaData in ddbc-tests.o
  "_sqlite3_column_origin_name", referenced from:     _D4ddbc7drivers10sqliteddbc23SQLITEPreparedStatement14createMetadataMFZC4ddbc4core17ResultSetMetaData in ddbc-tests.o
  "_sqlite3_column_table_name", referenced from:     _D4ddbc7drivers10sqliteddbc23SQLITEPreparedStatement14createMetadataMFZC4ddbc4core17ResultSetMetaData in ddbc-tests.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

once this gets resolved the line that allows osx builds to fail should be removed from .travis.yml

SingingBush commented 8 years ago

thanks for merging the changes. I think until the OSX travis problem is resolved it may be worth putting - os: osx back under _allowfailures. Also, the before install part should probably be changed to run a brew update prior to installing sqlite:

before_install:
- "if [ ${TRAVIS_OS_NAME} = 'osx' ]; then brew update && brew install sqlite && brew link --force sqlite; fi"

I've started doing a unittest-cov build in my project which should push the results to coveralls.io. Not sure if that's something you may find useful.

buggins commented 8 years ago

could you please send PR for fixing travis CI?

SingingBush commented 8 years ago

done a PR for this