Opened 12 years ago
Closed 11 years ago
#39062 closed defect (fixed)
TeXShop3 build failure (OS 10.7.5, Xcode 4.6.2)
| Reported by: | gallafent | Owned by: | jyrkiwahlstedt |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 2.1.3 |
| Keywords: | Cc: | anddam (Andrea D'Amore), mathsmac@…, cooljeanius (Eric Gallager) | |
| Port: | TeXShop3 |
Description (last modified by mf2k (Frank Schima))
Upon modifying the Portfile so that the checksum matches the downloaded source (see ticket:39061 ), the build commences but fails, since the type NSSharingServicePicker is not present. Build log attached. This is probably related to ticket:38981 .
Is there a single variable which may be used to specify the SDK used for building? Searching within http://guide.macports.org/ doesn't mention one, with SDK being specified in various places (universal C++ and C flags, and so on).
Attachments (3)
Change History (19)
Changed 12 years ago by gallafent
comment:1 follow-up: 3 Changed 12 years ago by gallafent
comment:2 Changed 12 years ago by mf2k (Frank Schima)
| Cc: | jwa@… openmaintainer@… removed |
|---|---|
| Description: | modified (diff) |
| Owner: | changed from macports-tickets@… to jwa@… |
| Port: | TeXShop3 added; texshop3 removed |
comment:3 Changed 12 years ago by larryv (Lawrence Velázquez)
Replying to william@…:
Incidentally, I tried setting the SDK in the environment:
export SDKROOT=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.8.sdk… but this made no difference.
MacPorts ignores the user’s environment.
comment:4 follow-up: 6 Changed 12 years ago by larryv (Lawrence Velázquez)
Have any of you reported this upstream? TeXShop 3 claims that it supports Lion.
comment:5 Changed 12 years ago by jyrkiwahlstedt
There's a new version of the package now with a correction for a another SDK-related issue, can you try again with the new one? But looking at source code, this is in TSDocument.m still, so could stop build:(
comment:6 Changed 12 years ago by anddam (Andrea D'Amore)
Replying to jwa@…:
There's a new version of the package now with a correction for a another SDK-related issue, can you try again with the new one? But looking at source code, this is in TSDocument.m still, so could stop build:(
The updated tarball fixed the specific error but the port build is still broken. I got new, different errors while building, the same the OP did.
Replying to larryv@…:
Have any of you reported this upstream? TeXShop 3 claims that it supports Lion.
Richard Koch replied:
TeXShop must be compiled on Mountain Lion because it supports sharing. […] TeXShop RUNS on Lion.
We should just disable the port building on 10.7 until it is confirmed to be fixed.
comment:8 Changed 11 years ago by gallafent
It seems odd to me that a specific OS version should be required to build! Surely that's the whole point of the SDK system, to isolate the build environment from the currently running OS. I'll have a look to see if this can be fixed by forcing the 10.8 SDK. I note that the build line includes “SDKROOT=""” … so perhaps if that can be set to the 10.8 SDK …
comment:9 follow-up: 10 Changed 11 years ago by anddam (Andrea D'Amore)
I was already passing -sdk macosx10.8 to build command but I just noticed the issue happens in destroot phase not in build so it was just a matter of passing the argument to both phases. I'm attaching a patch.
Changed 11 years ago by anddam (Andrea D'Amore)
| Attachment: | port-texshop3.diff added |
|---|
comment:10 Changed 11 years ago by gallafent
Replying to and.damore@…:
I was already passing -sdk macosx10.8 to build command but I just noticed the issue happens in destroot phase not in build so it was just a matter of passing the argument to both phases. I'm attaching a patch.
I was just testing with the commandline “sudo port install texshop3 configure.sdkroot=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.8.sdk”, and can confirm that this also worked fine :)
(so the statement “TeXShop must be compiled on Mountain Lion because it supports sharing.” is false, thankfully :)
comment:12 follow-up: 13 Changed 11 years ago by gallafent
Still broken in 3.18, as of today! Skimming the log indicates that this is still the same problem, requiring the use of the 10.8 SDK.
comment:13 follow-up: 14 Changed 11 years ago by anddam (Andrea D'Amore)
Replying to william@…:
Still broken in 3.18, as of today! Skimming the log indicates that this is still the same problem, requiring the use of the 10.8 SDK.
The patch has never been committed. Did you try it?
@jwa: any reason for not committing it?
comment:14 Changed 11 years ago by jyrkiwahlstedt
Replying to and.damore@…:
Replying to william@…:
Still broken in 3.18, as of today! Skimming the log indicates that this is still the same problem, requiring the use of the 10.8 SDK.
The patch has never been committed. Did you try it?
@jwa: any reason for not committing it?
None whatsoever, just had all kinds of things going on, now committed in r106673, thanks, please check and confirm, whether it works. I did not bump revision, as the existing installations don't have to be rebuilt, only those are affected that didn't build…
comment:16 Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
The Lion and Mountain Lion buildbots succeeded building the port. The Snow Leopard one of course still fails with the message "This version is for 10.7, Lion, and later only".
Incidentally, I tried setting the SDK in the environment:
… but this made no difference.