[MacPorts] #55471: Use separate portindexes for libc++ on older systems

MacPorts noreply at macports.org
Sun Apr 22 21:53:41 UTC 2018


#55471: Use separate portindexes for libc++ on older systems
--------------------------+----------------------------
  Reporter:  ryandesign   |      Owner:  neverpanic
      Type:  enhancement  |     Status:  assigned
  Priority:  Normal       |  Milestone:  MacPorts 2.5.0
 Component:  base         |    Version:
Resolution:               |   Keywords:
      Port:               |
--------------------------+----------------------------

Comment (by raimue):

 As Mojca [https://github.com/macports/macports-
 base/pull/83#issuecomment-383409609 asked] why I did not want to commit
 the changes to `macports.tcl`, let me explain that here.

 In the proposed changes to `macports.tcl`, `${platindex_cxxlib}` will be
 added to the remote PortIndex filename. It will only do this if the value
 of `cxx_stdlib` does not match the default.

 Side note: I do not understand why you would want to hardcode the default
 cxx_stdlib at this place. I would just always add `${platindex_cxxlib}` to
 the remote filename, isn't this much simpler? Wasn't your plan to
 eventually change the default on older systems at some point? Hardcoding
 default values seems to work against that.

 I also have reservations because of users that already have a non-default
 setting for `cxx_stdlib`. With this change, mportsync would expect a
 remote filename that does not yet exist. What are the consequences of
 this? Is the plan to deploy the changes to mprsyncup immediately after the
 2.5.0 release without ever testing the changes against rsync? Will
 mportsync fail gracefully or hard without a remote PortIndex file? Will
 base fall back to generate a PortIndex locally in this case? I see no
 reports of tests or consideration of it in this ticket.

 The changes to `macports.tcl` require the changes to `mprsyncup`, but that
 requires the changes to `portindex`.

 I can only review the change to portindex as ready-to-go. The new
 portindex would already fix the problem that PortIndex was generated with
 the wrong value of cxx_stdlib in mprsyncup for legacy platforms. This
 seemed to be the most important fix of this pull request to go into a
 release. I also thought this is what we discussed to do for MacPorts 2.5.0
 in our online meeting.

-- 
Ticket URL: <https://trac.macports.org/ticket/55471#comment:26>
MacPorts <https://www.macports.org/>
Ports system for macOS


More information about the macports-tickets mailing list