* [PATCH] attr: attr_2.4.47 release tarball missing configure.ac
@ 2013-11-25 8:18 Nathan Rossi
2013-12-02 20:00 ` Saul Wold
0 siblings, 1 reply; 5+ messages in thread
From: Nathan Rossi @ 2013-11-25 8:18 UTC (permalink / raw)
To: openembedded-core
The release tarball of attr 2.4.47 is missing the configure.ac file.
This prevents the autotools bbclass from being able to regenerate the
configure script which leads to other side affects. (e.g. using
outdated config.sub)
This patch adds the configure.ac file via a patch.
(obtained via the source repository at the v2.4.47 tag)
See the mailing list thread for additional information:
http://lists.nongnu.org/archive/html/acl-devel/2013-05/msg00024.html
Signed-off-by: Nathan Rossi <nathan.rossi@xilinx.com>
---
On an additional note, acl 2.2.52 is also affected by this issue.
However it does not affect oe-core as acl has yet to be updated to this
version and currently sits on the older 2.2.51 version.
---
meta/recipes-support/attr/attr_2.4.47.bb | 3 +
.../attr/files/attr-Missing-configure.ac.patch | 62 ++++++++++++++++++++
2 files changed, 65 insertions(+)
create mode 100644 meta/recipes-support/attr/files/attr-Missing-configure.ac.patch
diff --git a/meta/recipes-support/attr/attr_2.4.47.bb b/meta/recipes-support/attr/attr_2.4.47.bb
index a1f2e05..ad40c9b 100644
--- a/meta/recipes-support/attr/attr_2.4.47.bb
+++ b/meta/recipes-support/attr/attr_2.4.47.bb
@@ -1,5 +1,8 @@
require attr.inc
+# configure.ac was missing from the release tarball. This should be fixed in
+# future releases of attr, remove this when updating the recipe.
+SRC_URI_append += "file://attr-Missing-configure.ac.patch"
SRC_URI[md5sum] = "84f58dec00b60f2dc8fd1c9709291cc7"
SRC_URI[sha256sum] = "25772f653ac5b2e3ceeb89df50e4688891e21f723c460636548971652af0a859"
diff --git a/meta/recipes-support/attr/files/attr-Missing-configure.ac.patch b/meta/recipes-support/attr/files/attr-Missing-configure.ac.patch
new file mode 100644
index 0000000..5887ed5
--- /dev/null
+++ b/meta/recipes-support/attr/files/attr-Missing-configure.ac.patch
@@ -0,0 +1,62 @@
+Subject: [PATCH] attr: Missing configure.ac
+
+Upstream-Status: Backport [Upstream released tarball missing this file]
+---
+ configure.ac | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
+ 1 file changed, 48 insertions(+)
+ create mode 100644 configure.ac
+
+diff --git a/configure.ac b/configure.ac
+new file mode 100644
+index 0000000..b966d0e
+--- /dev/null
++++ b/configure.ac
+@@ -0,0 +1,48 @@
++# Copyright (C) 2009 Andreas Gruenbacher <agruen@suse.de>
++#
++# This program is free software: you can redistribute it and/or modify it
++# under the terms of the GNU General Public License as published by
++# the Free Software Foundation, either version 2 of the License, or
++# (at your option) any later version.
++#
++# This program is distributed in the hope that it will be useful,
++# but WITHOUT ANY WARRANTY; without even the implied warranty of
++# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
++# GNU General Public License for more details.
++#
++# You should have received a copy of the GNU General Public License
++# along with this program. If not, see <http://www.gnu.org/licenses/>.
++#
++AC_INIT(include/attributes.h)
++AC_CONFIG_AUX_DIR([.])
++AC_CONFIG_MACRO_DIR([m4])
++AC_CONFIG_HEADER(include/config.h)
++AC_PREFIX_DEFAULT(/usr)
++
++AC_PROG_LIBTOOL
++
++AC_ARG_ENABLE(shared,
++[ --enable-shared=[yes/no] Enable use of shared libraries [default=yes]],,
++ enable_shared=yes)
++AC_SUBST(enable_shared)
++
++AC_ARG_ENABLE(gettext,
++[ --enable-gettext=[yes/no] Enable alternate language support [default=yes]],,
++ enable_gettext=yes)
++AC_SUBST(enable_gettext)
++
++AC_ARG_ENABLE(lib64,
++[ --enable-lib64=[yes/no] Enable lib64 support [default=no]],,
++ enable_lib64=no)
++AC_SUBST(enable_lib64)
++
++AC_PACKAGE_GLOBALS(attr)
++AC_PACKAGE_UTILITIES(attr)
++AC_MANUAL_FORMAT
++AC_MULTILIB($enable_lib64)
++
++AC_C_CONST
++AC_TYPE_MODE_T
++AC_FUNC_ALLOCA
++
++AC_OUTPUT(include/builddefs)
--
1.7.9.5
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] attr: attr_2.4.47 release tarball missing configure.ac
2013-11-25 8:18 [PATCH] attr: attr_2.4.47 release tarball missing configure.ac Nathan Rossi
@ 2013-12-02 20:00 ` Saul Wold
2013-12-03 1:08 ` Nathan Rossi
0 siblings, 1 reply; 5+ messages in thread
From: Saul Wold @ 2013-12-02 20:00 UTC (permalink / raw)
To: Nathan Rossi, openembedded-core
On 11/25/2013 12:18 AM, Nathan Rossi wrote:
> The release tarball of attr 2.4.47 is missing the configure.ac file.
> This prevents the autotools bbclass from being able to regenerate the
> configure script which leads to other side affects. (e.g. using
> outdated config.sub)
>
> This patch adds the configure.ac file via a patch.
> (obtained via the source repository at the v2.4.47 tag)
>
> See the mailing list thread for additional information:
> http://lists.nongnu.org/archive/html/acl-devel/2013-05/msg00024.html
>
> Signed-off-by: Nathan Rossi <nathan.rossi@xilinx.com>
> ---
>
> On an additional note, acl 2.2.52 is also affected by this issue.
> However it does not affect oe-core as acl has yet to be updated to this
> version and currently sits on the older 2.2.51 version.
>
> ---
> meta/recipes-support/attr/attr_2.4.47.bb | 3 +
> .../attr/files/attr-Missing-configure.ac.patch | 62 ++++++++++++++++++++
> 2 files changed, 65 insertions(+)
> create mode 100644 meta/recipes-support/attr/files/attr-Missing-configure.ac.patch
>
> diff --git a/meta/recipes-support/attr/attr_2.4.47.bb b/meta/recipes-support/attr/attr_2.4.47.bb
> index a1f2e05..ad40c9b 100644
> --- a/meta/recipes-support/attr/attr_2.4.47.bb
> +++ b/meta/recipes-support/attr/attr_2.4.47.bb
> @@ -1,5 +1,8 @@
> require attr.inc
>
> +# configure.ac was missing from the release tarball. This should be fixed in
> +# future releases of attr, remove this when updating the recipe.
> +SRC_URI_append += "file://attr-Missing-configure.ac.patch"
>
> SRC_URI[md5sum] = "84f58dec00b60f2dc8fd1c9709291cc7"
> SRC_URI[sha256sum] = "25772f653ac5b2e3ceeb89df50e4688891e21f723c460636548971652af0a859"
> diff --git a/meta/recipes-support/attr/files/attr-Missing-configure.ac.patch b/meta/recipes-support/attr/files/attr-Missing-configure.ac.patch
> new file mode 100644
> index 0000000..5887ed5
> --- /dev/null
> +++ b/meta/recipes-support/attr/files/attr-Missing-configure.ac.patch
> @@ -0,0 +1,62 @@
> +Subject: [PATCH] attr: Missing configure.ac
> +
> +Upstream-Status: Backport [Upstream released tarball missing this file]
You need a Signed-off-by: tag in this patch also
Thanks
Sau!
> +---
> + configure.ac | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
> + 1 file changed, 48 insertions(+)
> + create mode 100644 configure.ac
> +
> +diff --git a/configure.ac b/configure.ac
> +new file mode 100644
> +index 0000000..b966d0e
> +--- /dev/null
> ++++ b/configure.ac
> +@@ -0,0 +1,48 @@
> ++# Copyright (C) 2009 Andreas Gruenbacher <agruen@suse.de>
> ++#
> ++# This program is free software: you can redistribute it and/or modify it
> ++# under the terms of the GNU General Public License as published by
> ++# the Free Software Foundation, either version 2 of the License, or
> ++# (at your option) any later version.
> ++#
> ++# This program is distributed in the hope that it will be useful,
> ++# but WITHOUT ANY WARRANTY; without even the implied warranty of
> ++# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> ++# GNU General Public License for more details.
> ++#
> ++# You should have received a copy of the GNU General Public License
> ++# along with this program. If not, see <http://www.gnu.org/licenses/>.
> ++#
> ++AC_INIT(include/attributes.h)
> ++AC_CONFIG_AUX_DIR([.])
> ++AC_CONFIG_MACRO_DIR([m4])
> ++AC_CONFIG_HEADER(include/config.h)
> ++AC_PREFIX_DEFAULT(/usr)
> ++
> ++AC_PROG_LIBTOOL
> ++
> ++AC_ARG_ENABLE(shared,
> ++[ --enable-shared=[yes/no] Enable use of shared libraries [default=yes]],,
> ++ enable_shared=yes)
> ++AC_SUBST(enable_shared)
> ++
> ++AC_ARG_ENABLE(gettext,
> ++[ --enable-gettext=[yes/no] Enable alternate language support [default=yes]],,
> ++ enable_gettext=yes)
> ++AC_SUBST(enable_gettext)
> ++
> ++AC_ARG_ENABLE(lib64,
> ++[ --enable-lib64=[yes/no] Enable lib64 support [default=no]],,
> ++ enable_lib64=no)
> ++AC_SUBST(enable_lib64)
> ++
> ++AC_PACKAGE_GLOBALS(attr)
> ++AC_PACKAGE_UTILITIES(attr)
> ++AC_MANUAL_FORMAT
> ++AC_MULTILIB($enable_lib64)
> ++
> ++AC_C_CONST
> ++AC_TYPE_MODE_T
> ++AC_FUNC_ALLOCA
> ++
> ++AC_OUTPUT(include/builddefs)
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] attr: attr_2.4.47 release tarball missing configure.ac
2013-12-02 20:00 ` Saul Wold
@ 2013-12-03 1:08 ` Nathan Rossi
2013-12-03 4:02 ` Saul Wold
0 siblings, 1 reply; 5+ messages in thread
From: Nathan Rossi @ 2013-12-03 1:08 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
> -----Original Message-----
> From: Saul Wold [mailto:sgw@linux.intel.com]
> Sent: Tuesday, December 03, 2013 6:00 AM
> To: Nathan Rossi; openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH] attr: attr_2.4.47 release tarball missing
> configure.ac
>
> On 11/25/2013 12:18 AM, Nathan Rossi wrote:
> > The release tarball of attr 2.4.47 is missing the configure.ac file.
> > This prevents the autotools bbclass from being able to regenerate the
> > configure script which leads to other side affects. (e.g. using
> > outdated config.sub)
> >
> > This patch adds the configure.ac file via a patch.
> > (obtained via the source repository at the v2.4.47 tag)
> >
> > See the mailing list thread for additional information:
> > http://lists.nongnu.org/archive/html/acl-devel/2013-05/msg00024.html
> >
> > Signed-off-by: Nathan Rossi <nathan.rossi@xilinx.com>
> > ---
> >
> > On an additional note, acl 2.2.52 is also affected by this issue.
> > However it does not affect oe-core as acl has yet to be updated to this
> > version and currently sits on the older 2.2.51 version.
> >
> > ---
> > meta/recipes-support/attr/attr_2.4.47.bb | 3 +
> > .../attr/files/attr-Missing-configure.ac.patch | 62
> ++++++++++++++++++++
> > 2 files changed, 65 insertions(+)
> > create mode 100644 meta/recipes-support/attr/files/attr-Missing-
> configure.ac.patch
> >
> > diff --git a/meta/recipes-support/attr/attr_2.4.47.bb b/meta/recipes-
> support/attr/attr_2.4.47.bb
> > index a1f2e05..ad40c9b 100644
> > --- a/meta/recipes-support/attr/attr_2.4.47.bb
> > +++ b/meta/recipes-support/attr/attr_2.4.47.bb
> > @@ -1,5 +1,8 @@
> > require attr.inc
> >
> > +# configure.ac was missing from the release tarball. This should be
> fixed in
> > +# future releases of attr, remove this when updating the recipe.
> > +SRC_URI_append += "file://attr-Missing-configure.ac.patch"
> >
> > SRC_URI[md5sum] = "84f58dec00b60f2dc8fd1c9709291cc7"
> > SRC_URI[sha256sum] =
> "25772f653ac5b2e3ceeb89df50e4688891e21f723c460636548971652af0a859"
> > diff --git a/meta/recipes-support/attr/files/attr-Missing-
> configure.ac.patch b/meta/recipes-support/attr/files/attr-Missing-
> configure.ac.patch
> > new file mode 100644
> > index 0000000..5887ed5
> > --- /dev/null
> > +++ b/meta/recipes-support/attr/files/attr-Missing-configure.ac.patch
> > @@ -0,0 +1,62 @@
> > +Subject: [PATCH] attr: Missing configure.ac
> > +
> > +Upstream-Status: Backport [Upstream released tarball missing this file]
>
> You need a Signed-off-by: tag in this patch also
>
Hi Saul,
This patch has already been pushed onto the master branch. http://git.openembedded.org/openembedded-core/commit/?id=b28f12a272a9e2f0c3084a58b91605acb05f58f8
Would you like me to submit a follow up patch which adds just the signed-off-by line? Or would you prefer a V2 of this patch with the additional change?
Regards,
Nathan
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] attr: attr_2.4.47 release tarball missing configure.ac
2013-12-03 1:08 ` Nathan Rossi
@ 2013-12-03 4:02 ` Saul Wold
0 siblings, 0 replies; 5+ messages in thread
From: Saul Wold @ 2013-12-03 4:02 UTC (permalink / raw)
To: Nathan Rossi; +Cc: openembedded-core
On 12/02/2013 05:08 PM, Nathan Rossi wrote:
>> -----Original Message-----
>> From: Saul Wold [mailto:sgw@linux.intel.com]
>> Sent: Tuesday, December 03, 2013 6:00 AM
>> To: Nathan Rossi; openembedded-core@lists.openembedded.org
>> Subject: Re: [OE-core] [PATCH] attr: attr_2.4.47 release tarball missing
>> configure.ac
>>
>> On 11/25/2013 12:18 AM, Nathan Rossi wrote:
>>> The release tarball of attr 2.4.47 is missing the configure.ac file.
>>> This prevents the autotools bbclass from being able to regenerate the
>>> configure script which leads to other side affects. (e.g. using
>>> outdated config.sub)
>>>
>>> This patch adds the configure.ac file via a patch.
>>> (obtained via the source repository at the v2.4.47 tag)
>>>
>>> See the mailing list thread for additional information:
>>> http://lists.nongnu.org/archive/html/acl-devel/2013-05/msg00024.html
>>>
>>> Signed-off-by: Nathan Rossi <nathan.rossi@xilinx.com>
>>> ---
>>>
>>> On an additional note, acl 2.2.52 is also affected by this issue.
>>> However it does not affect oe-core as acl has yet to be updated to this
>>> version and currently sits on the older 2.2.51 version.
>>>
>>> ---
>>> meta/recipes-support/attr/attr_2.4.47.bb | 3 +
>>> .../attr/files/attr-Missing-configure.ac.patch | 62
>> ++++++++++++++++++++
>>> 2 files changed, 65 insertions(+)
>>> create mode 100644 meta/recipes-support/attr/files/attr-Missing-
>> configure.ac.patch
>>>
>>> diff --git a/meta/recipes-support/attr/attr_2.4.47.bb b/meta/recipes-
>> support/attr/attr_2.4.47.bb
>>> index a1f2e05..ad40c9b 100644
>>> --- a/meta/recipes-support/attr/attr_2.4.47.bb
>>> +++ b/meta/recipes-support/attr/attr_2.4.47.bb
>>> @@ -1,5 +1,8 @@
>>> require attr.inc
>>>
>>> +# configure.ac was missing from the release tarball. This should be
>> fixed in
>>> +# future releases of attr, remove this when updating the recipe.
>>> +SRC_URI_append += "file://attr-Missing-configure.ac.patch"
>>>
>>> SRC_URI[md5sum] = "84f58dec00b60f2dc8fd1c9709291cc7"
>>> SRC_URI[sha256sum] =
>> "25772f653ac5b2e3ceeb89df50e4688891e21f723c460636548971652af0a859"
>>> diff --git a/meta/recipes-support/attr/files/attr-Missing-
>> configure.ac.patch b/meta/recipes-support/attr/files/attr-Missing-
>> configure.ac.patch
>>> new file mode 100644
>>> index 0000000..5887ed5
>>> --- /dev/null
>>> +++ b/meta/recipes-support/attr/files/attr-Missing-configure.ac.patch
>>> @@ -0,0 +1,62 @@
>>> +Subject: [PATCH] attr: Missing configure.ac
>>> +
>>> +Upstream-Status: Backport [Upstream released tarball missing this file]
>>
>> You need a Signed-off-by: tag in this patch also
>>
>
> Hi Saul,
>
> This patch has already been pushed onto the master branch. http://git.openembedded.org/openembedded-core/commit/?id=b28f12a272a9e2f0c3084a58b91605acb05f58f8
>
Yes, I noticed this after I sent the message!
> Would you like me to submit a follow up patch which adds just the signed-off-by line? Or would you prefer a V2 of this patch with the additional change?
>
Please send a patch to add the signed-off-by line to the patch, we try
to keep track of who added the patches so we have the history.
Thanks
Sau!
> Regards,
> Nathan
>
>
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] attr: attr_2.4.47 release tarball missing configure.ac
@ 2013-11-26 0:58 Nathan Rossi
0 siblings, 0 replies; 5+ messages in thread
From: Nathan Rossi @ 2013-11-26 0:58 UTC (permalink / raw)
To: openembedded-core
The release tarball of attr 2.4.47 is missing the configure.ac file.
This prevents the autotools bbclass from being able to regenerate the
configure script which leads to other side affects. (e.g. using
outdated config.sub)
This patch adds the configure.ac file via a patch.
(obtained via the source repository at the v2.4.47 tag)
See the mailing list thread for additional information:
http://lists.nongnu.org/archive/html/acl-devel/2013-05/msg00024.html
Signed-off-by: Nathan Rossi <nathan.rossi@xilinx.com>
---
On an additional note, acl 2.2.52 is also affected by this issue.
However it does not affect oe-core as acl has yet to be updated to this
version and currently sits on the older 2.2.51 version.
---
meta/recipes-support/attr/attr_2.4.47.bb | 3 +
.../attr/files/attr-Missing-configure.ac.patch | 62 ++++++++++++++++++++
2 files changed, 65 insertions(+)
create mode 100644 meta/recipes-support/attr/files/attr-Missing-configure.ac.patch
diff --git a/meta/recipes-support/attr/attr_2.4.47.bb b/meta/recipes-support/attr/attr_2.4.47.bb
index a1f2e05..ad40c9b 100644
--- a/meta/recipes-support/attr/attr_2.4.47.bb
+++ b/meta/recipes-support/attr/attr_2.4.47.bb
@@ -1,5 +1,8 @@
require attr.inc
+# configure.ac was missing from the release tarball. This should be fixed in
+# future releases of attr, remove this when updating the recipe.
+SRC_URI_append += "file://attr-Missing-configure.ac.patch"
SRC_URI[md5sum] = "84f58dec00b60f2dc8fd1c9709291cc7"
SRC_URI[sha256sum] = "25772f653ac5b2e3ceeb89df50e4688891e21f723c460636548971652af0a859"
diff --git a/meta/recipes-support/attr/files/attr-Missing-configure.ac.patch b/meta/recipes-support/attr/files/attr-Missing-configure.ac.patch
new file mode 100644
index 0000000..5887ed5
--- /dev/null
+++ b/meta/recipes-support/attr/files/attr-Missing-configure.ac.patch
@@ -0,0 +1,62 @@
+Subject: [PATCH] attr: Missing configure.ac
+
+Upstream-Status: Backport [Upstream released tarball missing this file]
+---
+ configure.ac | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
+ 1 file changed, 48 insertions(+)
+ create mode 100644 configure.ac
+
+diff --git a/configure.ac b/configure.ac
+new file mode 100644
+index 0000000..b966d0e
+--- /dev/null
++++ b/configure.ac
+@@ -0,0 +1,48 @@
++# Copyright (C) 2009 Andreas Gruenbacher <agruen@suse.de>
++#
++# This program is free software: you can redistribute it and/or modify it
++# under the terms of the GNU General Public License as published by
++# the Free Software Foundation, either version 2 of the License, or
++# (at your option) any later version.
++#
++# This program is distributed in the hope that it will be useful,
++# but WITHOUT ANY WARRANTY; without even the implied warranty of
++# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
++# GNU General Public License for more details.
++#
++# You should have received a copy of the GNU General Public License
++# along with this program. If not, see <http://www.gnu.org/licenses/>.
++#
++AC_INIT(include/attributes.h)
++AC_CONFIG_AUX_DIR([.])
++AC_CONFIG_MACRO_DIR([m4])
++AC_CONFIG_HEADER(include/config.h)
++AC_PREFIX_DEFAULT(/usr)
++
++AC_PROG_LIBTOOL
++
++AC_ARG_ENABLE(shared,
++[ --enable-shared=[yes/no] Enable use of shared libraries [default=yes]],,
++ enable_shared=yes)
++AC_SUBST(enable_shared)
++
++AC_ARG_ENABLE(gettext,
++[ --enable-gettext=[yes/no] Enable alternate language support [default=yes]],,
++ enable_gettext=yes)
++AC_SUBST(enable_gettext)
++
++AC_ARG_ENABLE(lib64,
++[ --enable-lib64=[yes/no] Enable lib64 support [default=no]],,
++ enable_lib64=no)
++AC_SUBST(enable_lib64)
++
++AC_PACKAGE_GLOBALS(attr)
++AC_PACKAGE_UTILITIES(attr)
++AC_MANUAL_FORMAT
++AC_MULTILIB($enable_lib64)
++
++AC_C_CONST
++AC_TYPE_MODE_T
++AC_FUNC_ALLOCA
++
++AC_OUTPUT(include/builddefs)
--
1.7.9.5
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-12-03 4:02 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-11-25 8:18 [PATCH] attr: attr_2.4.47 release tarball missing configure.ac Nathan Rossi
2013-12-02 20:00 ` Saul Wold
2013-12-03 1:08 ` Nathan Rossi
2013-12-03 4:02 ` Saul Wold
2013-11-26 0:58 Nathan Rossi
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.