<@wumpus> luke-jr: no, I don't
<@wumpus> luke-jr: we do test windows on travis AFAIK?
<@wumpus> e.g. cross-build for windows, then run the tests in wine
<@wumpus> but not sure this covers all the tests
< luke-jr> wumpus: the Trusty changes appear to have disabled the GUI build and WINE testing :/
< luke-jr> ie, just builds bitcoind and doesn't test it
< sipa> seems it does
< luke-jr> hm
< sipa> ah, but not the RPC tests
< sipa> only comparison tool and unit tests
< sipa> wumpus: does that still apply?
< luke-jr> - env: HOST=i686-w64-mingw32 PPA="ppa:ubuntu-wine/ppa" PACKAGES="nsis gcc-mingw-w64-i686 g++-mingw-w64-i686 binutils-mingw-w64-i686 mingw-w64-dev wine1.7 bc" RUN_TESTS=true GOAL="deploy" BITCOIN_CONFIG="--enable-gui --enable-reduce-exports" MAKEJOBS="-j2"
< luke-jr> + env: HOST=i686-w64-mingw32 DPKG_ADD_ARCH="i386" DEP_OPTS="NO_QT=1" PACKAGES="nsis g++-mingw-w64-i686 wine1.6 bc" RUN_TESTS=true GOAL="install" BITCOIN_CONFIG="--enable-reduce-exports"
< luke-jr> I guess I misread the RUN_TESTS part, but what does the GOAL change imply?
< sipa> i think it just means we don't build the installer
< Chris_Stewart_5> sipa: Do you have any thoughts on property based checking? I'm starting to integrate it into a library I'm writing and it seems it would be useful in Bitcoin Core. I was looking at this library for C++
< Chris_Stewart_5> I'm wondering if there is any outright objections before I put more time into investigating this for Core..
<@wumpus> luke-jr: yes the GUI build is disabled for travis, for all platforms, that's intentional as there is still some caching issue
<@wumpus> sipa: no idea, could be that the RPC tests could be re-enabled, just no one has tried since
< sipa> Chris_Stewart_5: no objections to the concept, but i have no experience with the code/logistics of integrating it
< sipa> Chris_Stewart_5: perhaps you want to talk to Marco Falke (our qa maintainer)
<@wumpus> Chris_Stewart_5: I have no experience with it either, if you can apply it to improve test coverage of bitcoin core, why not try it
<@wumpus> at first glance it looks like a type of fuzzing, you could take a look at https://github.com/bitcoin/bitcoin/pull/7940
< Chris_Stewart_5> Will do. Is it best to submit a pull request with a [WIP] on it, or just develop on my own branch and submit a pull request later on?
<@wumpus> that's really up to you
<@wumpus> in both cases you develop on your own branch, in the former cast it has somewhat more visibility, which is useful if you want other people's opinions about things
< Chris_Stewart_5> good point, another question, is there any vetting process for dependencies that are being added to our src files?
< Chris_Stewart_5> I get that this one isn't as crucial since it is test dependecy, but it would be important if a dependency was affect src files
< Chris_Stewart_5> src meaning everything but the child test dir
< sipa> Chris_Stewart_5: if you add a dependency, you need to make sure that it's available in the depends system (see the depends subdir), so that it's available to the deterministic build system
< Chris_Stewart_5> Hmm ok. I'm just starting to look at this stuff. Doesn't look quite as simple as maven on the JVM :-). Cory is the build expert right?
< sipa> Chris_Stewart_5: indeed
< sipa> Chris_Stewart_5: deterministic building is a pretty new concept, and there are few tools available to do it for you, so it's all pretty custom
< sipa> Chris_Stewart_5: though, generally don't worry about that, if it's useful we'll find a way to integrate it
< Chris_Stewart_5> sipa: Ok, I'm just going to hack it together to start and submit a pull request testing a few simple properties
< GitHub4> [bitcoin] wbinns opened pull request #8224: readme: Revise 'an experimental new' phrasing in reference to Bitcoin (master...binns-update-readme) https://github.com/bitcoin/bitcoin/pull/8224
< Chris_Stewart_5> So i've created a file inside of package giving the instructions of how to build the library, what is the next step? I've tried to add the header to the test file and it fails.
< Chris_Stewart_5> I've also added it to the first line in packages.mk
< GitHub63> [bitcoin] wbinns opened pull request #8226: contributing.md: Fix formatting (line lengths and smart quotes) (master...binns-contributing-formatting) https://github.com/bitcoin/bitcoin/pull/8226