From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Giulio Benetti <giulio.benetti@benettiengineering.com>,
Romain Naour <romain.naour@gmail.com>,
Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>,
buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH] Makefile, docs/manual, support, toolchain: remove Eclipse integration
Date: Sat, 30 Jul 2022 12:34:10 +0200 [thread overview]
Message-ID: <20220730103410.GO3168@scaer> (raw)
In-Reply-To: <20220727190511.2859278-1-thomas.petazzoni@bootlin.com>
Thomas, All,
On 2022-07-27 21:05 +0200, Thomas Petazzoni via buildroot spake thusly:
> Back many years ago, we developed an Eclipse plugin that simplified
> the usage of Buildroot toolchains. Enabling the BR2_ECLIPSE_REGISTER=y
> was registering the Buildroot toolchain into a special file in your
> HOME folder that the Eclipse plugin would recognize to allow to
> directly use the Buildroot cross-compiler.
>
> This Eclipse plugin has not been maintained for years. The last commit
> in the repository dates back from September 2017. Since then Eclipse
> has moved on, and the plugin is no longer compatible with current
> versions of Eclipse.
>
> Also, Eclipse is probably no longer that widely used in the embedded
> Linux space, as other more modern IDEs have become more popular.
>
> All in all, it's time to say good bye to this Eclipse integration.
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Applied to master, thanks.
Regards,
Yann E. MORIN.
> ---
> Config.in.legacy | 9 +++
> Makefile | 8 ---
> docs/manual/common-usage.txt | 2 -
> docs/manual/eclipse-integration.txt | 30 ---------
> support/scripts/eclipse-register-toolchain | 76 ----------------------
> toolchain/Config.in | 7 --
> 6 files changed, 9 insertions(+), 123 deletions(-)
> delete mode 100644 docs/manual/eclipse-integration.txt
> delete mode 100755 support/scripts/eclipse-register-toolchain
>
> diff --git a/Config.in.legacy b/Config.in.legacy
> index 24b4236ade..80c443d9fb 100644
> --- a/Config.in.legacy
> +++ b/Config.in.legacy
> @@ -146,6 +146,15 @@ endif
>
> comment "Legacy options removed in 2022.08"
>
> +config BR2_ECLIPSE_REGISTER
> + bool "Eclipse integration removed"
> + select BR2_LEGACY
> + help
> + The Buildroot integration with the Eclipse IDE has been
> + removed, as the corresponding Eclipse plugin is no longer
> + maintained, and is no longer usable with current versions of
> + Eclipse.
> +
> config BR2_csky
> bool "csky architecture removed"
> select BR2_LEGACY
> diff --git a/Makefile b/Makefile
> index 7d2081b98e..0993417ae0 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -646,14 +646,6 @@ STRIP_FIND_SPECIAL_LIBS_CMD = \
> \( -name 'ld-*.so*' -o -name 'libpthread*.so*' \) \
> -print0
>
> -ifeq ($(BR2_ECLIPSE_REGISTER),y)
> -define TOOLCHAIN_ECLIPSE_REGISTER
> - ./support/scripts/eclipse-register-toolchain `readlink -f $(O)` \
> - $(notdir $(TARGET_CROSS)) $(BR2_ARCH)
> -endef
> -TARGET_FINALIZE_HOOKS += TOOLCHAIN_ECLIPSE_REGISTER
> -endif
> -
> # Generate locale data.
> ifeq ($(BR2_TOOLCHAIN_USES_GLIBC),y)
> GLIBC_GENERATE_LOCALES = $(call qstrip,$(BR2_GENERATE_LOCALE))
> diff --git a/docs/manual/common-usage.txt b/docs/manual/common-usage.txt
> index 9ba87a8339..9abd3799dd 100644
> --- a/docs/manual/common-usage.txt
> +++ b/docs/manual/common-usage.txt
> @@ -417,6 +417,4 @@ effects:
> be empty and it's only at the very end of the build that they will
> be populated.
>
> -include::eclipse-integration.txt[]
> -
> include::advanced.txt[]
> diff --git a/docs/manual/eclipse-integration.txt b/docs/manual/eclipse-integration.txt
> deleted file mode 100644
> index 1f726d8240..0000000000
> --- a/docs/manual/eclipse-integration.txt
> +++ /dev/null
> @@ -1,30 +0,0 @@
> -// -*- mode:doc; -*-
> -// vim: set syntax=asciidoc:
> -
> -=== Integration with Eclipse
> -
> -While a part of the embedded Linux developers like classical text
> -editors like Vim or Emacs, and command-line based interfaces, a number
> -of other embedded Linux developers like richer graphical interfaces to
> -do their development work. Eclipse being one of the most popular
> -Integrated Development Environment, Buildroot integrates with Eclipse
> -in order to ease the development work of Eclipse users.
> -
> -Our integration with Eclipse simplifies the compilation, remote
> -execution and remote debugging of applications and libraries that are
> -built on top of a Buildroot system. It does not integrate the
> -Buildroot configuration and build processes themselves with
> -Eclipse. Therefore, the typical usage model of our Eclipse integration
> -would be:
> -
> -* Configure your Buildroot system with +make menuconfig+, +make
> - xconfig+ or any other configuration interface provided with
> - Buildroot.
> -* Build your Buildroot system by running +make+.
> -* Start Eclipse to develop, execute and debug your own custom
> - applications and libraries, that will rely on the libraries built
> - and installed by Buildroot.
> -
> -The Buildroot Eclipse integration installation process and usage is
> -described in detail at
> -https://github.com/mbats/eclipse-buildroot-bundle/wiki.
> diff --git a/support/scripts/eclipse-register-toolchain b/support/scripts/eclipse-register-toolchain
> deleted file mode 100755
> index 891d29ee6f..0000000000
> --- a/support/scripts/eclipse-register-toolchain
> +++ /dev/null
> @@ -1,76 +0,0 @@
> -#!/bin/sh
> -
> -# This script registers the toolchain of a Buildroot project into the
> -# Eclipse plugin. To do so, it adds a new line for the Buildroot
> -# toolchain into the $HOME/.buildroot-eclipse.toolchains file, which
> -# the Eclipse Buildroot plugin reads to discover automatically the
> -# available Buildroot toolchains on the system.
> -#
> -# This script should typically not be called manually. Instead, one
> -# should enable the BR2_ECLIPSE_REGISTER configuration option, which
> -# will lead Buildroot to automatically call this script with the
> -# appropriate arguments.
> -#
> -# Usage:
> -# eclipse-register-toolchain project-directory toolchain-prefix architecture
> -#
> -# project-directory is the absolute path to the Buildroot project
> -# output directory (which contains the host/, target/, build/,
> -# images/, etc. subdirectories). It should be an absolute and
> -# canonical path.
> -#
> -# toolchain-prefix is the prefix of the cross-compilation tools, i.e
> -# 'arm-linux-' if the cross-compiler executable is 'arm-linux-gcc'.
> -#
> -# architecture is the lower-cased name of the architecture targetted
> -# by the Buildroot project.
> -
> -if test $# -ne 3; then
> - echo "Invalid number of arguments."
> - echo "Usage: $0 project-directory toolchain-prefix architecture"
> - exit 1
> -fi
> -
> -project_directory=$1
> -toolchain_prefix=$2
> -architecture=$3
> -
> -if test ! -d ${project_directory} ; then
> - echo "Non-existing project directory ${project_directory}"
> - exit 1
> -fi
> -
> -if test ! -d ${project_directory}/host ; then
> - echo "Your project directory does not look like a Buildroot output"
> - exit 1
> -fi
> -
> -if test ! -e ${project_directory}/host/bin/${toolchain_prefix}gcc ; then
> - echo "Cannot find the cross-compiler in the project directory"
> - exit 1
> -fi
> -
> -TOOLCHAIN_ECLIPSE_FILE=${HOME}/.buildroot-eclipse.toolchains
> -
> -# First, we remove all lines from the ${TOOLCHAIN_ECLISPE_FILE} that
> -# correspond to toolchains that no longer exist.
> -if test -f ${TOOLCHAIN_ECLIPSE_FILE} ; then
> - mv ${TOOLCHAIN_ECLIPSE_FILE} ${TOOLCHAIN_ECLIPSE_FILE}.tmp
> - cat ${TOOLCHAIN_ECLIPSE_FILE}.tmp | while read toolchain ; do
> - path=$(echo ${toolchain} | cut -f1 -d ':')
> - # Filter lines corresponding to still existing projects
> - echo "Testing ${path} ..."
> - if ! test -d ${path} ; then
> - continue
> - fi
> - # .. and the current project
> - if test ${path} = ${project_directory} ; then
> - continue
> - fi
> - echo ${toolchain} >> ${TOOLCHAIN_ECLIPSE_FILE}
> - done
> - rm ${TOOLCHAIN_ECLIPSE_FILE}.tmp
> -fi
> -
> -# Add the toolchain
> -echo "${project_directory}:${toolchain_prefix}:${architecture}" >> ${TOOLCHAIN_ECLIPSE_FILE}
> diff --git a/toolchain/Config.in b/toolchain/Config.in
> index 5f79c07df0..05725394ea 100644
> --- a/toolchain/Config.in
> +++ b/toolchain/Config.in
> @@ -331,13 +331,6 @@ config BR2_TARGET_LDFLAGS
> Note that options with a '$' sign (eg.
> -Wl,-rpath='$ORIGIN/../lib') are not supported.
>
> -config BR2_ECLIPSE_REGISTER
> - bool "Register toolchain within Eclipse Buildroot plug-in"
> - help
> - This options tells Buildroot to generate the necessary
> - configuration files to make your toolchain appear within
> - Eclipse, through the Eclipse Buildroot plugin.
> -
> # Options for packages to depend on, if they require at least a
> # specific version of the kernel headers.
> # Toolchains should choose the adequate option (ie. the highest
> --
> 2.37.1
>
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
prev parent reply other threads:[~2022-07-30 10:34 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-27 19:05 [Buildroot] [PATCH] Makefile, docs/manual, support, toolchain: remove Eclipse integration Thomas Petazzoni via buildroot
2022-07-30 10:34 ` Yann E. MORIN [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220730103410.GO3168@scaer \
--to=yann.morin.1998@free.fr \
--cc=buildroot@buildroot.org \
--cc=giulio.benetti@benettiengineering.com \
--cc=romain.naour@gmail.com \
--cc=thomas.de_schampheleire@nokia.com \
--cc=thomas.petazzoni@bootlin.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox