November 2011 - darwinbuild-changes

[969] tags/darwinbuild-26/
by source_changes＠macosforge.org 07 Nov '11

07 Nov '11

Revision: 969 http://trac.macosforge.org/projects/darwinbuild/changeset/969 Author: kvv(a)apple.com Date: 2011-11-07 13:05:19 -0800 (Mon, 07 Nov 2011) Log Message: ----------- "Tagging darwinbuild-26 from http://svn.macosforge.org/repository/darwinbuild/trunk" Added Paths: ----------- tags/darwinbuild-26/ Property changes on: tags/darwinbuild-26 ___________________________________________________________________ Added: svn:mergeinfo + /branches/PR-10363375:959-960 /branches/PR-10397485:967 /branches/PR-4841388:399-419 /branches/PR-6358021:442-443 /branches/PR-6392966:423-427 /branches/PR-6398060:433-434 /branches/PR-6493844:460-461 /branches/PR-6497694:466-468,471 /branches/PR-6634286:632-650 /branches/PR-6688645:479-490 /branches/PR-6722857:495-499 /branches/PR-6729491:655-664 /branches/PR-6973110:804-813 /branches/PR-7250612:635-650 /branches/PR-7341154:682-694 /branches/PR-7431723:660-664 /branches/PR-7461534:650-664 /branches/PR-7482850:670-671 /branches/PR-7489777:676-731 /branches/PR-7529688:692-694 /branches/PR-7593824:739-772 /branches/PR-7598640:703-731 /branches/PR-7748469:777-785 /branches/PR-7765119:790-791 /branches/PR-7798586:796-799 /branches/PR-7872907:830-840 /branches/PR-7935095:819-821 /branches/PR-8116613:849 /branches/PR-8279204:854-862 /branches/PR-8416637:870-880 /branches/PR-8486662:885-889 /branches/PR-8488185:894-898 /branches/PR-8604911:903-905 /branches/PR-8817822:917-933 /branches/PR-8908468:912

1 0

[968] trunk
by source_changes＠macosforge.org 07 Nov '11

07 Nov '11

Revision: 968 http://trac.macosforge.org/projects/darwinbuild/changeset/968 Author: kvv(a)apple.com Date: 2011-11-07 13:05:14 -0800 (Mon, 07 Nov 2011) Log Message: ----------- Merged PR-10397485 Modified Paths: -------------- trunk/CHANGES trunk/darwintrace/darwintrace.c Property Changed: ---------------- trunk/ Property changes on: trunk ___________________________________________________________________ Modified: svn:mergeinfo - /branches/PR-10363375:959-960 /branches/PR-4841388:399-419 /branches/PR-6358021:442-443 /branches/PR-6392966:423-427 /branches/PR-6398060:433-434 /branches/PR-6493844:460-461 /branches/PR-6497694:466-468,471 /branches/PR-6634286:632-650 /branches/PR-6688645:479-490 /branches/PR-6722857:495-499 /branches/PR-6729491:655-664 /branches/PR-6973110:804-813 /branches/PR-7250612:635-650 /branches/PR-7341154:682-694 /branches/PR-7431723:660-664 /branches/PR-7461534:650-664 /branches/PR-7482850:670-671 /branches/PR-7489777:676-731 /branches/PR-7529688:692-694 /branches/PR-7593824:739-772 /branches/PR-7598640:703-731 /branches/PR-7748469:777-785 /branches/PR-7765119:790-791 /branches/PR-7798586:796-799 /branches/PR-7872907:830-840 /branches/PR-7935095:819-821 /branches/PR-8116613:849 /branches/PR-8279204:854-862 /branches/PR-8416637:870-880 /branches/PR-8486662:885-889 /branches/PR-8488185:894-898 /branches/PR-8604911:903-905 /branches/PR-8817822:917-933 /branches/PR-8908468:912 + /branches/PR-10363375:959-960 /branches/PR-10397485:967 /branches/PR-4841388:399-419 /branches/PR-6358021:442-443 /branches/PR-6392966:423-427 /branches/PR-6398060:433-434 /branches/PR-6493844:460-461 /branches/PR-6497694:466-468,471 /branches/PR-6634286:632-650 /branches/PR-6688645:479-490 /branches/PR-6722857:495-499 /branches/PR-6729491:655-664 /branches/PR-6973110:804-813 /branches/PR-7250612:635-650 /branches/PR-7341154:682-694 /branches/PR-7431723:660-664 /branches/PR-7461534:650-664 /branches/PR-7482850:670-671 /branches/PR-7489777:676-731 /branches/PR-7529688:692-694 /branches/PR-7593824:739-772 /branches/PR-7598640:703-731 /branches/PR-7748469:777-785 /branches/PR-7765119:790-791 /branches/PR-7798586:796-799 /branches/PR-7872907:830-840 /branches/PR-7935095:819-821 /branches/PR-8116613:849 /branches/PR-8279204:854-862 /branches/PR-8416637:870-880 /branches/PR-8486662:885-889 /branches/PR-8488185:894-898 /branches/PR-8604911:903-905 /branches/PR-8817822:917-933 /branches/PR-8908468:912 Modified: trunk/CHANGES =================================================================== --- trunk/CHANGES 2011-11-07 19:03:17 UTC (rev 967) +++ trunk/CHANGES 2011-11-07 21:05:14 UTC (rev 968) @@ -1,7 +1,11 @@ Darwin Build Scripts Change History ----------------------------------- +Release 26 [6-Nov-2011] + - darwintrace: preserve DYLD_INSERT_LIBRARIES and DARWINTRACE_LOG + - darwintrace: interpose posix_spawn + Release 25 [4-Nov-2011] - - darwinup: support for rebuilding xpchelper cache + - darwinup: support for rebuilding xpchelper cache Release 24.1 [17-Jun-2011] - Build fixes for newer compilers Modified: trunk/darwintrace/darwintrace.c =================================================================== --- trunk/darwintrace/darwintrace.c 2011-11-07 19:03:17 UTC (rev 967) +++ trunk/darwintrace/darwintrace.c 2011-11-07 21:05:14 UTC (rev 968) @@ -1,5 +1,5 @@ /* - * Copyright (c) 2005 Apple Computer, Inc. All rights reserved. + * Copyright (c) 2005-2011 Apple Inc. All rights reserved. * * @APPLE_BSD_LICENSE_HEADER_START@ * @@ -32,6 +32,7 @@ #include <crt_externs.h> #include <fcntl.h> +#include <spawn.h> #include <stdarg.h> #include <stdio.h> #include <stdlib.h> @@ -47,7 +48,7 @@ #define DARWINTRACE_DEBUG_OUTPUT 0 #define DARWINTRACE_START_FD 101 #define DARWINTRACE_STOP_FD 200 -#define DARWINTRACE_BUFFER_SIZE 1024 +#define DARWINTRACE_BUFFER_SIZE 1024 #if DARWINTRACE_DEBUG_OUTPUT #define dprintf(...) fprintf(stderr, __VA_ARGS__) @@ -93,11 +94,15 @@ static char **darwintrace_ignores = NULL; static size_t *darwintrace_ignore_lens = NULL; +/* store environment variables to preserve them on exec */ +static char *darwintrace_dylib_path; +static char *darwintrace_log_path; + /* check if str starts with one of the exceptions */ static inline bool darwintrace_except(const char *str) { size_t c = sizeof(darwintrace_exceptions)/sizeof(*darwintrace_exceptions); size_t i; - for (i = 0; i < c; i++) { + for (i = 0; i < c; i++) { if (strncmp(darwintrace_exceptions[i], str, strlen(darwintrace_exceptions[i])) == 0) { return true; } @@ -128,7 +133,7 @@ } static inline void darwintrace_setup() { - if (darwintrace_fd != -2) return; + if (darwintrace_fd != -2) return; char* path = getenv("DARWINTRACE_LOG"); if (path != NULL) { @@ -145,6 +150,7 @@ break; } } + darwintrace_log_path = strdup(path); errno = olderrno; } @@ -181,6 +187,30 @@ dprintf("unable to allocate memory for darwintrace_ignores"); } } + + /* find the install path of the darwintrace dylib for later use */ + path = getenv("DYLD_INSERT_LIBRARIES"); + if (path != NULL) { + char *ptr = strstr(path, "darwintrace.dylib"); + if (ptr) { + /* scan backward for : or start of string */ + while (ptr > path) { + if (*ptr == ':') { + ++ptr; + break; + } + --ptr; + } + + darwintrace_dylib_path = strdup(ptr); + + /* scan forward for : or end of string and terminate */ + ptr = strchr(darwintrace_dylib_path, ':'); + if (ptr) { + *ptr = 0; + } + } + } } /* darwintrace_setup must have been called already */ @@ -267,7 +297,7 @@ char realpath[MAXPATHLEN]; #if DARWINTRACE_LOG_FULL_PATH int usegetpath = 1; -#else +#else int usegetpath = 0; #endif @@ -334,11 +364,84 @@ } DARWINTRACE_INTERPOSE(darwintrace_readlink, readlink) +static inline int has_prefix(const char *s, const char *p) { + return (strncmp(s, p, strlen(p)) == 0); +} -int darwintrace_execve(const char* path, char* const argv[], char* const envp[]) { - int result; - - char* redirpath = darwintrace_redirect_path(path); +/* force the values of several environment variables */ +static char *const *darwintrace_make_environ(char *const envp[]) { + static const char *DARWINTRACE_IGNORE_ROOTS = "DARWINTRACE_IGNORE_ROOTS="; + static const char *DYLD_INSERT_LIBRARIES = "DYLD_INSERT_LIBRARIES="; + static const char *DARWINTRACE_LOG = "DARWINTRACE_LOG="; + static const char *DARWINTRACE_PLACEHOLDER = "__DARWINTRACE_PLACEHOLDER=UNUSED"; + + char **result = NULL; + char *libs = NULL; + int count = 0; + + /* count the environment variables */ + while (envp[count] != NULL) { + if (has_prefix(envp[count], DYLD_INSERT_LIBRARIES)) { + libs = envp[count] + strlen(DYLD_INSERT_LIBRARIES); + } + ++count; + } + + /* allocate size of envp with enough space for three more values and NULL */ + result = (char **)calloc(count + 4, sizeof(char *)); + if (result != NULL) { + int i = 0; + + if (darwintrace_log_path) { + asprintf(&result[i], "%s%s", DARWINTRACE_LOG, darwintrace_log_path); + } else { + result[i] = strdup(DARWINTRACE_PLACEHOLDER); + } + ++i; + + if (darwintrace_ignores) { + asprintf(&result[i], "%s%s", DARWINTRACE_IGNORE_ROOTS, "1"); + } else { + result[i] = strdup(DARWINTRACE_PLACEHOLDER); + } + ++i; + + if (darwintrace_dylib_path) { + int add_dylib = (strstr(libs, darwintrace_dylib_path) == NULL); + asprintf(&result[i], + "%s%s%s%s", + DYLD_INSERT_LIBRARIES, + add_dylib ? darwintrace_dylib_path : "", + (add_dylib && libs) ? ":" : "", + libs ? libs : ""); + } else { + result[i] = strdup(DARWINTRACE_PLACEHOLDER); + } + ++i; + + memcpy(&result[i], envp, count * sizeof(char *)); + + while (result[i] != NULL) { + if (has_prefix(result[i], DARWINTRACE_IGNORE_ROOTS) || + has_prefix(result[i], DYLD_INSERT_LIBRARIES) || + has_prefix(result[i], DARWINTRACE_LOG)) { + result[i] = (char *)DARWINTRACE_PLACEHOLDER; + } + ++i; + } + } + + return result; +} + +static void darwintrace_free_environ(char *const envp[]) { + free(envp[0]); + free(envp[1]); + free(envp[2]); + free((char*)envp); +} + +void darwintrace_log_exec(const char* redirpath, char* const argv[]) { darwintrace_setup(); if (darwintrace_fd >= 0) { struct stat sb; @@ -348,7 +451,7 @@ int fd; #if DARWINTRACE_LOG_FULL_PATH int usegetpath = 1; -#else +#else int usegetpath = 0; #endif @@ -382,7 +485,7 @@ darwintrace_cleanup_path(realpath); darwintrace_logpath(darwintrace_fd, NULL, "execve", realpath); } - + fd = open(redirpath, O_RDONLY, 0); if (fd != -1) { @@ -456,14 +559,40 @@ } } } - - result = execve(redirpath, argv, envp); - darwintrace_free_path(redirpath, path); - return result; } + +int darwintrace_execve(const char* path, char* const argv[], char* const envp[]) { + int result; + char* redirpath = darwintrace_redirect_path(path); + darwintrace_log_exec(redirpath, argv); + envp = darwintrace_make_environ(envp); + result = execve(redirpath, argv, envp); + darwintrace_free_environ(envp); + darwintrace_free_path(redirpath, path); + return result; +} DARWINTRACE_INTERPOSE(darwintrace_execve, execve) +extern int __posix_spawn(pid_t * __restrict, const char * __restrict, + void *, + char *const argv[ __restrict], char *const envp[ __restrict]); +int darwintrace_posix_spawn(pid_t * __restrict pid, + const char * __restrict path, + void * __restrict desc, + char *const argv[__restrict], + char *const envp[__restrict]) { + int result; + char* redirpath = darwintrace_redirect_path(path); + darwintrace_log_exec(redirpath, argv); + envp = darwintrace_make_environ(envp); + result = __posix_spawn(pid, redirpath, desc, argv, envp); + darwintrace_free_environ(envp); + darwintrace_free_path(redirpath, path); + return result; +} +DARWINTRACE_INTERPOSE(darwintrace_posix_spawn, __posix_spawn) + /* if darwintrace has been initialized, trap attempts to close our file descriptor

1 0

[967] branches/PR-10397485
by source_changes＠macosforge.org 07 Nov '11

07 Nov '11

Revision: 967 http://trac.macosforge.org/projects/darwinbuild/changeset/967 Author: kvv(a)apple.com Date: 2011-11-07 11:03:17 -0800 (Mon, 07 Nov 2011) Log Message: ----------- - darwintrace: preserve DYLD_INSERT_LIBRARIES and DARWINTRACE_LOG - darwintrace: interpose posix_spawn Modified Paths: -------------- branches/PR-10397485/CHANGES branches/PR-10397485/darwintrace/darwintrace.c Modified: branches/PR-10397485/CHANGES =================================================================== --- branches/PR-10397485/CHANGES 2011-11-07 07:09:47 UTC (rev 966) +++ branches/PR-10397485/CHANGES 2011-11-07 19:03:17 UTC (rev 967) @@ -1,7 +1,11 @@ Darwin Build Scripts Change History ----------------------------------- +Release 26 [6-Nov-2011] + - darwintrace: preserve DYLD_INSERT_LIBRARIES and DARWINTRACE_LOG + - darwintrace: interpose posix_spawn + Release 25 [4-Nov-2011] - - darwinup: support for rebuilding xpchelper cache + - darwinup: support for rebuilding xpchelper cache Release 24.1 [17-Jun-2011] - Build fixes for newer compilers Modified: branches/PR-10397485/darwintrace/darwintrace.c =================================================================== --- branches/PR-10397485/darwintrace/darwintrace.c 2011-11-07 07:09:47 UTC (rev 966) +++ branches/PR-10397485/darwintrace/darwintrace.c 2011-11-07 19:03:17 UTC (rev 967) @@ -1,5 +1,5 @@ /* - * Copyright (c) 2005 Apple Computer, Inc. All rights reserved. + * Copyright (c) 2005-2011 Apple Inc. All rights reserved. * * @APPLE_BSD_LICENSE_HEADER_START@ * @@ -32,6 +32,7 @@ #include <crt_externs.h> #include <fcntl.h> +#include <spawn.h> #include <stdarg.h> #include <stdio.h> #include <stdlib.h> @@ -47,7 +48,7 @@ #define DARWINTRACE_DEBUG_OUTPUT 0 #define DARWINTRACE_START_FD 101 #define DARWINTRACE_STOP_FD 200 -#define DARWINTRACE_BUFFER_SIZE 1024 +#define DARWINTRACE_BUFFER_SIZE 1024 #if DARWINTRACE_DEBUG_OUTPUT #define dprintf(...) fprintf(stderr, __VA_ARGS__) @@ -93,11 +94,15 @@ static char **darwintrace_ignores = NULL; static size_t *darwintrace_ignore_lens = NULL; +/* store environment variables to preserve them on exec */ +static char *darwintrace_dylib_path; +static char *darwintrace_log_path; + /* check if str starts with one of the exceptions */ static inline bool darwintrace_except(const char *str) { size_t c = sizeof(darwintrace_exceptions)/sizeof(*darwintrace_exceptions); size_t i; - for (i = 0; i < c; i++) { + for (i = 0; i < c; i++) { if (strncmp(darwintrace_exceptions[i], str, strlen(darwintrace_exceptions[i])) == 0) { return true; } @@ -128,7 +133,7 @@ } static inline void darwintrace_setup() { - if (darwintrace_fd != -2) return; + if (darwintrace_fd != -2) return; char* path = getenv("DARWINTRACE_LOG"); if (path != NULL) { @@ -145,6 +150,7 @@ break; } } + darwintrace_log_path = strdup(path); errno = olderrno; } @@ -181,6 +187,30 @@ dprintf("unable to allocate memory for darwintrace_ignores"); } } + + /* find the install path of the darwintrace dylib for later use */ + path = getenv("DYLD_INSERT_LIBRARIES"); + if (path != NULL) { + char *ptr = strstr(path, "darwintrace.dylib"); + if (ptr) { + /* scan backward for : or start of string */ + while (ptr > path) { + if (*ptr == ':') { + ++ptr; + break; + } + --ptr; + } + + darwintrace_dylib_path = strdup(ptr); + + /* scan forward for : or end of string and terminate */ + ptr = strchr(darwintrace_dylib_path, ':'); + if (ptr) { + *ptr = 0; + } + } + } } /* darwintrace_setup must have been called already */ @@ -267,7 +297,7 @@ char realpath[MAXPATHLEN]; #if DARWINTRACE_LOG_FULL_PATH int usegetpath = 1; -#else +#else int usegetpath = 0; #endif @@ -334,11 +364,84 @@ } DARWINTRACE_INTERPOSE(darwintrace_readlink, readlink) +static inline int has_prefix(const char *s, const char *p) { + return (strncmp(s, p, strlen(p)) == 0); +} -int darwintrace_execve(const char* path, char* const argv[], char* const envp[]) { - int result; - - char* redirpath = darwintrace_redirect_path(path); +/* force the values of several environment variables */ +static char *const *darwintrace_make_environ(char *const envp[]) { + static const char *DARWINTRACE_IGNORE_ROOTS = "DARWINTRACE_IGNORE_ROOTS="; + static const char *DYLD_INSERT_LIBRARIES = "DYLD_INSERT_LIBRARIES="; + static const char *DARWINTRACE_LOG = "DARWINTRACE_LOG="; + static const char *DARWINTRACE_PLACEHOLDER = "__DARWINTRACE_PLACEHOLDER=UNUSED"; + + char **result = NULL; + char *libs = NULL; + int count = 0; + + /* count the environment variables */ + while (envp[count] != NULL) { + if (has_prefix(envp[count], DYLD_INSERT_LIBRARIES)) { + libs = envp[count] + strlen(DYLD_INSERT_LIBRARIES); + } + ++count; + } + + /* allocate size of envp with enough space for three more values and NULL */ + result = (char **)calloc(count + 4, sizeof(char *)); + if (result != NULL) { + int i = 0; + + if (darwintrace_log_path) { + asprintf(&result[i], "%s%s", DARWINTRACE_LOG, darwintrace_log_path); + } else { + result[i] = strdup(DARWINTRACE_PLACEHOLDER); + } + ++i; + + if (darwintrace_ignores) { + asprintf(&result[i], "%s%s", DARWINTRACE_IGNORE_ROOTS, "1"); + } else { + result[i] = strdup(DARWINTRACE_PLACEHOLDER); + } + ++i; + + if (darwintrace_dylib_path) { + int add_dylib = (strstr(libs, darwintrace_dylib_path) == NULL); + asprintf(&result[i], + "%s%s%s%s", + DYLD_INSERT_LIBRARIES, + add_dylib ? darwintrace_dylib_path : "", + (add_dylib && libs) ? ":" : "", + libs ? libs : ""); + } else { + result[i] = strdup(DARWINTRACE_PLACEHOLDER); + } + ++i; + + memcpy(&result[i], envp, count * sizeof(char *)); + + while (result[i] != NULL) { + if (has_prefix(result[i], DARWINTRACE_IGNORE_ROOTS) || + has_prefix(result[i], DYLD_INSERT_LIBRARIES) || + has_prefix(result[i], DARWINTRACE_LOG)) { + result[i] = (char *)DARWINTRACE_PLACEHOLDER; + } + ++i; + } + } + + return result; +} + +static void darwintrace_free_environ(char *const envp[]) { + free(envp[0]); + free(envp[1]); + free(envp[2]); + free((char*)envp); +} + +void darwintrace_log_exec(const char* redirpath, char* const argv[]) { darwintrace_setup(); if (darwintrace_fd >= 0) { struct stat sb; @@ -348,7 +451,7 @@ int fd; #if DARWINTRACE_LOG_FULL_PATH int usegetpath = 1; -#else +#else int usegetpath = 0; #endif @@ -382,7 +485,7 @@ darwintrace_cleanup_path(realpath); darwintrace_logpath(darwintrace_fd, NULL, "execve", realpath); } - + fd = open(redirpath, O_RDONLY, 0); if (fd != -1) { @@ -456,14 +559,40 @@ } } } - - result = execve(redirpath, argv, envp); - darwintrace_free_path(redirpath, path); - return result; } + +int darwintrace_execve(const char* path, char* const argv[], char* const envp[]) { + int result; + char* redirpath = darwintrace_redirect_path(path); + darwintrace_log_exec(redirpath, argv); + envp = darwintrace_make_environ(envp); + result = execve(redirpath, argv, envp); + darwintrace_free_environ(envp); + darwintrace_free_path(redirpath, path); + return result; +} DARWINTRACE_INTERPOSE(darwintrace_execve, execve) +extern int __posix_spawn(pid_t * __restrict, const char * __restrict, + void *, + char *const argv[ __restrict], char *const envp[ __restrict]); +int darwintrace_posix_spawn(pid_t * __restrict pid, + const char * __restrict path, + void * __restrict desc, + char *const argv[__restrict], + char *const envp[__restrict]) { + int result; + char* redirpath = darwintrace_redirect_path(path); + darwintrace_log_exec(redirpath, argv); + envp = darwintrace_make_environ(envp); + result = __posix_spawn(pid, redirpath, desc, argv, envp); + darwintrace_free_environ(envp); + darwintrace_free_path(redirpath, path); + return result; +} +DARWINTRACE_INTERPOSE(darwintrace_posix_spawn, __posix_spawn) + /* if darwintrace has been initialized, trap attempts to close our file descriptor

1 0

[966] branches/PR-10397485/
by source_changes＠macosforge.org 07 Nov '11

07 Nov '11

Revision: 966 http://trac.macosforge.org/projects/darwinbuild/changeset/966 Author: kvv(a)apple.com Date: 2011-11-06 23:09:47 -0800 (Sun, 06 Nov 2011) Log Message: ----------- "Branch for PR-10397485 from http://svn.macosforge.org/repository/darwinbuild/trunk" Added Paths: ----------- branches/PR-10397485/ Property changes on: branches/PR-10397485 ___________________________________________________________________ Added: svn:mergeinfo + /branches/PR-10363375:959-960 /branches/PR-4841388:399-419 /branches/PR-6358021:442-443 /branches/PR-6392966:423-427 /branches/PR-6398060:433-434 /branches/PR-6493844:460-461 /branches/PR-6497694:466-468,471 /branches/PR-6634286:632-650 /branches/PR-6688645:479-490 /branches/PR-6722857:495-499 /branches/PR-6729491:655-664 /branches/PR-6973110:804-813 /branches/PR-7250612:635-650 /branches/PR-7341154:682-694 /branches/PR-7431723:660-664 /branches/PR-7461534:650-664 /branches/PR-7482850:670-671 /branches/PR-7489777:676-731 /branches/PR-7529688:692-694 /branches/PR-7593824:739-772 /branches/PR-7598640:703-731 /branches/PR-7748469:777-785 /branches/PR-7765119:790-791 /branches/PR-7798586:796-799 /branches/PR-7872907:830-840 /branches/PR-7935095:819-821 /branches/PR-8116613:849 /branches/PR-8279204:854-862 /branches/PR-8416637:870-880 /branches/PR-8486662:885-889 /branches/PR-8488185:894-898 /branches/PR-8604911:903-905 /branches/PR-8817822:917-933 /branches/PR-8908468:912

1 0

[965] branches/PR-10397485/
by source_changes＠macosforge.org 07 Nov '11

07 Nov '11

Revision: 965 http://trac.macosforge.org/projects/darwinbuild/changeset/965 Author: kvv(a)apple.com Date: 2011-11-06 23:09:00 -0800 (Sun, 06 Nov 2011) Log Message: ----------- Removed Paths: ------------- branches/PR-10397485/

1 0

[964] branches/PR-10397485/
by source_changes＠macosforge.org 07 Nov '11

07 Nov '11

Revision: 964 http://trac.macosforge.org/projects/darwinbuild/changeset/964 Author: kvv(a)apple.com Date: 2011-11-06 23:07:25 -0800 (Sun, 06 Nov 2011) Log Message: ----------- "Branch for PR-10397485 from http://svn.macosforge.org/repository/darwinbuild/trunk/darwintrace" Added Paths: ----------- branches/PR-10397485/ Property changes on: branches/PR-10397485 ___________________________________________________________________ Added: svn:mergeinfo + /branches/PR-10363375/darwintrace:959-960 /branches/PR-4841388/darwintrace:399-419 /branches/PR-6358021/darwintrace:442-443 /branches/PR-6392966/darwintrace:423-427 /branches/PR-6398060/darwintrace:433-434 /branches/PR-6493844/darwintrace:460-461 /branches/PR-6497694/darwintrace:466-468,471 /branches/PR-6634286/darwintrace:632-650 /branches/PR-6688645/darwintrace:479-490 /branches/PR-6722857/darwintrace:495-499 /branches/PR-6729491/darwintrace:655-664 /branches/PR-6973110/darwintrace:804-813 /branches/PR-7250612/darwintrace:635-650 /branches/PR-7341154/darwintrace:682-694 /branches/PR-7431723/darwintrace:660-664 /branches/PR-7461534/darwintrace:650-664 /branches/PR-7482850/darwintrace:670-671 /branches/PR-7489777/darwintrace:676-731 /branches/PR-7529688/darwintrace:692-694 /branches/PR-7593824/darwintrace:739-772 /branches/PR-7598640/darwintrace:703-731 /branches/PR-7748469/darwintrace:777-785 /branches/PR-7765119/darwintrace:790-791 /branches/PR-7798586/darwintrace:796-799 /branches/PR-7872907/darwintrace:830-840 /branches/PR-7935095/darwintrace:819-821 /branches/PR-8116613/darwintrace:849 /branches/PR-8279204/darwintrace:854-862 /branches/PR-8416637/darwintrace:870-880 /branches/PR-8486662/darwintrace:885-889 /branches/PR-8488185/darwintrace:894-898 /branches/PR-8604911/darwintrace:903-905 /branches/PR-8817822/darwintrace:917-933 /branches/PR-8908468/darwintrace:912

1 0

[963] tags/darwinbuild-25/
by source_changes＠macosforge.org 04 Nov '11

04 Nov '11

Revision: 963 http://trac.macosforge.org/projects/darwinbuild/changeset/963 Author: mww(a)apple.com Date: 2011-11-04 16:09:58 -0700 (Fri, 04 Nov 2011) Log Message: ----------- Tagging darwinbuild-25. Added Paths: ----------- tags/darwinbuild-25/ Property changes on: tags/darwinbuild-25 ___________________________________________________________________ Added: svn:mergeinfo + /branches/PR-10363375:959-960 /branches/PR-4841388:399-419 /branches/PR-6358021:442-443 /branches/PR-6392966:423-427 /branches/PR-6398060:433-434 /branches/PR-6493844:460-461 /branches/PR-6497694:466-468,471 /branches/PR-6634286:632-650 /branches/PR-6688645:479-490 /branches/PR-6722857:495-499 /branches/PR-6729491:655-664 /branches/PR-6973110:804-813 /branches/PR-7250612:635-650 /branches/PR-7341154:682-694 /branches/PR-7431723:660-664 /branches/PR-7461534:650-664 /branches/PR-7482850:670-671 /branches/PR-7489777:676-731 /branches/PR-7529688:692-694 /branches/PR-7593824:739-772 /branches/PR-7598640:703-731 /branches/PR-7748469:777-785 /branches/PR-7765119:790-791 /branches/PR-7798586:796-799 /branches/PR-7872907:830-840 /branches/PR-7935095:819-821 /branches/PR-8116613:849 /branches/PR-8279204:854-862 /branches/PR-8416637:870-880 /branches/PR-8486662:885-889 /branches/PR-8488185:894-898 /branches/PR-8604911:903-905 /branches/PR-8817822:917-933 /branches/PR-8908468:912

1 0

[962] trunk
by source_changes＠macosforge.org 04 Nov '11

04 Nov '11

Revision: 962 http://trac.macosforge.org/projects/darwinbuild/changeset/962 Author: mww(a)apple.com Date: 2011-11-04 16:08:50 -0700 (Fri, 04 Nov 2011) Log Message: ----------- Version bump for 25. Modified Paths: -------------- trunk/CHANGES trunk/darwinup/main.cpp Modified: trunk/CHANGES =================================================================== --- trunk/CHANGES 2011-11-04 22:59:53 UTC (rev 961) +++ trunk/CHANGES 2011-11-04 23:08:50 UTC (rev 962) @@ -1,5 +1,8 @@ Darwin Build Scripts Change History ----------------------------------- +Release 25 [4-Nov-2011] + - darwinup: support for rebuilding xpchelper cache + Release 24.1 [17-Jun-2011] - Build fixes for newer compilers Modified: trunk/darwinup/main.cpp =================================================================== --- trunk/darwinup/main.cpp 2011-11-04 22:59:53 UTC (rev 961) +++ trunk/darwinup/main.cpp 2011-11-04 23:08:50 UTC (rev 962) @@ -47,7 +47,7 @@ void usage(char* progname) { fprintf(stderr, "usage: %s [-v] [-p DIR] [command] [args] \n", progname); - fprintf(stderr, "version: 24.1 \n"); + fprintf(stderr, "version: 25 \n"); fprintf(stderr, " \n"); fprintf(stderr, "options: \n"); #if __MAC_OS_X_VERSION_MIN_REQUIRED >= 1060

1 0

[961] trunk
by source_changes＠macosforge.org 04 Nov '11

04 Nov '11

Revision: 961 http://trac.macosforge.org/projects/darwinbuild/changeset/961 Author: mww(a)apple.com Date: 2011-11-04 15:59:53 -0700 (Fri, 04 Nov 2011) Log Message: ----------- <rdar://problem/10363375> darwinup should rebuild xpc service cache where applicable Modified Paths: -------------- trunk/darwinup/Depot.cpp trunk/darwinup/Depot.h trunk/darwinup/Utils.cpp trunk/darwinup/Utils.h trunk/darwinup/main.cpp Property Changed: ---------------- trunk/ Property changes on: trunk ___________________________________________________________________ Modified: svn:mergeinfo - /branches/PR-4841388:399-419 /branches/PR-6358021:442-443 /branches/PR-6392966:423-427 /branches/PR-6398060:433-434 /branches/PR-6493844:460-461 /branches/PR-6497694:466-468,471 /branches/PR-6634286:632-650 /branches/PR-6688645:479-490 /branches/PR-6722857:495-499 /branches/PR-6729491:655-664 /branches/PR-6973110:804-813 /branches/PR-7250612:635-650 /branches/PR-7341154:682-694 /branches/PR-7431723:660-664 /branches/PR-7461534:650-664 /branches/PR-7482850:670-671 /branches/PR-7489777:676-731 /branches/PR-7529688:692-694 /branches/PR-7593824:739-772 /branches/PR-7598640:703-731 /branches/PR-7748469:777-785 /branches/PR-7765119:790-791 /branches/PR-7798586:796-799 /branches/PR-7872907:830-840 /branches/PR-7935095:819-821 /branches/PR-8116613:849 /branches/PR-8279204:854-862 /branches/PR-8416637:870-880 /branches/PR-8486662:885-889 /branches/PR-8488185:894-898 /branches/PR-8604911:903-905 /branches/PR-8817822:917-933 /branches/PR-8908468:912 + /branches/PR-10363375:959-960 /branches/PR-4841388:399-419 /branches/PR-6358021:442-443 /branches/PR-6392966:423-427 /branches/PR-6398060:433-434 /branches/PR-6493844:460-461 /branches/PR-6497694:466-468,471 /branches/PR-6634286:632-650 /branches/PR-6688645:479-490 /branches/PR-6722857:495-499 /branches/PR-6729491:655-664 /branches/PR-6973110:804-813 /branches/PR-7250612:635-650 /branches/PR-7341154:682-694 /branches/PR-7431723:660-664 /branches/PR-7461534:650-664 /branches/PR-7482850:670-671 /branches/PR-7489777:676-731 /branches/PR-7529688:692-694 /branches/PR-7593824:739-772 /branches/PR-7598640:703-731 /branches/PR-7748469:777-785 /branches/PR-7765119:790-791 /branches/PR-7798586:796-799 /branches/PR-7872907:830-840 /branches/PR-7935095:819-821 /branches/PR-8116613:849 /branches/PR-8279204:854-862 /branches/PR-8416637:870-880 /branches/PR-8486662:885-889 /branches/PR-8488185:894-898 /branches/PR-8604911:903-905 /branches/PR-8817822:917-933 /branches/PR-8908468:912 Modified: trunk/darwinup/Depot.cpp =================================================================== --- trunk/darwinup/Depot.cpp 2011-11-04 19:00:16 UTC (rev 960) +++ trunk/darwinup/Depot.cpp 2011-11-04 22:59:53 UTC (rev 961) @@ -62,6 +62,7 @@ m_depot_mode = 0750; m_is_dirty = false; m_modified_extensions = false; + m_modified_xpc_services = false; } Depot::Depot(const char* prefix) { @@ -71,6 +72,7 @@ m_build = NULL; m_is_dirty = false; m_modified_extensions = false; + m_modified_xpc_services = false; asprintf(&m_prefix, "%s", prefix); join_path(&m_depot_path, m_prefix, "/.DarwinDepot"); @@ -98,6 +100,7 @@ const char* Depot::prefix() { return m_prefix; } bool Depot::is_dirty() { return m_is_dirty; } bool Depot::has_modified_extensions() { return m_modified_extensions; } +bool Depot::has_modified_xpc_services(){ return m_modified_xpc_services; } int Depot::connect() { m_db = new DarwinupDatabase(m_database_path); @@ -542,6 +545,17 @@ this->m_modified_extensions = true; } + if (!this->m_modified_xpc_services && + (strstr(file->path(), ".xpc/") != NULL)) { + + bool modified = (has_suffix(file->path(), "Info.plist") || + has_suffix(file->path(), "framework.sb")); + + if (modified) { + IF_DEBUG("[analyze] xpc service detected\n"); + this->m_modified_xpc_services = true; + } + } } // if file == actual, but actual != preceding, then an external Modified: trunk/darwinup/Depot.h =================================================================== --- trunk/darwinup/Depot.h 2011-11-04 19:00:16 UTC (rev 960) +++ trunk/darwinup/Depot.h 2011-11-04 22:59:53 UTC (rev 961) @@ -133,6 +133,7 @@ bool is_dirty(); bool has_modified_extensions(); + bool has_modified_xpc_services(); protected: @@ -183,6 +184,7 @@ int m_is_locked; bool m_is_dirty; // track if we need to update dyld cache bool m_modified_extensions; // track if we need to touch /S/L/E + bool m_modified_xpc_services; // track if we need to run xpchelper }; Modified: trunk/darwinup/Utils.cpp =================================================================== --- trunk/darwinup/Utils.cpp 2011-11-04 19:00:16 UTC (rev 960) +++ trunk/darwinup/Utils.cpp 2011-11-04 22:59:53 UTC (rev 961) @@ -361,6 +361,42 @@ return res; } +int update_xpc_services_cache(const char* path) { + extern uint32_t verbosity; + int res; + char* base; + res = find_base_system_path(&base, path); + if (res) return res; + + if (verbosity) { + fprintf(stdout, "Updating xpc services cache for %s ...", base); + fflush(stdout); + } + + char* toolpath; + join_path(&toolpath, base, "/usr/libexec/xpchelper"); + + struct stat sb; + res = stat(toolpath, &sb); + if (res) { + return 1; + } + + const char* args[] = { + toolpath, + "--rebuild-cache", + "--root", base, + NULL + }; + res = exec_with_args(args); + + if (verbosity) fprintf(stdout, "Done updating xpc cache\n"); + + free(toolpath); + free(base); + return res; +} + #if __MAC_OS_X_VERSION_MIN_REQUIRED >= 1060 int build_number_for_path(char** build, const char* path) { int res = 0; Modified: trunk/darwinup/Utils.h =================================================================== --- trunk/darwinup/Utils.h 2011-11-04 19:00:16 UTC (rev 960) +++ trunk/darwinup/Utils.h 2011-11-04 22:59:53 UTC (rev 961) @@ -80,6 +80,7 @@ int find_base_system_path(char** output, const char* path); int update_dyld_shared_cache(const char* path); +int update_xpc_services_cache(const char* path); #if __MAC_OS_X_VERSION_MIN_REQUIRED >= 1060 int build_number_for_path(char** build, const char* path); Modified: trunk/darwinup/main.cpp =================================================================== --- trunk/darwinup/main.cpp 2011-11-04 19:00:16 UTC (rev 960) +++ trunk/darwinup/main.cpp 2011-11-04 22:59:53 UTC (rev 961) @@ -280,6 +280,15 @@ } free(sle_path); } +#endif +#if __MAC_OS_X_VERSION_MIN_REQUIRED >= 1070 + if (!disable_automation && depot->has_modified_xpc_services() && res == 0) { + res = update_xpc_services_cache(path); + if (res) fprintf(stderr, "Warning: could not update xpc services cache.\n"); + res = 0; + } +#endif +#if __MAC_OS_X_VERSION_MIN_REQUIRED >= 1060 if (restart && res == 0) { res = tell_finder_to_restart(); if (res) fprintf(stderr, "Warning: tried to tell Finder to restart"

1 0

[960] branches/PR-10363375/darwinup/Depot.h
by source_changes＠macosforge.org 04 Nov '11

04 Nov '11

Revision: 960 http://trac.macosforge.org/projects/darwinbuild/changeset/960 Author: mww(a)apple.com Date: 2011-11-04 12:00:16 -0700 (Fri, 04 Nov 2011) Log Message: ----------- Whitespace bogosity. Modified Paths: -------------- branches/PR-10363375/darwinup/Depot.h Modified: branches/PR-10363375/darwinup/Depot.h =================================================================== --- branches/PR-10363375/darwinup/Depot.h 2011-11-04 19:00:14 UTC (rev 959) +++ branches/PR-10363375/darwinup/Depot.h 2011-11-04 19:00:16 UTC (rev 960) @@ -133,7 +133,7 @@ bool is_dirty(); bool has_modified_extensions(); - bool has_modified_xpc_services(); + bool has_modified_xpc_services(); protected:

1 0