- 01 Aug, 2017 1 commit
-
-
Thomas Petazzoni authored
Both our target and host gdb depend on ncurses (host-ncurses for host-gdb, of course). However, while for the target we passs --with-curses, we are not doing this for the host variant. Due to this, host-gdb default to using the termcap library: if such a library is available on the build system, it will be used instead of the host-ncurses we have built. This causes the host gdb binary to depend on a library that we do not provide in $(HOST_DIR), breaking the principle of a standalone SDK (which should only depend on the C library). To solve this, we simply pass --with-curses in HOST_GDB_CONF_OPTS, which forces host-gdb to use the host-ncurses library. Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Reviewed-by:
Romain Naour <romain.naour@gmail.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 09 Jul, 2017 2 commits
-
-
Yann E. MORIN authored
It can be interesting to get the overlay from a remote server, rather than expect it to be present locally. Since that file can be any URL, we can't know its hash, so we just exclude it. Signed-off-by:
"Yann E. MORIN" <yann.morin.1998@free.fr> Signed-off-by:
Max Filippov <jcmvbkbc@gmail.com> [Thomas: use DL_DIR instead of BR2_DL_DIR.] Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
Yann E. MORIN authored
currently, specifying a custom Xtrensa core is done with two variables: - the core name - the directory containing the overlay tarball However, the core name only serves to construct the tarball name, and is not used whatsoever to configure any of the toolchain components (binutils, gcc or gdb), except through the files that are overlayed in their respective source trees. This has two main drawbacks: - the overlay file must be named after the core, - the tarball can not be compressed. Furthermore, it also makes it extremely complex to implement a download of that tarball. So, those two variables can be squeezed into a single variable, that is the complete path of the overlay tarball. Update the qemu-xtensa defconfig accordingly. Note: we do not add a legacy entry for BR2_XTENSA_CORE_NAME, since it was previously a blind option in the last release, and there's been no release since we removed BR2_XTENSA_CUSTOM_NAME. So, we just update the legacy comments for BR2_XTENSA_CUSTOM_NAME, since that's all the user could have seen in any of our releases so far. Signed-off-by:
"Yann E. MORIN" <yann.morin.1998@free.fr> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar> Signed-off-by:
Max Filippov <jcmvbkbc@gmail.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 05 Jul, 2017 2 commits
-
-
Arnout Vandecappelle authored
Since things are no longer installed in $(HOST_DIR)/usr, the callers should also not refer to it. This is a mechanical change with git grep -l '$(HOST_DIR)/usr/bin' | xargs sed -i 's%$(HOST_DIR)/usr/bin%$(HOST_DIR)/bin%g' Signed-off-by:
Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
Arnout Vandecappelle authored
Signed-off-by:
Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Reviewed-by:
Romain Naour <romain.naour@smile.fr> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 01 Apr, 2017 4 commits
-
-
Rahul Bedarkar authored
We want to use SPDX identifier for license string as much as possible. SPDX short identifier for LGPLv2/LGPLv2+ is LGPL-2.0/LGPL-2.0+. This change is done using following command. find . -name "*.mk" | xargs sed -ri '/LICENSE( )?[\+:]?=/s/LGPLv2(\+)?/LGPL-2 .0\1/g' Signed-off-by:
Rahul Bedarkar <rahulbedarkar89@gmail.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
Rahul Bedarkar authored
We want to use SPDX identifier for license string as much as possible. SPDX short identifier for LGPLv3/LGPLv3+ is LGPL-3.0/LGPL-3.0+. This change is done using following command. find . -name "*.mk" | xargs sed -ri '/LICENSE( )?[\+:]?=/s/LGPLv3(\+)?/LGPL-3 .0\1/g' Signed-off-by:
Rahul Bedarkar <rahulbedarkar89@gmail.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
Rahul Bedarkar authored
We want to use SPDX identifier for license string as much as possible. SPDX short identifier for GPLv3/GPLv3+ is GPL-3.0/GPL-3.0+. This change is done using following command. find . -name "*.mk" | xargs sed -ri '/LICENSE( )?[\+:]?=/s/\<GPLv3\>/GPL-3 .0/g' Signed-off-by:
Rahul Bedarkar <rahulbedarkar89@gmail.com> Acked-by:
Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
Rahul Bedarkar authored
We want to use SPDX identifier for license strings as much as possible. SPDX short identifier for GPLv2/GPLv2+ is GPL-2.0/GPL-2.0+. This change is done by using following command. find . -name "*.mk" | xargs sed -ri '/LICENSE( )?[\+:]?=/s/\<GPLv2\>/GPL-2 .0/g' Signed-off-by:
Rahul Bedarkar <rahulbedarkar89@gmail.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 28 Mar, 2017 1 commit
-
-
Arnout Vandecappelle authored
Starting with glibc 2.25, the proc_service.h header has been copied from gdb to glibc so other tools can use it. However, that makes it necessary to make sure that declaration of prfpregset_t declaration is consistent between gdb and glibc. In gdb, however, there is a workaround for a broken prfpregset_t declaration in glibc 2.3 which uses AC_TRY_RUN to detect if it's needed, which doesn't work in cross-compilation. So pass the cache option to configure. It needs to be passed to GDB_CONF_ENV to build gdbserver only but also to GDB_MAKE_ENV, because otherwise it does not get passed to the configure script of nested packages while building gdbserver with full debugger. Signed-off-by:
Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 26 Mar, 2017 3 commits
-
-
Yann E. MORIN authored
... like it is done for gcc. Signed-off-by:
"Yann E. MORIN" <yann.morin.1998@free.fr> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by:
Max Filippov <jcmvbkbc@gmail.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
Yann E. MORIN authored
Signed-off-by:
"Yann E. MORIN" <yann.morin.1998@free.fr> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by:
Max Filippov <jcmvbkbc@gmail.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
Yann E. MORIN authored
... instead of re-computing them over-and-over-again. Signed-off-by:
"Yann E. MORIN" <yann.morin.1998@free.fr> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by:
Max Filippov <jcmvbkbc@gmail.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 18 Mar, 2017 1 commit
-
-
Romain Naour authored
The previous fix [1] reinroduced the issue fixed by [2]. So keep gl_cv_func_gettimeofday_clobber=no in GDB_CONF_ENV and GDB_MAKE_ENV. Fixes: http://autobuild.buildroot.net/results/ec6/ec60d436bd34a4f37b664e4124d7f0c96e90a1be [1] faf38b78 [2] 560334bb Signed-off-by:
Romain Naour <romain.naour@gmail.com> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 12 Mar, 2017 1 commit
-
-
Romain Naour authored
The commit [1] doesn't fix this issue for all cases (it doesn't work when BR2_PACKAGE_GDB_DEBUGGER=y). So, leave the configure script alone and override gl_cv_func_gettimeofday_clobber from GDB_MAKE_ENV. [1] 560334bb Fixes: http://autobuild.buildroot.net/results/719/719a441421030b79c0aa1bbfb707130f3ac87338 Signed-off-by:
Romain Naour <romain.naour@gmail.com> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 23 Jan, 2017 1 commit
-
-
Peter Korsgaard authored
gdb 7.12+ by default builds with a C++ compiler, which naturally doesn't work when we don't have C++ support in the toolchain. Fix it by passing --disable-build-with-cxx for such setups. Signed-off-by:
Peter Korsgaard <peter@korsgaard.com>
-
- 15 Nov, 2016 1 commit
-
-
Zakharov Vlad authored
gdb arc-2016.09-rc1 is based on upstream 7.12 gdb version where we can still disable C++ build. For more information take a look at: https://sourceware.org/gdb/wiki/cxx-conversion We wanted to build gdb with C++ as it is used as default in 7.12 and is even a mandatory requirement since 2016/09. But unfortunately we missed one runtime failure that C++ build causes and so now we have to disable C++ build. When gdb arc-2016.09-rc1 is built as a C++ application it segfaults at runtime for ARC. We are going to fix the issue in 2017.03 ARC toolchain release. and before this we remove C++ dependency are adding temporary workaround to prevent runtime segfaults. The workaround is to disable building gdb as C++ application via passing --disable-build-with-cxx config option when building gdb for ARC. Signed-off-by:
Vlad Zakharov <vzakhar@synopsys.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 14 Oct, 2016 1 commit
-
-
Peter Korsgaard authored
Commit 12306a81 (gdb: get rid of host-texinfo dependency) tried to append MAKEINFO=true to the host/target make install arguments, but as the default values for these are only added when (host-)autotools-package is evaluated (and only if empty), this effectively drops the default values and we ended up without the 'install' target and nothing got installed. To fix this, specify the full install arguments. Signed-off-by:
Peter Korsgaard <peter@korsgaard.com>
-
- 09 Oct, 2016 1 commit
-
-
Thomas De Schampheleire authored
The host-texinfo dependency in gdb/host-gdb was added because gdb insisted in building info documentation in recent versions, and we want to avoid 'makeinfo' to be present on the build system. However, there is another solution that does not require actually building host-texinfo: instruct the makefiles to use a dummy makeinfo command ('true'). Signed-off-by:
Thomas De Schampheleire <thomas.de.schampheleire@gmail.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 24 Jul, 2016 1 commit
-
-
Romain Naour authored
Starting with gdb 7.11, the bundled gnulib want to use rpl_gettimeofday (gettimeofday replacement) due bad guessing when cross-compiling with musl and uClibc toolchains. Instead of patching gnulib configure script to fix the test, use gl_cv_func_gettimeofday_clobber=no to not use rpl_gettimeofday. Thanks to Thomas Petazzoni to suggest this patch. Reported upstream: https://sourceware.org/bugzilla/show_bug.cgi?id=19798 Signed-off-by:
Romain Naour <romain.naour@gmail.com> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 06 Jul, 2016 1 commit
-
-
Romain Naour authored
Starting with gdb 7.10, gdb wants to re-generate its documentation. We were trying to avoid that by patching the Makefiles, but it wasn't working in all situations. So, we simply add a dependency on host-texinfo in all case. Fixes: http://autobuild.buildroot.net/results/f72/f72eac3e2f995e93f0e8f215e68ce4356a696dcb Signed-off-by:
Romain Naour <romain.naour@gmail.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 05 Jul, 2016 2 commits
-
-
Thomas Petazzoni authored
gdb versions fetched from git (such as the ARC version) will not build with host-flex and host-bison. Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Reviewed-by:
Romain Naour <romain.naour@gmail.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
Thomas Petazzoni authored
Just like binutils, when gdb is fetched from git, it wants to re-generate its documentation. We were trying to avoid that by patching the Makefiles, but it wasn't working in all situations. So, we simply add a dependency on host-texinfo when gdb is fetched from git, and remove our hackhish work-around. Fixes: http://autobuild.buildroot.net/results/80e63edc009d23cb939880a6f5ae2f220f07a96a/ Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Reviewed-by:
Romain Naour <romain.naour@gmail.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 31 May, 2016 1 commit
-
-
Waldemar Brodkorb authored
Add an option to enable GDB simulator support, so it can be used to test some no-MMU systems. Newer GDB Blackfin Simulator need a workaround for in-tree compile. It doesn't break for older Buildroot supported GDB Blackfin Simulator. Signed-off-by:
Waldemar Brodkorb <wbx@openadk.org> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 15 Mar, 2016 1 commit
-
-
Gustavo Zacarias authored
It's been deprecated for a year now so remove it. Signed-off-by:
Gustavo Zacarias <gustavo@zacarias.com.ar> Signed-off-by:
Peter Korsgaard <peter@korsgaard.com>
-
- 13 Mar, 2016 1 commit
-
-
Vicente Olivert Riera authored
As reported in... https://bugs.busybox.net/show_bug.cgi?id=8786 ..., gdb fails to build with xz and expat support at the same time when using an external toolchain. This is how the problem is shown: checking for liblzma... no configure: error: missing liblzma for --with-lzma And having a deeper look in gdb/config.log we can se a message like this one: /lib64/liblzma.so.5: undefined reference to `clock_gettime@GLIBC_2.17' As Thomas said in the bug report quoted above, that error is related to the fact that it find the host's lzma. Adding --with-lzma-prefix=$(STAGING_DIR)/usr to GDB_CONF_OPTS fixes the problem. Credit-to: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by:
Vicente Olivert Riera <Vincent.Riera@imgtec.com> Signed-off-by:
Peter Korsgaard <peter@korsgaard.com>
-
- 16 Feb, 2016 1 commit
-
-
Bernd Kuhls authored
When xz was compiled before, gdb will use it as optional dependency: $ output/host/usr/bin/i586-buildroot-linux-uclibc-readelf -a output/target/usr/bin/gdb | grep NEEDED 0x00000001 (NEEDED) Shared library: [libdl.so.1] 0x00000001 (NEEDED) Shared library: [libncurses.so.5] 0x00000001 (NEEDED) Shared library: [libz.so.1] 0x00000001 (NEEDED) Shared library: [libm.so.1] 0x00000001 (NEEDED) Shared library: [liblzma.so.5] 0x00000001 (NEEDED) Shared library: [libc.so.1] Signed-off-by:
Bernd Kuhls <bernd.kuhls@t-online.de> Reviewed-by:
Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 15 Dec, 2015 1 commit
-
-
Gustavo Zacarias authored
Signed-off-by:
Gustavo Zacarias <gustavo@zacarias.com.ar> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 03 Oct, 2015 1 commit
-
-
Vicente Olivert Riera authored
Our previous patch to disable the documentation in gdb wasn't complete. There are cases where the documentation under bfd and gprof directories is being built, causing the subsequent failure due to missing makeinfo. This patch fixes that problem. Fixes: http://autobuild.buildroot.org/results/244/2442e697d8a300496434fd42fcb1ee3941d13e06/ Signed-off-by:
Vicente Olivert Riera <Vincent.Riera@imgtec.com> Signed-off-by:
Peter Korsgaard <peter@korsgaard.com>
-
- 11 Sep, 2015 1 commit
-
-
Vicente Olivert Riera authored
Force gdb to not build the documentation. This way we avoid depending on host-texinfo. This is a temporary fix until upstream accepts a proposed --disable-docs configure option. Since the documentation will not be build at all, we can remove the parts related to host-texinfo and MAKEINFO in the gdb.mk file. Fixes: http://autobuild.buildroot.net/results/dd5/dd50ed99abb2c8495def826866b184030953f90e/ Signed-off-by:
Vicente Olivert Riera <Vincent.Riera@imgtec.com> Tested-by:
Brendan Heading <brendanheading@gmail.com> Acked-by:
Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 21 Jun, 2015 1 commit
-
-
Doug Kehn authored
Commit 23413b51 added --with-expat configure option and expat dependency when BR2_PACKAGE_EXPAT is configured. When cross-compiling, gdb configure fails because the host system libexpat is referenced. Configuring gdb with --with-libexpat-prefix=$(STAGING_DIR)/usr fixes the reference and allows gdb to build successfully. Tested with arm-buildroot-linux-gnueabihf toolchain. Signed-off-by:
Doug Kehn <rdkehn@yahoo.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 12 Jun, 2015 1 commit
-
-
Bernd Kuhls authored
To allow reproducable builds of target gdb add some optional dependencies $ output/host/usr/bin/i586-buildroot-linux-uclibc-readelf -a output/target/usr/bin/gdb | grep NEEDED 0x00000001 (NEEDED) Shared library: [libdl.so.0] 0x00000001 (NEEDED) Shared library: [libncurses.so.5] 0x00000001 (NEEDED) Shared library: [libz.so.1] 0x00000001 (NEEDED) Shared library: [libm.so.0] 0x00000001 (NEEDED) Shared library: [libiconv.so.2] 0x00000001 (NEEDED) Shared library: [libpthread.so.0] 0x00000001 (NEEDED) Shared library: [libutil.so.0] 0x00000001 (NEEDED) Shared library: [libpython2.7.so.1.0] 0x00000001 (NEEDED) Shared library: [libexpat.so.1] 0x00000001 (NEEDED) Shared library: [libc.so.0] [Thomas: explicitly use --{with,without}-{expat,zlib}. There is no such option for libiconv, so for libiconv, the code was left as is.] Signed-off-by:
Bernd Kuhls <bernd.kuhls@t-online.de> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 31 Mar, 2015 1 commit
-
-
Jerzy Grzegorek authored
This commit doesn't touch infra packages. Signed-off-by:
Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 29 Mar, 2015 1 commit
-
-
Gustavo Zacarias authored
Add gdb version 7.9, and adjust the tarball logic to default to .tar.xz (for 7.8 & 7.9), fall back to .tar.bz2 for 7.7.x and leave the rest alone. Signed-off-by:
Gustavo Zacarias <gustavo@zacarias.com.ar> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 16 Mar, 2015 1 commit
-
-
Alexey Brodkin authored
Currently we only add "host-texinfo" dependency for host GDB built from git sources. But in case of target gdb texinfo won't be built and so gdb building will fail on attempt to build documentation. Fix is trivial - add "host-texinfo" as a dependency for target gdb in the sme way as we disable documentation building via both GDB_CONF_ENV (for target) and HOST_GDB_CONF_ENV (host) GDB flavors. Signed-off-by:
Alexey Brodkin <abrodkin@synopsys.com> Cc: Anton Kolesov <akolesov@synopsys.com> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Cc: Peter Korsgaard <peter@korsgaard.com> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 14 Feb, 2015 1 commit
-
-
Yann E. MORIN authored
Signed-off-by:
"Yann E. MORIN" <yann.morin.1998@free.fr> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 29 Dec, 2014 1 commit
-
-
Thomas Petazzoni authored
This commit bumps to 7.8.1, and updates the logic to use .tar.xz tarballs so that it applies to all 7.8.x versions. Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by:
Peter Korsgaard <peter@korsgaard.com>
-
- 21 Dec, 2014 1 commit
-
-
Romain Naour authored
The shared build only of gdb is actually not supported. When --disable-static is given to the gdb's main configure script, it also disable the static build of bundled libraries like bfd, readline and opcodes, which must be build statically. Force the static build of bundled libraries by adding --enable-static in GDB_CONF_OPTS. Fixes: http://autobuild.buildroot.net/results/219/21979e730dca1fce5bdda9a4c7fad4485e788866/ http://autobuild.buildroot.net/results/c10/c1096d0bd22de5c6feba848f743601ad0416a944/ http://autobuild.buildroot.net/results/d8a/d8a5bcc7fa374fb0c916a9d0f33ef283109cb404/ http://autobuild.buildroot.net/results/22a/22a86d0f1df0fc4698c0f734f3d659e6317404a4/ http://autobuild.buildroot.net/results/404/404e61e5c30040ee5756f4b5839149dca38660d4/ And many more. Signed-off-by:
Romain Naour <romain.naour@openwide.fr> Acked-by:
"Yann E. MORIN" <yann.morin.1998@free.fr> Signed-off-by:
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-
- 01 Dec, 2014 1 commit
-
-
Peter Korsgaard authored
Fixes http://autobuild.buildroot.net/results/ef4/ef487b495e57f098af0fe9508d430b56b0750339/ Explicitly pass the path to host-python to ensure it gets used. Signed-off-by:
Peter Korsgaard <peter@korsgaard.com>
-
- 07 Oct, 2014 1 commit
-
-
Thomas De Schampheleire authored
The Buildroot coding style defines one space around make assignments and does not align the assignment symbols. This patch does a bulk fix of offending packages. The package infrastructures (or more in general assignments to calculated variable names, like $(2)_FOO) are not touched. Alignment of line continuation characters (\) is kept as-is. The sed command used to do this replacement is: find * -name "*.mk" | xargs sed -i \ -e 's#^\([A-Z0-9a-z_]\+\)\s*\([?:+]\?=\)\s*$#\1 \2#' -e 's#^\([A-Z0-9a-z_]\+\)\s*\([?:+]\?=\)\s*\([^\\]\+\)$#\1 \2 \3#' -e 's#^\([A-Z0-9a-z_]\+\)\s*\([?:+]\?=\)\s*\([^\\ \t]\+\s*\\\)\s*$#\1 \2 \3#' -e 's#^\([A-Z0-9a-z_]\+\)\s*\([?:+]\?=\)\(\s*\\\)#\1 \2\3#' Brief explanation of this command: ^\([A-Z0-9a-z_]\+\) a regular variable at the beginning of the line \([?:+]\?=\) any assignment character =, :=, ?=, += \([^\\]\+\) any string not containing a line continuation \([^\\ \t]\+\s*\\\) string, optional whitespace, followed by a line continuation character \(\s*\\\) optional whitespace, followed by a line continuation character Hence, the first subexpression handles empty assignments, the second handles regular assignments, the third handles regular assignments with line continuation, and the fourth empty assignments with line continuation. This expression was tested on following test text: (initial tab not included) FOO = spaces before FOO = spaces before and after FOO = tab before FOO = tab and spaces before FOO = tab after FOO = tab and spaces after FOO = spaces and tab after FOO = \ FOO = bar \ FOO = bar space \ FOO = \ GENIMAGE_DEPENDENCIES = host-pkgconf libconfuse FOO += spaces before FOO ?= spaces before and after FOO := FOO = FOO = FOO = FOO = $(MAKE1) CROSS_COMPILE=$(TARGET_CROSS) -C AT91BOOTSTRAP3_DEFCONFIG = \ AXEL_DISABLE_I18N=--i18n=0 After this bulk change, following manual fixups were done: - fix line continuation alignment in cegui06 and spice (the sed expression leaves the number of whitespace between the value and line continuation character intact, but the whitespace before that could have changed, causing misalignment. - qt5base was reverted, as this package uses extensive alignment which actually makes the code more readable. Finally, the end result was manually reviewed. Signed-off-by:
Thomas De Schampheleire <thomas.de.schampheleire@gmail.com> Cc: Yann E. Morin <yann.morin.1998@free.fr> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by:
Peter Korsgaard <peter@korsgaard.com>
-