03:00 < GitHub88> [bitcoin] alfiedotwtf opened pull request #7612: [doc] Typo fix (master...master) https://github.com/bitcoin/bitcoin/pull/7612

03:08 < gamersg> is it ok to use C++11 in the master branch?

03:30 < sipa> gamersg: not yet

03:31 < sipa> but probably soon

03:33 < gamersg> alrite. Just thinking, do we plan to keep the current C and C++ styles mixed in the codebase

04:46 < GitHub42> [bitcoin] GamerSg opened pull request #7613: Add autocomplete to bitcoin-qt's console window. (master...autocomplete) https://github.com/bitcoin/bitcoin/pull/7613

04:48 < Luke-Jr> ooh

04:51 < gamersg> its a pretty small change, first time looking into bitcoin code

04:52 < PRab_> gamersg: Does it only autocomplete the first word, or the entire command?

04:54 < PRab_> (No big deal if its only the first word, thats still a big improvement)

04:56 < gamersg> just the command, lemme take a screenshot

04:56 < gamersg> was much easier than i thought, QT has a class to do it

04:57 < PRab_> gotcha. I had thought about it previously, but when I looked at https://github.com/bitcoin/bitcoin/blob/master/contrib/bitcoind.bash-completion I ran away.

05:03 < gamersg> http://imgur.com/wF667ky

05:04 < gamersg> i probably more time figuring out git since im new to it

05:04 < gamersg> spent*

05:05 < PRab_> Looks good to me.

05:06 < PRab_> Git is a bit tricky. I just spend an hour today teaching a coworker about branches, forks, remotes, etc.

05:06 < PRab_> Once you get the hang of it, it is very powerful.

05:06 < PRab_> I'm just sad that sourcetree constantly hangs for me.

05:08 < PRab_> gamersg: It probably wouldn't be a bad idea to link to the image from your pull request. It helps make it a bit more concrete.

05:14 < gamersg> PRab_: ok now, to figure out how to do that

05:15 < PRab_> Just add a comment on github with a link. If you want I can do it for you.

05:15 < PRab_> Oh, looks like you did it!

05:16 < gamersg> ya good that it supports file attachments

05:16 < PRab_> Yep, github is pretty nice.

05:17 < gamersg> bash completion is probably a nightmare

05:17 < gamersg> unless u are a shell scripting expert

05:21 < gamersg> is there a define signifying NOWALLET?

05:22 < PRab_> #ifdef ENABLE_WALLET?

05:24 < gamersg> hmm ok, travis seems to fail for NO WALLET builds

05:25 < gamersg> there ought to be a variable as well, since wallet can be disabled from command line

05:25 < PRab_> I'm heading to bed. Good luck.

05:26 < PRab_> It wasn't #ifdef ENABLE_WALLET ?

05:34 < gamersg> define is compile time

05:34 < gamersg> variable will affect runtime

05:35 < gamersg> yeah, i gtg as well, will look into it tmr

06:11 < GitHub146> [bitcoin] luke-jr opened pull request #7614: Bugfix: gitian: Add curl to packages (now needed for depends) (master...depends_curl) https://github.com/bitcoin/bitcoin/pull/7614

12:23 < GitHub93> [bitcoin] MarcoFalke opened pull request #7615: [wallet] Couple MIN_CHANGE with minimum relay fee (master...Mf1602-walletMinChange) https://github.com/bitcoin/bitcoin/pull/7615

14:00 < GitHub56> [bitcoin] MarcoFalke opened pull request #7616: Revert "depends: fix Boost 1.55 build on GCC 5" (master...Mf1602-boost155) https://github.com/bitcoin/bitcoin/pull/7616

17:21 < GitHub116> [bitcoin] MarcoFalke opened pull request #7617: [doc] Minor corrections to release notes and links (master...Mf1602-trivial9) https://github.com/bitcoin/bitcoin/pull/7617

20:54 < btcdrak> does anyone know why gitian builder keeps asking for sudo password before making new image? It's supposed to be in the sudoers, I think there might be some other command that needs to be whitelisted.

20:55 < btcdrak> I am sure I remember this being discussed before in this channel, but cant find it in the logs

20:55 < sipa> i think it is vmbuilder that builds the image

20:59 < PRab> btcdrak: I remember that discussion also.

21:00 < btcdrak> oh I found it https://botbot.me/freenode/bitcoin-core-dev/2016-02-18/?msg=60419903&page=3

21:01 < btcdrak> need to add an entry for lxc-execute as well

21:06 < PRab> btcdrak: Testing a change to /doc/gitian-building.md to see if I can automate it.

21:06 < btcdrak> It works. I'm submitting a PR

21:07 < PRab> Ok, I'll just test your PR.

21:14 < GitHub34> [bitcoin] btcdrak opened pull request #7619: Add missing sudo entry in gitian VM setup. (master...gitian-sudo) https://github.com/bitcoin/bitcoin/pull/7619

21:17 < PRab> btcdrak: Doesn't > replace the entire file?

21:17 < PRab> I think it needs >> instead

21:17 < btcdrak> oops :) I assumed the original line was >> fixing

21:18 < btcdrak> fixed

21:19 < PRab> Looks better.