Bug 49051 - build-webkit spams system.log by repeatedly checking for PBXProductDirectory in com.apple.Xcode
Summary: build-webkit spams system.log by repeatedly checking for PBXProductDirectory ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-04 20:51 PDT by Eric Seidel (no email)
Modified: 2010-11-05 02:14 PDT (History)
8 users (show)

See Also:


Attachments
Patch (4.15 KB, patch)
2010-11-04 22:30 PDT, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2010-11-04 20:51:46 PDT
build-webkit spams system.log by repeatedly checking for PBXProductDirectory in com.apple.Xcode

When it's missing, you get this spam:

Nov  3 00:32:55 eseidel defaults[38804]: \nThe domain/default pair of (com.apple.Xcode, PBXProductDirectory) does not exist
Nov  3 00:32:55 eseidel defaults[38814]: \nThe domain/default pair of (com.apple.Xcode, PBXProductDirectory) does not exist
Nov  3 00:32:55 eseidel defaults[38824]: \nThe domain/default pair of (com.apple.Xcode, PBXProductDirectory) does not exist
Nov  3 00:32:56 eseidel defaults[38834]: \nThe domain/default pair of (com.apple.Xcode, PBXProductDirectory) does not exist
Nov  3 00:32:56 eseidel defaults[38844]: \nThe domain/default pair of (com.apple.Xcode, PBXProductDirectory) does not exist
Nov  3 00:32:56 eseidel defaults[38854]: \nThe domain/default pair of (com.apple.Xcode, PBXProductDirectory) does not exist

I have thousands of these in my system.log.
Comment 1 Eric Seidel (no email) 2010-11-04 20:54:26 PDT
I've also filed radar 8634566 about defaults logging to system.log in the first place.
Comment 2 Eric Seidel (no email) 2010-11-04 20:59:10 PDT
Actually, it's probably coming from the python code using webkit-build-directory.
Comment 3 Eric Seidel (no email) 2010-11-04 22:30:10 PDT
Created attachment 73036 [details]
Patch
Comment 4 Eric Seidel (no email) 2010-11-04 22:35:12 PDT
This will conflict with one of dpranke's pending patches, so I should land this by hand.  (Unless he'd like me to go first in which case he should feel free to set cq+ or cq? on it.)
Comment 5 Eric Seidel (no email) 2010-11-05 01:54:17 PDT
Comment on attachment 73036 [details]
Patch

Eh.  On second thought, it's actually easier to just land this and let Dirk merge in the morning.
Comment 6 WebKit Commit Bot 2010-11-05 02:14:33 PDT
Comment on attachment 73036 [details]
Patch

Clearing flags on attachment: 73036

Committed r71398: <http://trac.webkit.org/changeset/71398>
Comment 7 WebKit Commit Bot 2010-11-05 02:14:39 PDT
All reviewed patches have been landed.  Closing bug.