Skip to content

isolate-check-environment assumes isolate is in PATH for its cgroup check #189

@gkreitz

Description

@gkreitz

The isolate-check-environment script assumes the command isolate is available in $PATH for its cgroup check when it runs isolate to figure out the cgroup root on https://github.com/ioi/isolate/blob/master/isolate-check-environment#L118 . Running isolate-check-environment when isolate is not in the path gives a misleading warning, "cgroup root not found. isolate --cg cannot be used."

This is basically a non-issue with the default configuration where the install goes in /usr/bin. We recently moved where we install isolate and ran into this, so I figured I'd open an issue. Completely understandable if you feel this is a wontfix - we've worked around it, I doubt many will ever be affected, and the fix doesn't feel super clean.

Options I see for fixing:

  • Run $(dirname $0)/isolate, assuming the script is in the same directory as the binary (which is what we did for our workaround)
  • Hard code a full path to the executable based on BINDIR via Makefile
  • Check command -v isolate and give a different error message (or fall back to one of the methods above) when isolate isn't in path.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions