Summary: | games-engines/box2d-2.4.1 failed to emerge - SANDBOX ACCESS VIOLATION with /doc | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Leonid Kopylov <leonchik1976> |
Component: | Current packages | Assignee: | Gentoo Games <games> |
Status: | CONFIRMED --- | ||
Severity: | normal | CC: | dememax, esigra, jaak, john, kripton, walch.martin |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
build.log
Alter WARN_LOGFILE to avoid sandbox violation Reproduced on 2024-05-06 |
Description
Leonid Kopylov
2023-05-24 14:21:52 UTC
Created attachment 862372 [details]
build.log
F: mkdir S: deny P: /docs A: /docs R: /docs C: /usr/bin/doxygen [31;01m*[0m ---- Created attachment 883012 [details, diff]
Alter WARN_LOGFILE to avoid sandbox violation
The sandbox violation appears to happen because doxygen is attempting to make a warnings.txt file. The location is set by the variable:
WARN_LOGFILE = "@BOX2D_BINARY_DIR@/docs/warnings.txt"
BOX2D_BINARY_DIR is not set, so this attempts to create and use a /docs directory, resulting in the sandbox violation.
This patch attempts to fix the problem by swapping CMAKE_BINARY_DIR in for BOX2D_BINARY_DIR. CMAKE_BINARY_DIR is set, and this results in the warnings.txt file being written in the build directory (not somewhere where it gets installed, but in a directory that contains other information about building the docs). This seems like a decent location for the warnings; alternatively, that WARN_LOGFILE may be set to an empty string, which causes the warnings to be written to stderr (and therefore ending up in the build log instead of a file in the build directory).
I can confirm the issue and that the attached patch fixes it. The original issue might have been caused by case sensitivity as using box2d_BINARY_DIR instead of BOX2D_BINARY_DIR would also work. The attached patch instead changed it to CMAKE_BINARY_DIR, and this actually seems to be a better approach as the docs/Doxygen.in file already contains similar references to CMAKE_BINARY_DIR. Created attachment 892349 [details]
Reproduced on 2024-05-06
Reproduced with the current up-do-date Gentoo.
confirmed still an issue |