All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Hatle <mark.hatle@windriver.com>
To: <openembedded-core@lists.openembedded.org>
Subject: FYI -- Re: [PATCH 16/17] lrzsz: sz, sx and sb were linked incorrectly
Date: Tue, 17 Apr 2012 12:03:29 -0500	[thread overview]
Message-ID: <4F8DA261.2050605@windriver.com> (raw)
In-Reply-To: <4da378412f0dceff4b3c3757a185ffa67997599c.1334616144.git.mark.hatle@windriver.com>

Just wanted to highlight this patch.  It should probably go into oe-core before 
the freeze, as the existing lrzsz package is completely broken.  (None of the 
send functions are linked properly and will fail to work.)

Since nobody has reported this, I do find it highly suspect this this component 
is even being used these days.

--Mark

On 4/16/12 6:16 PM, Mark Hatle wrote:
> The sz, sx and sb links were created incorrectly to lrz, they should
> point to lsz.
>
> Signed-off-by: Mark Hatle<mark.hatle@windriver.com>
> ---
>   meta/recipes-bsp/lrzsz/lrzsz_0.12.20.bb |   12 +++++++++---
>   1 files changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/meta/recipes-bsp/lrzsz/lrzsz_0.12.20.bb b/meta/recipes-bsp/lrzsz/lrzsz_0.12.20.bb
> index 414f2f4..7f86157 100644
> --- a/meta/recipes-bsp/lrzsz/lrzsz_0.12.20.bb
> +++ b/meta/recipes-bsp/lrzsz/lrzsz_0.12.20.bb
> @@ -10,7 +10,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
>   			file://src/lrz.c;beginline=1;endline=10;md5=5276956373ff7d8758837f6399a1045f"
>   SECTION = "console/network"
>   DEPENDS = ""
> -PR = "r3"
> +PR = "r4"
>
>   SRC_URI = "http://www.ohse.de/uwe/releases/lrzsz-${PV}.tar.gz \
>   	   file://autotools.patch \
> @@ -28,13 +28,19 @@ do_install() {
>   }
>
>   pkg_postinst_${PN}() {
> -	for util in rz rx rb sz sx sb; do
> +	for util in rz rx rb; do
>   		update-alternatives --install ${bindir}/$util $util lrz 100
>   	done
> +	for util in sz sx sb; do
> +		update-alternatives --install ${bindir}/$util $util lsz 100
> +	done
>   }
>
>   pkg_postrm_${PN}() {
> -	for util in rz rx rb sz sx sb; do
> +	for util in rz rx rb; do
>   		update-alternatives --remove $util ${bindir}/lrz
>   	done
> +	for util sz sx sb; do
> +		update-alternatives --remove $util ${bindir}/lsz
> +	done
>   }




  reply	other threads:[~2012-04-17 17:12 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-16 22:45 [PATCH 00/17] Fix update-alternatives and RPM package dependencies Mark Hatle
2012-04-16 22:45 ` [PATCH 01/17] update-alternatives.bbclass: Ensure alternatives end up in per file deps Mark Hatle
2012-04-17  8:43   ` Richard Purdie
2012-04-17 16:50     ` Mark Hatle
2012-04-16 22:45 ` [PATCH 02/17] coreutils_*.bb: Use update alternatives and add missing manual alt links Mark Hatle
2012-04-16 22:45 ` [PATCH 03/17] coreutils_*.bb: Sync up file path with busybox and minor cleanup Mark Hatle
2012-04-17  7:04   ` Koen Kooi
2012-04-17  7:22   ` Martin Jansa
2012-04-16 22:45 ` [PATCH 04/17] grep: Use update-alternatives Mark Hatle
2012-04-16 22:45 ` [PATCH 05/17] openssh: " Mark Hatle
2012-04-16 23:04 ` [PATCH 06/17] hdparm: " Mark Hatle
2012-04-16 23:05 ` [PATCH 07/17] iputils: " Mark Hatle
2012-04-16 23:15 ` [PATCH 08/17] net-tools: " Mark Hatle
2012-04-16 23:16 ` [PATCH 09/17] shadow: " Mark Hatle
2012-04-16 23:16 ` [PATCH 10/17] findutils: " Mark Hatle
2012-04-16 23:16 ` [PATCH 11/17] gzip: Use update-alternatives class Mark Hatle
2012-04-16 23:16 ` [PATCH 12/17] module-init-tools: Update to use " Mark Hatle
2012-04-16 23:16 ` [PATCH 13/17] kbd: Use update-alternatives Mark Hatle
2012-04-16 23:16 ` [PATCH 14/17] console-tools: " Mark Hatle
2012-04-16 23:16 ` [PATCH 15/17] sysvinit: Use update-alternatives in a different way Mark Hatle
2012-04-16 23:16 ` [PATCH 16/17] lrzsz: sz, sx and sb were linked incorrectly Mark Hatle
2012-04-17 17:03   ` Mark Hatle [this message]
2012-04-16 23:16 ` [PATCH 17/17] lrzsz: Use update-alternatives to set provides Mark Hatle
2012-04-17  7:03 ` [PATCH 00/17] Fix update-alternatives and RPM package dependencies Koen Kooi
2012-04-17 11:20   ` Andreas Oberritter
2012-04-27 21:25 ` Saul Wold

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=4F8DA261.2050605@windriver.com \
    --to=mark.hatle@windriver.com \
    --cc=openembedded-core@lists.openembedded.org \
    /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.