Opened 6 years ago

Closed 6 years ago

#56379 closed defect (fixed)

root5: remove +clang35 +clang36 variants

Reported by: dbevans (David B. Evans) Owned by: cjones051073 (Chris Jones)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: root5

Description

These variants can be removed since the corresponding compiler ports (clang-3.5, clang-3.6) have been removed from MacPorts. Recommended replacement is clang-5.0.

$ port info clang-3.5 clang-3.6
clang-3.5 @3.5.2_10 (lang)
Replaced by:          clang-5.0
Description:          This port has been replaced by clang-5.0.
--
clang-3.6 @3.6.2_6 (lang)
Replaced by:          clang-5.0
Description:          This port has been replaced by clang-5.0.

Change History (10)

comment:1 Changed 6 years ago by cjones051073 (Chris Jones)

Indeed. Please submit a GitHub PR with this change.

comment:2 Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)

Port: root5 added

You're the maintainer. Why don't you do it? :)

comment:3 Changed 6 years ago by cjones051073 (Chris Jones)

Indeed... I don't really care about root5 any more though, its dead w.r.t. root6, so someone else who did has the update ready all the better.

comment:4 Changed 6 years ago by cjones051073 (Chris Jones)

I also would prefer to not follow trac any more, and just focus on GitHub ....

comment:5 Changed 6 years ago by pmetzger (Perry E. Metzger)

cjones051073: should we in fact be obsoleting root5 then?

comment:6 Changed 6 years ago by cjones051073 (Chris Jones)

That might be an idea...

comment:7 Changed 6 years ago by pmetzger (Perry E. Metzger)

cjones051073: As you are the maintainer, submit a pull request, and then root5 will never result in another Trac ticket. :)

comment:8 Changed 6 years ago by cjones051073 (Chris Jones)

I first want to make sure no ports in MacPorts still use root5, and if there are get them motivated to migrate to root6. But then, yes, it would be time to think about obsoleting root5.

comment:9 Changed 6 years ago by cjones051073 (Chris Jones)

I just pushed an update for this, removing the old variants. I also updated the compiler whitelist to force the use of MPs clang 5.0, as this seems to now give better support than last time I tried. In particular it builds on maxOS10.13.

We still should push users of root5 (Users or ports) to root6. However thinking a bit more (better) than last time it might be worth keeping root5 for a bit, as root6 does not work on very old OSX versions.

comment:10 Changed 6 years ago by cjones051073 (Chris Jones)

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.