Opened 6 years ago

Closed 6 years ago

#57253 closed defect (fixed)

TeXShop4 fails upgrade 10.13.6

Reported by: basmac Owned by: Schamschula (Marius Schamschula)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: TeXShop4

Description

TeXShop4 debug attached

Attachments (1)

log (168.2 KB) - added by basmac 6 years ago.
debug log

Download all attachments as: .zip

Change History (7)

Changed 6 years ago by basmac

Attachment: log added

debug log

comment:1 Changed 6 years ago by jmroot (Joshua Root)

Keywords: TeXShop4 removed
Owner: set to Schamschula
Status: newassigned

comment:2 Changed 6 years ago by Schamschula (Marius Schamschula)

Looks like the error is

/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_macports_release_tarballs_ports_aqua_TeXShop4/TeXShop4/work/texshopsource412/Sources/TSTextView.m:1869:62: error: use of undeclared identifier 'NSAppearanceNameDarkAqua'; did you mean 'NSAppearanceNameAqua'?
349	    if ((atLeastMojave) && (self.effectiveAppearance.name == NSAppearanceNameDarkAqua))
350	                                                             ^~~~~~~~~~~~~~~~~~~~~~~~
351	                                                             NSAppearanceNameAqua

I'm not sure what is happening here. I built TeXShop 4.12 on my machine running 10.13.6 before committing. Note: I have Xcode 10 installed.

comment:3 Changed 6 years ago by basmac

We have Xcode 9.4 and CLT for that version.

comment:4 Changed 6 years ago by Schamschula (Marius Schamschula)

It is clearly looking for Mojave's Dark Mode. I will contact the developer.

comment:5 Changed 6 years ago by basmac

It does build without error after updating to Xcode and CLT 10.0

comment:6 Changed 6 years ago by Schamschula (Marius Schamschula)

Resolution: fixed
Status: assignedclosed

In 3cbcf3c0ac09ef99c0bc0797345c6015045a7b1f/macports-ports (master):

TeXShop4: fix build issues for Xcode < 10 and macOS < 10.13

Closes: #56857
Closes: #57253

Note: See TracTickets for help on using tickets.