<div dir="ltr"><div dir="ltr"><div dir="ltr" class="gmail_attr">On Sat, Dec 11, 2021 at 5:46 PM Ryan Schmidt <<a href="mailto:ryandesign@macports.org">ryandesign@macports.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">On Dec 11, 2021, at 16:41, Jason Liu wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><br>and be done with it, instead of needing to add one each and every time I have a conditional involving `${os.major}`?</blockquote><br>I wouldn't expect you to need to write more than a couple such conditionals in a portfile so I wouldn't expect it to be that inconvenient.</blockquote><div><br></div><div>Actually, I find myself needing to use them more and more. For example, I now often have to check for macOS < 10.12, due to the large overhaul that happened in AppKit between 10.11 and 10.12. My workarounds for older versions of macOS are contained inside those `<font face="monospace">if {${os.major} <= 15}</font>` conditionals.</div></div><div dir="ltr"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Josh did recently commit "darwin" as a default for platforms, so it is no longer mandatory to explicitly put a platforms line in each portfile.<br></blockquote><div dir="ltr"><br></div><div>Does this mean that we can start removing `<font face="monospace">platforms darwin</font>` from our portfiles? Or is it one of those "despite the fact that it's a default, you have to put it in the file anyway"?</div><div><br></div><div dir="ltr"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>-- </div><div>Jason Liu</div></div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Dec 11, 2021 at 5:46 PM Ryan Schmidt <<a href="mailto:ryandesign@macports.org">ryandesign@macports.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">On Dec 11, 2021, at 16:41, Jason Liu wrote:<br>
<br>
> Then would it be easier (or even kosher) to simply wrap the majority of the Portfile inside of a single<br>
> <br>
> if {${os.platform} eq "darwin"} {<br>
<br>
The shorthand way to do that is:<br>
<br>
platform darwin {<br>
<br>
<br>
> and be done with it, instead of needing to add one each and every time I have a conditional involving `${os.major}`?<br>
<br>
I wouldn't expect you to need to write more than a couple such conditionals in a portfile so I wouldn't expect it to be that inconvenient.<br>
<br>
We use platform darwin blocks to isolate things that are obviously Mac-specific. It would be odd to see such a block enclosing most of a portfile, even bits which are obviously not Mac-specific.<br>
<br>
<br>
>> The platforms line is not used by MacPorts in any way at this time, other than to display it in the output of "port info". There is a ticket about possibly using it in the future as a way to indicate which OS versions the port is compatible with, but I don't think that got beyond the idea phase.<br>
> <br>
> That seems like a pity, and a bit of a waste, considering that platforms is being included in each and every Portfile.<br>
<br>
Well, it's as much a pity as any unimplemented feature in MacPorts.<br>
<br>
Josh did recently commit "darwin" as a default for platforms, so it is no longer mandatory to explicitly put a platforms line in each portfile.<br>
<br>
</blockquote></div></div>