search

decred / gominer

Go (golang) based GPU miner for Decred.
GNU General Public License v3.0
160 stars 80 forks source link

Some file location updates for linux. #147

Closed Baggins800 closed 7 years ago

Baggins800 commented 7 years ago

Update such that it compiles on arch linux.

jcvernaleo commented 7 years ago

I that we had already been building the cuda versions on arch linux Seems to build fine on the box I had been testing with. @jolan is more of an arch linux guy than I am so he should probably comment here and decide.

jolan commented 7 years ago

It compiles fine for me on Arch Linux with gcc 6.3.1-2 / gcc version 6.3.1 20170306 (GCC). I'm curious to know what exactly you're seeing and if the gcc version matches.

Baggins800 commented 7 years ago

Got /usr/bin/ld: xx/xxx.o: relocation R_X86_64_32 against [.bss'/.rodata'/symbol `xxx'] can not be used when making a shared object; recompile with -fPIC. errors, seems like the same gcc version, I added the -fPIC and it fixed it (with the "< >" removed). I dont know if it is the nvcc version? I used v8.0.61?

chappjc commented 7 years ago

Why the ./? Are there errors with quotes, but not the dot slash? I agree about quotes vs. angle brackets though.

Baggins800 commented 7 years ago

The ./ can be removed, is shows current directory, but is redundant in this case.

jolan commented 7 years ago

Can you please go ahead and remove ./?

jcvernaleo commented 7 years ago

Agreed, not a fan of ./ in build things like this.

jolan commented 7 years ago

Thank you. I still need to test compile on Windows to make sure it's okay with this change. Unfortunately my method of remote access broke so I can't do so now.