| Summary: | toolchain eclipse register : fails with a custom BR2_HOST_DIR | ||
|---|---|---|---|
| Product: | buildroot | Reporter: | Frederic MATHIEU <frederic.mathieu> |
| Component: | Other | Assignee: | Arnout Vandecappelle <arnout> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | buildroot |
| Priority: | P5 | ||
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Host: | Target: | ||
| Build: | |||
|
Description
Frederic MATHIEU
2017-08-03 09:25:50 UTC
Indeed, the eclipse-register-toolchain script doesn't seem to support a non-default BR2_HOST_DIR. A simple solution would be to symlink host to BR2_HOST_DIR if it's not the default. Any objections to that? I'm not really happy to add this host symlink just for the sake of making the eclipse-register-toolchain script work. Instead, we should pass $(HOST_DIR) as argument to eclipse-register-toolchain. I agree with Thomas, it is better to solve the problem at its source, ie the main makefile. (In reply to Thomas Petazzoni from comment #2) > I'm not really happy to add this host symlink just for the sake > of making the eclipse-register-toolchain script work. It's more that scripts in general (e.g. post-build scripts) may assume that $(O)/host exists, and it just simplifies life in general if you can rely on it being there. A bit like the staging symlink. Note that things *will* break dramatically if you change BR2_HOST_DIR and try to rebuild. So there can be no argument of "the symlink could dangle". > Instead, we should pass $(HOST_DIR) as argument to eclipse-register-toolchain. HOST_DIR is already exported, no need to add an argument. However, if you look at the script, you'll see that it just checks that ${project_directory}/host exists, but this directory is *not* encoded in the eclipse toolchains file. Since there is apparently no way to pass BR2_HOST_DIR to eclipse, I guess a non-default BR2_HOST_DIR can't be supported without changing both the eclipse plugin and the toolchains file. That, in turn, requires a synchronised update, and old Buildroots won't run with the new eclipse plugin and vice versa... In short: I propose to add the symlink :-) http://patchwork.ozlabs.org/patch/797910/ Fixed by commit https://git.buildroot.org/buildroot/commit/Makefile?id=91b4a4525bf30a03c1f2542b3c4d340e5e4ca42a. Thanks! |