Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unescaped left brace in regex is deprecated #18

Open
hobbes1069 opened this issue Mar 6, 2019 · 5 comments
Open

Unescaped left brace in regex is deprecated #18

hobbes1069 opened this issue Mar 6, 2019 · 5 comments

Comments

@hobbes1069
Copy link

From the Fedora Bugzilla report:

/usr/bin/abi-dumper contains a Perl expression that will stop working in Perl 5.30 (late May 2019):

$ abi-dumper --help >/dev/null
Unescaped left brace in regex is deprecated here (and will be fatal in Perl 5.30), passed through in regex; marked by <-- HERE in m/(\w){ <-- HERE / at /usr/bin/abi-dumper line 3200.

You need to apply this fix:

--- /usr/bin/abi-dumper 2019-01-31 13:54:11.000000000 +0100
+++ abi-dumper  2019-03-05 09:39:06.432000000 +0100
@@ -3197,7 +3197,7 @@
     
     if(defined $LambdaSupport)
     { # struct {lambda()}
-        $N=~s/(\w){/$1 {/g;
+        $N=~s/(\w)\{/$1 {/g;
     }
     
     return ($Cache{"formatName"}{$_[1]}{$_[0]} = $N);

abi-dumper-1.1-6.fc30.noarch is affected.

https://bugzilla.redhat.com/show_bug.cgi?id=1685441

@rhaschke
Copy link

I'm also affected by this issue. Is there a reason to not apply this patch?

@t-b
Copy link

t-b commented Jan 23, 2020

This is fixed in #11 already.

@hobbes1069
Copy link
Author

Good deal.

@hobbes1069
Copy link
Author

Actually, since this hasn't made it into a release maybe this should stay open. There has not been a release in some time so it may be time for one.

@hobbes1069 hobbes1069 reopened this Jan 23, 2020
@adriaandegroot
Copy link

This still seems to apply to what is available in Debian (1.1) but there was a 1.2 release which hasn't made it to Debian on Nov. 30 2020. So it's been fixed in a release, in the meantime. The fix was in 29ab01c (2017).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants