Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Saul Wold <sgw@linux.intel.com>
To: Yasir Khan <yasir_khan@mentor.com>,
	 openembedded-core@lists.openembedded.org
Subject: Re: [PATCH V2] openssl.inc: Install c_rehash utility with openssl
Date: Mon, 16 Dec 2013 12:23:00 -0800	[thread overview]
Message-ID: <52AF6124.7040101@linux.intel.com> (raw)
In-Reply-To: <1387194688-11022-1-git-send-email-yasir_khan@mentor.com>

On 12/16/2013 03:51 AM, Yasir Khan wrote:
> From: mykhani <yasir_khan@mentor.com>
>
> c_rehash utility is not being installed with openssl.It conveniently
> generates hash and symbolic links based on it for CA certificates
> stored locally for SSL based server authentication
>
> Signed-off-by: mykhani <yasir_khan@mentor.com>
> ---
>   meta/recipes-connectivity/openssl/openssl.inc |   11 ++++++++++-
>   1 file changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/meta/recipes-connectivity/openssl/openssl.inc b/meta/recipes-connectivity/openssl/openssl.inc
> index 78ff7ae..5cb67df 100644
> --- a/meta/recipes-connectivity/openssl/openssl.inc
> +++ b/meta/recipes-connectivity/openssl/openssl.inc
> @@ -34,10 +34,14 @@ export AS = "${CC} -c"
>   inherit pkgconfig siteinfo multilib_header
>
>   PACKAGES =+ "libcrypto libssl ${PN}-misc"
> +
> +#Adding RDEPENDS for perl scripts
> +RDEPENDS_${PN}-misc +="perl"
> +
>   FILES_libcrypto = "${base_libdir}/libcrypto${SOLIBS}"
>   FILES_libssl = "${libdir}/libssl.so.*"
>   FILES_${PN} =+ " ${libdir}/ssl/*"
> -FILES_${PN}-misc = "${libdir}/ssl/misc ${libdir}/ssl/openssl.cnf"
> +FILES_${PN}-misc = "${libdir}/ssl/misc ${libdir}/ssl/openssl.cnf ${bindir}/c_rehash"
>   FILES_${PN}-dev += "${base_libdir}/libcrypto${SOLIBSDEV}"
>

This patch did not apply cleanly, you might be based on an older version 
of this file.  Please rebase against master.

>   do_configure_prepend_darwin () {
> @@ -148,5 +152,10 @@ do_install () {
>   	oe_multilib_header openssl/opensslconf.h
>   }
>
> +do_install_append() {
> +    # The c_rehash utility isn't installed by the normal installation process.
> +    install -m 0755 ${S}/tools/c_rehash ${D}${bindir}
> +}
> +
Also, I think this should just be part of the do_install lower in this file.

Sau!

>   BBCLASSEXTEND = "native nativesdk"
>
>


      reply	other threads:[~2013-12-16 20:23 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-16 11:51 [PATCH V2] openssl.inc: Install c_rehash utility with openssl Yasir Khan
2013-12-16 20:23 ` Saul Wold [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=52AF6124.7040101@linux.intel.com \
    --to=sgw@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=yasir_khan@mentor.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