[MacPorts] #49446: TeXShop3 @3.57: identical patch blocks for darwin 12, 13, 14
MacPorts
noreply at macports.org
Sun Oct 25 06:20:45 PDT 2015
#49446: TeXShop3 @3.57: identical patch blocks for darwin 12, 13, 14
--------------------------+----------------------
Reporter: ryandesign@… | Owner: jwa@…
Type: enhancement | Status: new
Priority: Normal | Milestone:
Component: ports | Version: 2.3.4
Keywords: | Port: TeXShop3
--------------------------+----------------------
I noticed that the TeXShop3 port has identical patch blocks for darwin 12,
13 and 14:
{{{
platform darwin 12 {
patchfiles-append patch-MyPDFKitView.m.diff patch-
TSAppDelegate.m.diff \
patch-MyPDFKitView-Magnification.m.diff
}
platform darwin 13 {
patchfiles-append patch-MyPDFKitView.m.diff patch-
TSAppDelegate.m.diff \
patch-MyPDFKitView-Magnification.m.diff
}
platform darwin 14 {
patchfiles-append patch-TSAppDelegate.m.diff \
patch-MyPDFKitView-Magnification.m.diff
}
}}}
Why not make these a single block?
{{{
platform darwin {
if {${os.major} >= 12 && ${os.major} <= 14} {
patchfiles-append patch-MyPDFKitView.m.diff \
patch-MyPDFKitView-Magnification.m.diff \
patch-TSAppDelegate.m.diff
}
}
}}}
Even better would be if the patches were written in such a way that it
will work correctly for any version of OS, and applied unconditionally.
That's what would be required for the patch to be accepted by the upstream
developers.
--
Ticket URL: <https://trac.macports.org/ticket/49446>
MacPorts <https://www.macports.org/>
Ports system for OS X
More information about the macports-tickets
mailing list