Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Failed to compile on solaris #58

Closed
chrislusf opened this issue Jul 24, 2014 · 5 comments
Closed

Failed to compile on solaris #58

chrislusf opened this issue Jul 24, 2014 · 5 comments
Assignees
Labels

Comments

@chrislusf
Copy link

Got this error when compiling for solaris:

github.com/syndtr/goleveldb/leveldb/storage

../../../../../github.com/syndtr/goleveldb/leveldb/storage/file_storage.go:68: undefined: newFileLock
../../../../../github.com/syndtr/goleveldb/leveldb/storage/file_storage.go:79: undefined: rename
../../../../../github.com/syndtr/goleveldb/leveldb/storage/file_storage.go:274: undefined: rename
../../../../../github.com/syndtr/goleveldb/leveldb/storage/file_storage.go:316: undefined: rename

@calmh
Copy link
Contributor

calmh commented Jul 24, 2014

I've noticed the same, considered fixing it but not gotten that far yet. :)

@syndtr syndtr added the todo label Jul 24, 2014
@syndtr syndtr self-assigned this Jul 24, 2014
syndtr added a commit that referenced this issue Jul 24, 2014
@syndtr
Copy link
Owner

syndtr commented Jul 24, 2014

Let me know if it works or not. Also please run manualtest/filelock. Thanks.

@calmh
Copy link
Contributor

calmh commented Jul 24, 2014

Looks good?

jb@zlogin3:~ $ ./test 
Using path: /tmp/goleveldb_filelock_test
Executing child -- first test (expecting error)
[Child] Using path: /tmp/goleveldb_filelock_test
[Child] Child flag set.
[Child] Could not open storage: resource temporarily unavailable
Got error from child: exit status 10 (expected)
Executing child -- second test
[Child] Using path: /tmp/goleveldb_filelock_test
[Child] Child flag set.
jb@zlogin3:~ $ uname -a
SunOS zlogin3 5.11 joyent_20140710T224431Z i86pc i386 i86pc Solaris

@syndtr
Copy link
Owner

syndtr commented Jul 24, 2014

Yes it is.

@chrislusf
Copy link
Author

Thanks for resolving this so quickly!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants