Skip to content

feat(platform): add missing freebsd headers#192

Merged
etr merged 1 commit intoetr:masterfrom
fzipi:add-freebsd-support
Apr 14, 2020
Merged

feat(platform): add missing freebsd headers#192
etr merged 1 commit intoetr:masterfrom
fzipi:add-freebsd-support

Conversation

@fzipi
Copy link
Copy Markdown
Contributor

@fzipi fzipi commented Apr 13, 2020

Signed-off-by: Felipe Zipitria fzipitria@perceptyx.com

Identify the Bug

This PR fixes #191.

Description of the Change

Only when compiling in FreeBSD, include the netinet/in.h header.

Alternate Designs

None.

Possible Drawbacks

None.

Verification Process

Created patch. Compiled. Compilation finished properly.

Release Notes

  • Fixed compilation in FreeBSD.

Signed-off-by: Felipe Zipitria <fzipitria@perceptyx.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 13, 2020

Codecov Report

Merging #192 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #192   +/-   ##
=======================================
  Coverage   95.19%   95.19%           
=======================================
  Files          35       35           
  Lines        3140     3140           
=======================================
  Hits         2989     2989           
  Misses        151      151           
Impacted Files Coverage Δ
src/http_utils.cpp 99.17% <ø> (ø)
src/webserver.cpp 90.47% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a6c82e5...9ecfd06. Read the comment docs.

@etr etr merged commit b2d69d9 into etr:master Apr 14, 2020
@etr
Copy link
Copy Markdown
Owner

etr commented Apr 14, 2020

Thanks for fixing this. Merging.

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.

[BUG] FreeBSD build fails using arpa/inet.h

2 participants