Re: [U-Boot] [PATCH v2] doc: README.mxs: Add instruction to install 'libssl-dev'
Dear Fabio Estevam, From: Fabio Estevam fabio.este...@freescale.com Since commit bce883707 (ARM: mxs: tools: Add mkimage support for MXS bootstream) the following build error is seen when doing a MAKEALL build: $ ./MAKEALL mx28evk Configuring for mx28evk - Board: mx28evk, Options: ENV_IS_IN_MMC mxsimage.c:18:25: fatal error: openssl/evp.h: No such file or directory Add an entry about the need of installing the 'libssl-dev' package. Signed-off-by: Fabio Estevam fabio.este...@freescale.com --- Changes since v1: - Do not remove the 'elftosb' text. doc/README.mxs | 5 + 1 file changed, 5 insertions(+) diff --git a/doc/README.mxs b/doc/README.mxs index 5d9e72f..1e5bc5c 100644 --- a/doc/README.mxs +++ b/doc/README.mxs @@ -63,6 +63,11 @@ copy the binary by hand: Make sure the elftosb binary can be found in your $PATH, in this case this means /usr/local/bin/ has to be in your $PATH. +Install the 'libssl-dev' package as well. On a Ubuntu-based distribution, this +package can be installed as follows: + + $ sudo apt-get install libssl-dev + Since when is Ubuntu not a Debian based distro ? :-( I think you should stick with the roots here, use Debian. 2) Compiling U-Boot for a MXS based board --- Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] doc: README.mxs: Add instruction to install 'libssl-dev'
On Mon, Sep 16, 2013 at 10:49 AM, Marek Vasut ma...@denx.de wrote: Since when is Ubuntu not a Debian based distro ? :-( I think you should stick with the roots here, use Debian. Yes, I wrote Debian-based in v1. The change here was not on purpose, so I can fix it in v3. Actually, I have been thinking about this fix: wouldn't it be better to copy the 'openssl/evp.h' header into U-boot tool directory to avoid people getting failures with MAKEALL if they do not have libssl-dev installed? ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] doc: README.mxs: Add instruction to install 'libssl-dev'
Dear Fabio Estevam, On Mon, Sep 16, 2013 at 10:49 AM, Marek Vasut ma...@denx.de wrote: Since when is Ubuntu not a Debian based distro ? :-( I think you should stick with the roots here, use Debian. Yes, I wrote Debian-based in v1. The change here was not on purpose, so I can fix it in v3. Actually, I have been thinking about this fix: wouldn't it be better to copy the 'openssl/evp.h' header into U-boot tool directory to avoid people getting failures with MAKEALL if they do not have libssl-dev installed? No, because they won't be able to link without the dev package anyway. Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] doc: README.mxs: Add instruction to install 'libssl-dev'
Dear Stefano Babic, On 16/09/2013 16:19, Fabio Estevam wrote: On Mon, Sep 16, 2013 at 10:49 AM, Marek Vasut ma...@denx.de wrote: Since when is Ubuntu not a Debian based distro ? :-( I think you should stick with the roots here, use Debian. Yes, I wrote Debian-based in v1. The change here was not on purpose, so I can fix it in v3. Actually, I have been thinking about this fix: wouldn't it be better to copy the 'openssl/evp.h' header into U-boot tool directory to avoid people getting failures with MAKEALL if they do not have libssl-dev installed? I disagree : it is not different if another include file is missing, or the host is not installed. Then the hint is to install build-essential (Ubuntu) or all packages, not to move them inside u-boot. Hey, build-essential is also Debian (lol) ! ;-) Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] doc: README.mxs: Add instruction to install 'libssl-dev'
On 16/09/2013 16:19, Fabio Estevam wrote: On Mon, Sep 16, 2013 at 10:49 AM, Marek Vasut ma...@denx.de wrote: Since when is Ubuntu not a Debian based distro ? :-( I think you should stick with the roots here, use Debian. Yes, I wrote Debian-based in v1. The change here was not on purpose, so I can fix it in v3. Actually, I have been thinking about this fix: wouldn't it be better to copy the 'openssl/evp.h' header into U-boot tool directory to avoid people getting failures with MAKEALL if they do not have libssl-dev installed? I disagree : it is not different if another include file is missing, or the host is not installed. Then the hint is to install build-essential (Ubuntu) or all packages, not to move them inside u-boot. Regards, Stefano -- = DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] doc: README.mxs: Add instruction to install 'libssl-dev'
On Mon, Sep 16, 2013 at 12:55 PM, Marek Vasut ma...@denx.de wrote: Dear Stefano Babic, On 16/09/2013 16:19, Fabio Estevam wrote: On Mon, Sep 16, 2013 at 10:49 AM, Marek Vasut ma...@denx.de wrote: Since when is Ubuntu not a Debian based distro ? :-( I think you should stick with the roots here, use Debian. Yes, I wrote Debian-based in v1. The change here was not on purpose, so I can fix it in v3. Actually, I have been thinking about this fix: wouldn't it be better to copy the 'openssl/evp.h' header into U-boot tool directory to avoid people getting failures with MAKEALL if they do not have libssl-dev installed? I disagree : it is not different if another include file is missing, or the host is not installed. Then the hint is to install build-essential (Ubuntu) or all packages, not to move them inside u-boot. Hey, build-essential is also Debian (lol) ! ;-) Debian +1 :-) -- Otavio Salvador O.S. Systems http://www.ossystems.com.brhttp://code.ossystems.com.br Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot