All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.