Opened 7 years ago

Last modified 3 years ago

#56223 assigned defect

clang-6.0 unused regex?

Reported by: gaming-hacker (G Alexander) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.4.2
Keywords: Cc: larryv (Lawrence Velázquez), jeremyhu (Jeremy Huddleston Sequoia)
Port: clang-6.0

Description

Hello, I upgraded to latest clang-6.0 and received the following warning.

Full Name: clang-6.0 @6.0.0+analyzer+libstdcxx

--->  Applying patches to clang-6.0
Warning: reinplace s|/usr/bin/env python|/opt/local/bin/python2.7|g didn't change anything in /opt/local/var/macports/build/_HOME_local_macports-ports_lang_llvm-6.0/clang-6.0/work/llvm-6.0.0.src/tools/clang/tools/scan-build/bin/set-xcode-analyzer

port seems to build ok and running tests now.

Change History (8)

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

Cc: jeremyhu removed
Keywords: regex removed
Owner: set to jeremyhu
Status: newassigned

comment:2 Changed 6 years ago by chrstphrchvz (Christopher Chavez)

This warning appears with other versions of clang as well, since the same replacement is being done by their portfiles (I just observed it for clang-7.0 7.0.1).

According to https://github.com/llvm-mirror/clang/commit/5a15d92206f8ef357c49d6993a6ec4a85bde7b8f, the shebang in set-xcode-analyzer has been /usr/bin/python for several years.

comment:3 in reply to:  2 Changed 6 years ago by chrstphrchvz (Christopher Chavez)

Replying to chrstphrchvz:

This warning appears with other versions of clang as well, since the same replacement is being done by their portfiles

See #54972 (clang-4.0)

comment:4 Changed 5 years ago by kencu (Ken)

Cc: kencu added

comment:5 Changed 4 years ago by kencu (Ken)

Cc: jeremyhu added; kencu removed
Owner: changed from jeremyhu to kencu

comment:6 Changed 4 years ago by gaming-hacker (G Alexander)

ok, can someone close this?

comment:7 Changed 4 years ago by kencu (Ken)

I'm not sure I fixed it yet.

It's pretty low on the priority list, TBH. Just finished fixing up icu, and now working on clang-12, and making sure it works properly for 10.6 to BigSur is a bit of a project.

These little reinplace warnings are on the list, but not the top of the list.

comment:8 Changed 3 years ago by kencu (Ken)

Owner: changed from kencu to macports-tickets@…
Note: See TracTickets for help on using tickets.