dinedal / textql

Execute SQL against structured text like CSV or TSV
MIT License
9.05k stars 300 forks source link

cc: warning: argument unused during compilation: '-pthread' #48

Closed jungle-boogie closed 8 years ago

jungle-boogie commented 8 years ago

Hello,

Trying from your latest release:

 % go get -u github.com/dinedal/textql/...
# github.com/mattn/go-sqlite3
cc: warning: argument unused during compilation: '-pthread' 

What's this mean to you?

dinedal commented 8 years ago

Looks like you used clang to compile go-sqlite3

This is probably not an issue.

Might be of some help to you: http://stackoverflow.com/questions/13238511/xcode-clang-clang-warning-argument-unused-during-compilation-fcheck-new

jungle-boogie commented 8 years ago

Hi @dinedal,

Thanks for the reply.

In my gopath, the binary is called cmd. Is that correct?

dinedal commented 8 years ago

Err, sorry about that. :) fixing it

jungle-boogie commented 8 years ago

Err, sorry about that. :) fixing it

Thanks!

can you post a message when its corrected?

dinedal commented 8 years ago

Can you attempt it now?

jungle-boogie commented 8 years ago

Good now but what's causing this:

% textql -version
        [blank]           

No version number is displayed.

dinedal commented 8 years ago

You built from source, that's expected :)

If you download a release and make from there you get a build that's the version

Sorry, but go doesn't really support versioning with go get...

jungle-boogie commented 8 years ago

Sorry, but go doesn't really support versioning with go get...

I can do it with https://github.com/mholt/caddy but maybe he did something different.

At any rate, thanks for the new release and correcting the issue so quickly.

Thanks! :+1:

dinedal commented 8 years ago

He has the version as a constant in the source. It's a trade off :) There's no good way for anyone to know if a caddy version is 0.8 from source or 0.8 as released.

I might change over though, later, and it would fix it.

jungle-boogie commented 8 years ago

okay, got it. Thanks for your expert eyes into this matter!