Bug 240450

Summary: x11-toolkits/p5-Alien-wxWidgets: remove extraneous escapes
Product: Ports & Packages Reporter: Kyle Evans <kevans>
Component: Individual Port(s)Assignee: freebsd-perl (Nobody) <perl>
Status: Closed FIXED    
Severity: Affects Only Me CC: kevans
Priority: --- Flags: bugzilla: maintainer-feedback? (perl)
Version: Latest   
Hardware: Any   
OS: Any   
Bug Depends on:    
Bug Blocks: 229925    
Attachments:
Description Flags
svn(1) diff against the ports tree none

Description Kyle Evans freebsd_committer 2019-09-09 16:18:44 UTC
Created attachment 207330 [details]
svn(1) diff against the ports tree

post-patch phase of this port uses sed to replace 'wx-config with '${WX_CONFIG:T}. Previously it escaped the ', but this is not necessary in this context.

Remove the extraneous escape so that it doesn't get passed through to sed(1) as an escape of an ordinary character. This is important as the patch in bug #229925 will make such escapes an error to ease transition into a world where some escaped-ordinaries will be granted a special meaning.

This patch also contains a pkg-plist update to include Config/gtk_3* -- I do not know if this is correct, but it was needed to get through a poudriere testport so I've included it.

QA:
 * portlint: OK (minor warnings that should be addressed separately.)
 * testport: OK (poudriere: 12.0-RELEASE, amd64[*])

[*] Only minimal testing done because the nature of the patch would lead to arch-independent catastrophic meltdown
Comment 1 commit-hook freebsd_committer 2020-02-20 18:58:49 UTC
A commit references this bug:

Author: kevans
Date: Thu Feb 20 18:58:18 UTC 2020
New revision: 526580
URL: https://svnweb.freebsd.org/changeset/ports/526580

Log:
  x11-toolkits/p5-Alien-wxWidgets: remove extraneous escapes

  post-patch phase of this port uses sed to replace 'wx-config with
  '${WX_CONFIG:T}. Previously it escaped the ', but this is not necessary in
  this context.

  Remove the extraneous escape so that it doesn't get passed through to sed(1)
  as an escape of an ordinary character. This is important as the patch in bug
  #229925 will make such escapes an error to ease transition into a world
  where some escaped-ordinaries will be granted a special meaning.

  PR:		240450
  Approved by:	koobs (mentor), bapt (mentor)
  Approved by:	portmgr (maintainer timeout: 4 months)
  MFH:		2020Q1 (future build fix)
  Differential Revision:	https://reviews.freebsd.org/D23702

Changes:
  head/x11-toolkits/p5-Alien-wxWidgets/Makefile
Comment 2 commit-hook freebsd_committer 2020-02-20 19:08:54 UTC
A commit references this bug:

Author: kevans
Date: Thu Feb 20 19:08:31 UTC 2020
New revision: 526585
URL: https://svnweb.freebsd.org/changeset/ports/526585

Log:
  MFH: r526580

  x11-toolkits/p5-Alien-wxWidgets: remove extraneous escapes

  post-patch phase of this port uses sed to replace 'wx-config with
  '${WX_CONFIG:T}. Previously it escaped the ', but this is not necessary in
  this context.

  Remove the extraneous escape so that it doesn't get passed through to sed(1)
  as an escape of an ordinary character. This is important as the patch in bug
  #229925 will make such escapes an error to ease transition into a world
  where some escaped-ordinaries will be granted a special meaning.

  PR:		240450
  Approved by:	koobs (mentor), bapt (mentor)
  Approved by:	portmgr (maintainer timeout: 4 months)
  Differential Revision:	https://reviews.freebsd.org/D23702

  Approved by:	ports-secteam (implicit, future build fix)

Changes:
_U  branches/2020Q1/
  branches/2020Q1/x11-toolkits/p5-Alien-wxWidgets/Makefile