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

Jump to bottom

Support for DragonFlyBSD/NetBSD/OpenBSD #65

Closed
cmumford opened this issue Sep 9, 2014 · 5 comments
Closed

Support for DragonFlyBSD/NetBSD/OpenBSD #65

cmumford opened this issue Sep 9, 2014 · 5 comments
Assignees
@cmumford

Comments

@cmumford
Copy link
Contributor

cmumford commented Sep 9, 2014

Original issue 59 created by andrew@hijacked.us on 2011-12-03T03:42:50.000Z:

What steps will reproduce the problem?

  1. Try to compile on one of the above operating systems
  2. The compile will fail because of missing includes and missing fdatasync, etc

What is the expected output? What do you see instead?

leveldb should compile correctly, instead it fails.

Please provide any additional information below.

Patch is attached. This supersedes issue 31.
@cmumford
Copy link
Contributor Author

cmumford commented Sep 9, 2014

Comment #1 originally posted by piotr.sikora@frickle.com on 2011-12-13T06:32:39.000Z:

Andrew, there are two bugs in your patch:

  1. OpenBSD is using "-lpthread" instead of "-pthread" (like it was in patch for # 31),
  2. DragonFly defines PLATFORM=OS_OPENBSD.

@cmumford cmumford self-assigned this Sep 9, 2014
@cmumford
Copy link
Contributor Author

cmumford commented Sep 9, 2014

Comment #2 originally posted by andrew@hijacked.us on 2011-12-17T04:37:12.000Z:

Updated patch.

@cmumford
Copy link
Contributor Author

cmumford commented Sep 9, 2014

Comment #3 originally posted by andrew@hijacked.us on 2012-03-01T14:11:34.000Z:

Is this ever going to go anywhere?

@cmumford
Copy link
Contributor Author

cmumford commented Sep 9, 2014

Comment #4 originally posted by sanjay@google.com on 2012-03-05T18:40:31.000Z:

Fixed.

@cmumford
Copy link
Contributor Author

cmumford commented Sep 9, 2014

Comment #5 originally posted by dgrogan@chromium.org on 2012-06-01T23:57:04.000Z:

Issue 31 has been merged into this issue.

@cmumford cmumford closed this as completed Sep 9, 2014
maochongxin pushed a commit to maochongxin/leveldb that referenced this issue Jul 21, 2022
@dmah42
Fix example in README.md to compile.
4932b72 
Fixes google#65
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cmumford cmumford

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@cmumford