Summary: | dev-libs/libgpg-error-1.32-r1 with gawk-5: fatal: cannot use gawk builtin `namespace' as variable name | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Patrick Lauer <patrick> |
Component: | Current packages | Assignee: | Lars Wendler (Polynomial-C) (RETIRED) <polynomial-c> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | anarchy, axl, base-system, hydrapolic, jarymut+gentoo.bugs, l-1649a, matoro_bugzilla_gentoo, zlogene |
Priority: | Normal | Keywords: | PATCH, UPSTREAM |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://dev.gnupg.org/T4459 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
libgpg-error-1.36-gawk5_namespace_reserved_word.patch
libgpg-error-1.36-gawk5_namespace_reserved_word.part2.patch upstream patch with additional Makefile.in fix Version bump along with upstream patch directly. version bump with gawk-5 upstream path version 2 |
Description
Patrick Lauer
2019-04-13 16:42:33 UTC
Created attachment 572680 [details, diff]
libgpg-error-1.36-gawk5_namespace_reserved_word.patch
Possible fix. Simply renamed all occurences of "namespace" with "libgpg_error_namespace". The patch requires an additional call from autotools because it changes one Makefile.am file.
FYI, I patched libgpg-error-1.36 which is the latest available release.
Downgrading to sys-apps/gawk-4.2.1-r1 and rebuilding dev-libs/libgpg-error-1.32-r1 succeeds. Created attachment 573002 [details, diff]
libgpg-error-1.36-gawk5_namespace_reserved_word.part2.patch
I can build it without problems after droping previous patch together with mine into /etc/portage/patches.
So thanks!
Thanks, is there an open upstream bug report on https://dev.gnupg.org for this issue? Yes, there is (see "See Also") (In reply to Lars Wendler (Polynomial-C) from comment #5) > Yes, there is (see "See Also") Thanks, I see it is also accepted upstream, so feel free to patch it (In reply to Kristian Fiskerstrand from comment #6) > (In reply to Lars Wendler (Polynomial-C) from comment #5) > > Yes, there is (see "See Also") > > Thanks, I see it is also accepted upstream, so feel free to patch it Eh, the upstream fix differs from my patch. According to dilfridge the upstream patch does still have some issues (breaks gawk-4 and/or doesn't apply cleanly to the libgpg-error versions in our tree). He seems to like mine better :) (In reply to Kristian Fiskerstrand from comment #6) > (In reply to Lars Wendler (Polynomial-C) from comment #5) > > Yes, there is (see "See Also") > > Thanks, I see it is also accepted upstream, so feel free to patch it Upstream's stuff doesnt build, neither with gawk 4 nor gawk 5. Ours does. https://gitweb.gentoo.org/proj/riscv.git/tree/dev-libs/libgpg-error (In reply to Andreas K. Hüttel from comment #8) > (In reply to Kristian Fiskerstrand from comment #6) > > (In reply to Lars Wendler (Polynomial-C) from comment #5) > > > Yes, there is (see "See Also") > > > > Thanks, I see it is also accepted upstream, so feel free to patch it > > Upstream's stuff doesnt build, neither with gawk 4 nor gawk 5. > Ours does. > > https://gitweb.gentoo.org/proj/riscv.git/tree/dev-libs/libgpg-error The real place for discussing that is really in the upstream bug, once it is upstreamed it can be included in our downstream. (In reply to Kristian Fiskerstrand from comment #9) > (In reply to Andreas K. Hüttel from comment #8) > > (In reply to Kristian Fiskerstrand from comment #6) > > > (In reply to Lars Wendler (Polynomial-C) from comment #5) > > > > Yes, there is (see "See Also") > > > > > > Thanks, I see it is also accepted upstream, so feel free to patch it > > > > Upstream's stuff doesnt build, neither with gawk 4 nor gawk 5. > > Ours does. > > > > https://gitweb.gentoo.org/proj/riscv.git/tree/dev-libs/libgpg-error > > The real place for discussing that is really in the upstream bug, once it is > upstreamed it can be included in our downstream. That's actually the job of the maintaining team, which should get moving. Since for weeks already this is the only package in ~arch which fails to build. And you wanna leave dev-libs/libgpg-error being broken in ~arch until upstream finally fixes this? Why not adding some intermediate fix at least? Created attachment 573756 [details, diff]
upstream patch with additional Makefile.in fix
Please test and apply, I have successfully used on both musl and glibc
Created attachment 573760 [details, diff]
Version bump along with upstream patch directly.
This is the version bump along with the upstream patch. We simply call eautomake to force the regeneration of src/Makefile.in which will fix the namespace not being updated by upstream patch.
The patches work. Thank you. According to https://dev.gnupg.org/T4459: Added a fix to GnuPG, too (master and stable 2.2). (In reply to Tomáš Mózes from comment #15) > According to https://dev.gnupg.org/T4459: > Added a fix to GnuPG, too (master and stable 2.2). The gnupg fix is same fix and only added for those who do not use libgpg-error as a standalone. The gnupg tree is fine in gentoo with gawk5 as we use libgpg-error and do not worry about what is bundled. Created attachment 573842 [details, diff]
version bump with gawk-5 upstream path version 2
This is based on actually gentoo tree instead of my overlay. It removes the libtoolize call and makes it eautoreconf as ~arch users are gonna be on automake-1.16 which will force a full rebuilt of autotools anyway.
K_F granted permission for me to push the update along with the upstream patch. I am leaving bug open for crypto team to really finalize the solution they want and to ensure they remember to swap back to libtoolize next release which will include the upstream patch directly. Thanks. (In reply to Jory A. Pratt from comment #18) > K_F granted permission for me to push the update along with the upstream > patch. I am leaving bug open for crypto team to really finalize the solution > they want and to ensure they remember to swap back to libtoolize next > release which will include the upstream patch directly. So, this issue is long-fixed, but the eautoreconf is retained for the remove_broken_check patch for now (unrelated, https://gitweb.gentoo.org/repo/gentoo.git/tree/dev-libs/libgpg-error/files/libgpg-error-1.44-remove_broken_check.patch) |