* [Buildroot] [PATCH 1/1] automake: fix unescaped left brace warning patch.
@ 2017-02-09 0:07 Adam Duskett
2017-02-09 8:41 ` Thomas Petazzoni
0 siblings, 1 reply; 4+ messages in thread
From: Adam Duskett @ 2017-02-09 0:07 UTC (permalink / raw)
To: buildroot
Automake currently has a unescaped left brace in it's automake.in folder
that generates the warning:
Unescaped left brace in regex is deprecated, passed through in regex;
This patch fixes this warning.
Signed-off-by: Adam Duskett <Adamduskett@outlook.com>
---
.../0002-fix-unescaped-left-brace-warning.patch | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
create mode 100644 package/automake/0002-fix-unescaped-left-brace-warning.patch
diff --git a/package/automake/0002-fix-unescaped-left-brace-warning.patch b/package/automake/0002-fix-unescaped-left-brace-warning.patch
new file mode 100644
index 0000000..a33b1f1
--- /dev/null
+++ b/package/automake/0002-fix-unescaped-left-brace-warning.patch
@@ -0,0 +1,19 @@
+Automake currently has a unescaped left brace in it's automake.in folder
+that generates the warning:
+Unescaped left brace in regex is deprecated, passed through in regex;
+This patch fixes this warning.
+
+Signed-off-by: Adam Duskett <aduskett@gmail.com>
+
+diff -Naur a/bin/automake.in b/bin/automake.in
+--- a/bin/automake.in 2015-01-05 14:25:55.000000000 -0500
++++ b/bin/automake.in 2017-02-08 18:59:24.963611569 -0500
+@@ -3878,7 +3878,7 @@
+ sub substitute_ac_subst_variables
+ {
+ my ($text) = @_;
+- $text =~ s/\${([^ \t=:+{}]+)}/substitute_ac_subst_variables_worker ($1)/ge;
++ $text =~ s/\$\{([^ \t=:+{}]+)\}/substitute_ac_subst_variables_worker ($1)/ge;
+ return $text;
+ }
+
--
2.9.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] automake: fix unescaped left brace warning patch.
2017-02-09 0:07 [Buildroot] [PATCH 1/1] automake: fix unescaped left brace warning patch Adam Duskett
@ 2017-02-09 8:41 ` Thomas Petazzoni
2017-02-09 12:33 ` Adam Duskett
0 siblings, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2017-02-09 8:41 UTC (permalink / raw)
To: buildroot
Hello,
On Wed, 8 Feb 2017 19:07:10 -0500, Adam Duskett wrote:
> .../0002-fix-unescaped-left-brace-warning.patch | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
> create mode 100644 package/automake/0002-fix-unescaped-left-brace-warning.patch
>
> diff --git a/package/automake/0002-fix-unescaped-left-brace-warning.patch b/package/automake/0002-fix-unescaped-left-brace-warning.patch
> new file mode 100644
> index 0000000..a33b1f1
> --- /dev/null
> +++ b/package/automake/0002-fix-unescaped-left-brace-warning.patch
> @@ -0,0 +1,19 @@
> +Automake currently has a unescaped left brace in it's automake.in folder
> +that generates the warning:
> +Unescaped left brace in regex is deprecated, passed through in regex;
> +This patch fixes this warning.
> +
> +Signed-off-by: Adam Duskett <aduskett@gmail.com>
I am *really* uncomfortable about merging patches touching the automake
code base if those patches are not upstream. So please submit this
upstream, and if it gets accepted, then we can integrate it in
Buildroot.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] automake: fix unescaped left brace warning patch.
2017-02-09 8:41 ` Thomas Petazzoni
@ 2017-02-09 12:33 ` Adam Duskett
2017-02-09 13:25 ` Thomas Petazzoni
0 siblings, 1 reply; 4+ messages in thread
From: Adam Duskett @ 2017-02-09 12:33 UTC (permalink / raw)
To: buildroot
Hello;
On Thu, Feb 9, 2017 at 3:41 AM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Hello,
>
> On Wed, 8 Feb 2017 19:07:10 -0500, Adam Duskett wrote:
>
>> .../0002-fix-unescaped-left-brace-warning.patch | 19 +++++++++++++++++++
>> 1 file changed, 19 insertions(+)
>> create mode 100644 package/automake/0002-fix-unescaped-left-brace-warning.patch
>>
>> diff --git a/package/automake/0002-fix-unescaped-left-brace-warning.patch b/package/automake/0002-fix-unescaped-left-brace-warning.patch
>> new file mode 100644
>> index 0000000..a33b1f1
>> --- /dev/null
>> +++ b/package/automake/0002-fix-unescaped-left-brace-warning.patch
>> @@ -0,0 +1,19 @@
>> +Automake currently has a unescaped left brace in it's automake.in folder
>> +that generates the warning:
>> +Unescaped left brace in regex is deprecated, passed through in regex;
>> +This patch fixes this warning.
>> +
>> +Signed-off-by: Adam Duskett <aduskett@gmail.com>
>
> I am *really* uncomfortable about merging patches touching the automake
> code base if those patches are not upstream. So please submit this
> upstream, and if it gets accepted, then we can integrate it in
> Buildroot.
>
> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com
Huh, looks like it was submitted and accepted into their git
repository. I didn't even notice!
http://git.savannah.gnu.org/cgit/automake.git/commit/?id=13f00eb4493c217269b76614759e452d8302955e
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] automake: fix unescaped left brace warning patch.
2017-02-09 12:33 ` Adam Duskett
@ 2017-02-09 13:25 ` Thomas Petazzoni
0 siblings, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2017-02-09 13:25 UTC (permalink / raw)
To: buildroot
Hello,
On Thu, 9 Feb 2017 07:33:26 -0500, Adam Duskett wrote:
> Huh, looks like it was submitted and accepted into their git
> repository. I didn't even notice!
> http://git.savannah.gnu.org/cgit/automake.git/commit/?id=13f00eb4493c217269b76614759e452d8302955e
OK. Then please backport this upstreaming commit, and submit it as a
Buildroot patch. Thanks a lot!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-02-09 13:25 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-09 0:07 [Buildroot] [PATCH 1/1] automake: fix unescaped left brace warning patch Adam Duskett
2017-02-09 8:41 ` Thomas Petazzoni
2017-02-09 12:33 ` Adam Duskett
2017-02-09 13:25 ` Thomas Petazzoni
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.