Opened 4 years ago

Closed 20 months ago

#59811 closed enhancement (fixed)

ccls: enhancements: install default subport; add port-select functionality

Reported by: casouri (Yuan Fu) Owned by: mascguy (Christopher Nielsen)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: egorenar (Alexander Egorenkov)
Port: ccls

Description

ccls could have some enhancements:

  1. currently ccls port is a meta port and doesn't install any executable. That is confusing because people (me) type sudo port install ccls and expect ccls binary to be installed. I suggest hiding ccls port altogether to avoid confusion.
  1. The two subports ccls-clang-7.0 and ccls-clang-8.0 install executables named ccls-clang-7.0 and ccls-clang-8.0 - that's confusing too because most users and programs expect the "ccls" executable. We should add a selection group and a default selection so there is an executable named ccls automatically and users can easily switch between the two subports.

Change History (6)

comment:1 Changed 4 years ago by mf2k (Frank Schima)

Owner: set to egorenar
Status: newassigned
Type: defectenhancement
Version: 2.6.2

comment:2 Changed 20 months ago by mascguy (Christopher Nielsen)

Cc: mascguy added

comment:3 Changed 20 months ago by mascguy (Christopher Nielsen)

Cc: egorenar added; mascguy removed
Owner: changed from egorenar to mascguy

I have an open PR to update this port, and both of these are easy enough to add. So I'll take care of this too.

comment:4 in reply to:  3 Changed 20 months ago by mascguy (Christopher Nielsen)

Replying to mascguy:

I have an open PR to update this port, and both of these are easy enough to add. So I'll take care of this too.

Work completed, and added to open PR:

15870 - ccls: update to 20220729; add clang 12 thru 14; add port-select functionality

Last edited 20 months ago by mascguy (Christopher Nielsen) (previous) (diff)

comment:5 Changed 20 months ago by mascguy (Christopher Nielsen)

Summary: ccls enhancementccls: enhancements: install default subport; add port-select functionality

comment:6 Changed 20 months ago by Christopher Nielsen <mascguy@…>

Resolution: fixed
Status: assignedclosed

In d80096db2db4144c4cf880d1a8a2d671baacd4ff/macports-ports (master):

ccls: update to 20220729; add clang 12 thru 14

  • Use rapidjson, rather than building private version
  • Install ccls-clang-10 by default, for parent port
  • Add port-select functionality

Fixes: #59811

Note: See TracTickets for help on using tickets.