Opened 6 years ago

Closed 3 years ago

Last modified 3 years ago

#56629 closed enhancement (fixed)

Make cxx_stdlib mismatches more verbose

Reported by: majoc-at-astro (majoc-at-astro) Owned by: jmroot (Joshua Root)
Priority: Normal Milestone: MacPorts 2.7.0
Component: base Version: 2.5.2
Keywords: haspatch Cc: mojca (Mojca Miklavec), cooljeanius (Eric Gallager)
Port:

Description

When there's a cxx_stdlib conflict, a non-verbose log fails mysteriously; please see comment:ticket:56551:17 for an example. A verbose log will be excessively large, but will contain something of the form:

$portname is using libstdc++ (this installation is configured to use libc++)

May I suggest that this be present in the non-verbose output as well? I enclose an untested and extremely tentative patch on macports.tcl, which changes ui_info to ui_error in the statement issuing this message; please feel free to use ui_warn instead if that's more appropriate.

Apologies for errors, in particular not knowing who to CC this to.

Attachments (1)

macports.tcl_patch (663 bytes) - added by majoc-at-astro (majoc-at-astro) 6 years ago.

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by majoc-at-astro (majoc-at-astro)

Attachment: macports.tcl_patch added

comment:1 Changed 6 years ago by mojca (Mojca Miklavec)

May I please ask you to submit a pull request to macports-base? I'm just randomly browsing through tickets and I'm not senior enough to be able to tell whether there would be any undesirable side-effects. Opening a PR will give this patch way more visibility and the right people will look into it.

If you need to decide whom to CC for base-related patches, you'll get the best idea by looking at git history (the number of people involved is very limited).

comment:2 Changed 5 years ago by mojca (Mojca Miklavec)

Cc: mojca added

comment:3 Changed 3 years ago by jmroot (Joshua Root)

Owner: set to jmroot
Resolution: fixed
Status: newclosed

In 00a263133c8132220fd8505c9927e82893872222/macports-base (master):

rev-upgrade: report cxx_stdlib mismatches better

Closes: #56629

comment:4 Changed 3 years ago by jmroot (Joshua Root)

Milestone: MacPorts 2.7.0
Summary: Feature request: make cxx_stdlib conflicts more verboseMake cxx_stdlib mismatches more verbose

comment:5 Changed 3 years ago by cooljeanius (Eric Gallager)

Cc: cooljeanius added
Note: See TracTickets for help on using tickets.