We're not saving EULA acceptance, just putting it somewhere that xcodebuild can find it when we're running as the macports user.
If a different user runs port, their own plist is copied, so if they haven't accepted the EULA they can't still use xcode based on a previous user's acceptance. (Whereas that's exactly what can happen if one user runs 'sudo xcodebuild -license' and then another user runs port.) Can we even tell whether xcodebuild or xcrun is failing because it wants EULA acceptance as opposed to some other reason? - Josh On 2012-2-24 20:56 , Jeremy Huddleston wrote: > Wait a minute... I'm jumping in a bit at the end here and missed a bit of > this progress, so apologies if I'm a bit off based with my limited context... > are you guys saving EULA acceptance for the user by copying the plist? > IANAL, but that seems far inside the not-quite-legal zone. > > Macports should do no copying of the plist or otherwise "mess" with the > license setting for the user. If it encounters a problem, it should tell the > user to run 'sudo xcodebuild -license' ... not copy preferences around all > over the place. Again, IANAL, but that sounds like the safest bet to me. > > --Jeremy > > On Feb 21, 2012, at 12:08 PM, Dan Ports <[email protected]> wrote: > >> On Wed, Feb 22, 2012 at 02:39:42AM +1100, Joshua Root wrote: >>> And why did Dan set the ownership in the first place? I can see that >>> making sense in the per-port dirs but not really in the global one. >> >> Without it, the plist had 0600 permissions and it was owned by root, so >> xcodebuild couldn't access it once we dropped privileges. Changing the >> permissions works to. >> >> Speaking of which, do we actually need to create the plist in both the >> global and per-port home directories? It looks like only the global one >> is actually used, regardless of $HOME. (This surprised me, I figured >> it'd be the other way around.) >> >> Dan _______________________________________________ macports-dev mailing list [email protected] http://lists.macosforge.org/mailman/listinfo.cgi/macports-dev
