From bd0baf23383951182b10261b076cf6268152a68b Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Mon, 4 Jul 2011 14:29:59 +0200 Subject: [PATCH] Check that PR_SET_NAME is defined. Avoids breaking the build with older (pre-2006) linux kernels. Raises a JS exception if the script tries to assign to `process.title`. Fixes #840. --- src/platform_linux.cc | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/platform_linux.cc b/src/platform_linux.cc index 54937b487ad..827b8d4ea34 100644 --- a/src/platform_linux.cc +++ b/src/platform_linux.cc @@ -64,9 +64,15 @@ char** Platform::SetupArgs(int argc, char *argv[]) { void Platform::SetProcessTitle(char *title) { +#ifdef PR_SET_NAME if (process_title) free(process_title); process_title = strdup(title); prctl(PR_SET_NAME, process_title); +#else + Local ex = Exception::Error( + String::New("'process.title' is not writable on your system, sorry.")); + ThrowException(ex); // Safe, this method is only called from the main thread. +#endif }