Bug 247058

Summary: devel/llvm*: Avoid extraneous escapes
Product: Ports & Packages Reporter: Kyle Evans <kevans>
Component: Individual Port(s)Assignee: Kyle Evans <kevans>
Status: Closed FIXED    
Severity: Affects Only Me CC: brooks
Priority: --- Flags: brooks: maintainer-feedback+
Version: Latest   
Hardware: Any   
OS: Any   
Bug Depends on:    
Bug Blocks: 229925    
Attachments:
Description Flags
svn(1) diff against the ports tree koobs: maintainer-approval+

Description Kyle Evans freebsd_committer freebsd_triage 2020-06-07 18:21:30 UTC
Created attachment 215328 [details]
svn(1) diff against the ports tree

All of these single quotes are safe on their own, as they're enclosed in double quotes. Removes the escapes, because they're currently passed through to the underlying regex engine as \' to be interpreted as '.

A future change will forbid this escape, then an even further future change will actually imbue \' with special meaning based on GNU interpretation ("end of subject").

No PORTREVISION bump or rebuild is needed, as they're functionally equivalent.
Comment 1 Brooks Davis freebsd_committer freebsd_triage 2020-06-09 21:19:16 UTC
Please feel free to commit.
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2020-06-10 03:08:53 UTC
Comment on attachment 215328 [details]
svn(1) diff against the ports tree

Approved by: koobs (mentor)
MFH: <branch|No> <reason>
Comment 3 commit-hook freebsd_committer freebsd_triage 2020-06-10 04:08:23 UTC
A commit references this bug:

Author: kevans
Date: Wed Jun 10 04:07:21 UTC 2020
New revision: 538368
URL: https://svnweb.freebsd.org/changeset/ports/538368

Log:
  devel/llvm*: Avoid extraneous escapes

  All of these single quotes are safe on their own, as they're enclosed in
  double quotes. Removes the escapes, because they're currently passed through
  to the underlying regex engine as \' to be interpreted as '.

  A future change will forbid this escape, then an even further future change
  will actually imbue \' with special meaning based on GNU interpretation
  ("end of subject").

  No PORTREVISION bump or rebuild is needed, as they're functionally
  equivalent.

  PR:		247058
  Approved by:	koobs (mentor), brooks (maintainer)
  MFH:		2020Q2 (blanket: trivial build fix)

Changes:
  head/devel/llvm-cheri/Makefile
  head/devel/llvm-devel/Makefile
  head/devel/llvm10/Makefile
  head/devel/llvm60/Makefile
  head/devel/llvm70/Makefile
  head/devel/llvm80/Makefile
  head/devel/llvm90/Makefile
Comment 4 Kyle Evans freebsd_committer freebsd_triage 2020-06-10 04:15:45 UTC
I note that this line was actually only added with the sphinx upgrade @ r535791, so no further action will happen or is required.