Skip to content

Commit

Permalink
Config.uk: Imply LIBUKMMAP
Browse files Browse the repository at this point in the history
Change `select LIBUKMMAP` to `imply LIBUKMMAP`. This is because
`LIBUKMMAP` may not be required for the Nginx build (and use
`LIBPOSIX_MMAP` instead).

With `select` in place you cannot disable `LIBUKMMAP`
resulting in a dependency erorr when enabling `LIBPOSIX_MMAP`.

With `imply` as a soft select, `LIBUKMMAP` can be disabled, removing the
dependency error.

Co-authored-by: Marc Rittinghaus <[email protected]>
Signed-off-by: Marc Rittinghaus <[email protected]>
Signed-off-by: Razvan Deaconescu <[email protected]>
Reviewed-by: Marc Rittinghaus <[email protected]>
Approved-by: Simon Kuenzer <[email protected]>
Tested-by: Unikraft CI <[email protected]>
GitHub-Closes: #15
  • Loading branch information
2 people authored and unikraft-bot committed Aug 24, 2023
1 parent 1a35dbb commit 2266770
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Config.uk
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ menuconfig LIBNGINX
select LIBPOSIX_SYSINFO
select LIBPOSIX_SOCKET
select LIBPOSIX_EVENT
select LIBUKMMAP
imply LIBUKMMAP
select LIBUKTIME
select LIBUKSIGNAL
select LIBMUSL
Expand Down

0 comments on commit 2266770

Please sign in to comment.