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

Update httpserver.cpp header #740

Merged
merged 1 commit into from
Aug 7, 2020
Merged

Update httpserver.cpp header #740

merged 1 commit into from
Aug 7, 2020

Conversation

garretlaxton
Copy link
Contributor

httpserver.cpp:73:10: error: 'deque' in namespace 'std' does not name a template type
73 | std::deque<std::unique_ptr> queue;
| ^~~~~
httpserver.cpp:32:1: note: 'std::deque' is defined in header ''; did you forget to '#include '?

httpserver.cpp:73:10: error: 'deque' in namespace 'std' does not name a template type
   73 |     std::deque<std::unique_ptr<WorkItem>> queue;
      |          ^~~~~
httpserver.cpp:32:1: note: 'std::deque' is defined in header '<deque>'; did you forget to '#include <deque>'?
@navbuilder
Copy link

A new build of 7a2919b has completed succesfully!
Binaries available at https://build.nav.community/binaries/master

5 similar comments
@navbuilder
Copy link

A new build of 7a2919b has completed succesfully!
Binaries available at https://build.nav.community/binaries/master

@navbuilder
Copy link

A new build of 7a2919b has completed succesfully!
Binaries available at https://build.nav.community/binaries/master

@navbuilder
Copy link

A new build of 7a2919b has completed succesfully!
Binaries available at https://build.nav.community/binaries/master

@navbuilder
Copy link

A new build of 7a2919b has completed succesfully!
Binaries available at https://build.nav.community/binaries/master

@navbuilder
Copy link

A new build of 7a2919b has completed succesfully!
Binaries available at https://build.nav.community/binaries/master

@aguycalled
Copy link
Member

@garretlaxton in what enviroment were you seeing that build issue?

@garretlaxton
Copy link
Contributor Author

@aguycalled FreeBSD 12.1, I used the solution used on BTC for the same issue. bitcoin/bitcoin#17249

@mxaddict
Copy link
Contributor

mxaddict commented Aug 7, 2020

Linux build still works after the change, I will assume that this fixes the FreeBSD issue and approve the PR

@mxaddict mxaddict self-requested a review August 7, 2020 14:20
@mxaddict mxaddict merged commit 52f75ed into navcoin:master Aug 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants