Skip to content
This repository was archived by the owner on Feb 1, 2026. It is now read-only.

Added go get and godoc compatibility and improved error handling in driver#10

Open
silvasur wants to merge 4 commits intoweigj:masterfrom
silvasur:master
Open

Added go get and godoc compatibility and improved error handling in driver#10
silvasur wants to merge 4 commits intoweigj:masterfrom
silvasur:master

Conversation

@silvasur
Copy link

@silvasur silvasur commented Mar 3, 2013

  • The code does now live in the subdirectory odbc, which does not have hyphens and therefore has an unambiguous import name.
  • The driver now imports from github.com/kch42/go-odbc/odbc instead of odbc (if you want to pull the changeset, you need to change the kch42 to weigj, of course).
  • Since now every package import name is based on the github-URL, go get can automatically download and install the packages.
  • I also moved some documentation from the README files to the package sources. godoc is therefore able to display this documentation.
  • I also changed the error reporting of the driver, more details in the description of the second commit.

Kevin Chabowski and others added 4 commits March 3, 2013 21:34
* Moved sources to subdirectory that has a name, that can be imported
  (i.e. has no hyphen in it)
* Deleted build scripts (not needed, go get / go install will compile
  the package without additional build instructions)
* Also moved some documentation from README files to the packages, so
  they are accessible to godoc now.
When a (*ODBCError)(nil) error (returned by odbc functions) was returned
as an error, it was no longer treated as a nil value, therefore it was
not possible to detect them with the ususal `if err != nil {..}` method.
Since field_len is the size in (wide) characters and SQLGetData expects
the size in bytes, in some situations only half of the data was actually
read from the database.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant